On Fri Oct 18 14:34:27 2024 +0100, Lad Prabhakar wrote:
> The media_entity_pads_init() function was previously hardcoded to use a
> magic number for the number of pads. Replace the magic number with the
> ARRAY_SIZE() macro to determine the number of pads dynamically.
> 
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad...@bp.renesas.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart+rene...@ideasonboard.com>
> Link: 
> https://lore.kernel.org/r/20241018133446.223516-5-prabhakar.mahadev-lad...@bp.renesas.com
> Signed-off-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

---

diff --git a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c 
b/drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c
index b6a118ce9fcb..5d1050e685c6 100644
--- a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c
+++ b/drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c
@@ -804,7 +804,8 @@ static int rzg2l_csi2_probe(struct platform_device *pdev)
         */
        csi2->pads[RZG2L_CSI2_SOURCE].flags = MEDIA_PAD_FL_SOURCE |
                                              MEDIA_PAD_FL_MUST_CONNECT;
-       ret = media_entity_pads_init(&csi2->subdev.entity, 2, csi2->pads);
+       ret = media_entity_pads_init(&csi2->subdev.entity, 
ARRAY_SIZE(csi2->pads),
+                                    csi2->pads);
        if (ret)
                goto error_pm;
 

Reply via email to