Skip to content

Commit 531306f

Browse files
committed
drm: xlnx: zynqmp_dpsub: Remplace hardcoded values with ARRAY_SIZE()
Use the ARRAY_SIZE() macro to iterate over arrays, instead of hardcoding their size. This makes the code less error-prone should the array size change. Signed-off-by: Laurent Pinchart <[email protected]>
1 parent 98c4ece commit 531306f

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

drivers/gpu/drm/xlnx/zynqmp_disp.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1222,7 +1222,7 @@ static int zynqmp_disp_create_planes(struct zynqmp_disp *disp)
12221222
unsigned int i, j;
12231223
int ret;
12241224

1225-
for (i = 0; i < ZYNQMP_DISP_NUM_LAYERS; i++) {
1225+
for (i = 0; i < ARRAY_SIZE(disp->layers); i++) {
12261226
struct zynqmp_disp_layer *layer = &disp->layers[i];
12271227
enum drm_plane_type type;
12281228
u32 *drm_formats;
@@ -1293,7 +1293,7 @@ static void zynqmp_disp_destroy_layers(struct zynqmp_disp *disp)
12931293
{
12941294
unsigned int i;
12951295

1296-
for (i = 0; i < ZYNQMP_DISP_NUM_LAYERS; i++)
1296+
for (i = 0; i < ARRAY_SIZE(disp->layers); i++)
12971297
zynqmp_disp_layer_release_dma(disp, &disp->layers[i]);
12981298
}
12991299

@@ -1355,7 +1355,7 @@ static int zynqmp_disp_create_layers(struct zynqmp_disp *disp)
13551355
unsigned int i;
13561356
int ret;
13571357

1358-
for (i = 0; i < ZYNQMP_DISP_NUM_LAYERS; i++) {
1358+
for (i = 0; i < ARRAY_SIZE(disp->layers); i++) {
13591359
struct zynqmp_disp_layer *layer = &disp->layers[i];
13601360

13611361
layer->id = i;
@@ -1592,7 +1592,7 @@ static void zynqmp_disp_map_crtc_to_plane(struct zynqmp_disp *disp)
15921592
u32 possible_crtcs = drm_crtc_mask(&disp->crtc);
15931593
unsigned int i;
15941594

1595-
for (i = 0; i < ZYNQMP_DISP_NUM_LAYERS; i++)
1595+
for (i = 0; i < ARRAY_SIZE(disp->layers); i++)
15961596
disp->layers[i].plane.possible_crtcs = possible_crtcs;
15971597
}
15981598

0 commit comments

Comments
 (0)