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 <laurent.pinch...@ideasonboard.com>
---
 drivers/gpu/drm/xlnx/zynqmp_disp.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/xlnx/zynqmp_disp.c 
b/drivers/gpu/drm/xlnx/zynqmp_disp.c
index 54aa9772e9b9..4767d3a7929a 100644
--- a/drivers/gpu/drm/xlnx/zynqmp_disp.c
+++ b/drivers/gpu/drm/xlnx/zynqmp_disp.c
@@ -1217,7 +1217,7 @@ static int zynqmp_disp_create_planes(struct zynqmp_disp 
*disp)
        unsigned int i, j;
        int ret;
 
-       for (i = 0; i < ZYNQMP_DISP_NUM_LAYERS; i++) {
+       for (i = 0; i < ARRAY_SIZE(disp->layers); i++) {
                struct zynqmp_disp_layer *layer = &disp->layers[i];
                enum drm_plane_type type;
                u32 *drm_formats;
@@ -1288,7 +1288,7 @@ static void zynqmp_disp_destroy_layers(struct zynqmp_disp 
*disp)
 {
        unsigned int i;
 
-       for (i = 0; i < ZYNQMP_DISP_NUM_LAYERS; i++)
+       for (i = 0; i < ARRAY_SIZE(disp->layers); i++)
                zynqmp_disp_layer_release_dma(disp, &disp->layers[i]);
 }
 
@@ -1350,7 +1350,7 @@ static int zynqmp_disp_create_layers(struct zynqmp_disp 
*disp)
        unsigned int i;
        int ret;
 
-       for (i = 0; i < ZYNQMP_DISP_NUM_LAYERS; i++) {
+       for (i = 0; i < ARRAY_SIZE(disp->layers); i++) {
                struct zynqmp_disp_layer *layer = &disp->layers[i];
 
                layer->id = i;
@@ -1587,7 +1587,7 @@ static void zynqmp_disp_map_crtc_to_plane(struct 
zynqmp_disp *disp)
        u32 possible_crtcs = drm_crtc_mask(&disp->crtc);
        unsigned int i;
 
-       for (i = 0; i < ZYNQMP_DISP_NUM_LAYERS; i++)
+       for (i = 0; i < ARRAY_SIZE(disp->layers); i++)
                disp->layers[i].plane.possible_crtcs = possible_crtcs;
 }
 
-- 
Regards,

Laurent Pinchart

Reply via email to