On Fri Oct 18 14:34:42 2024 +0100, Lad Prabhakar wrote:
> Make use of `rzg2l_csi2_formats` array in rzg2l_csi2_enum_frame_size() to
> validate if the `fse->code` is supported.
> 
> 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-20-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, 3 insertions(+)

---

diff --git a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c 
b/drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c
index 7a658518b4d6..4073452f3a66 100644
--- a/drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c
+++ b/drivers/media/platform/renesas/rzg2l-cru/rzg2l-csi2.c
@@ -573,6 +573,9 @@ static int rzg2l_csi2_enum_frame_size(struct v4l2_subdev 
*sd,
        if (fse->index != 0)
                return -EINVAL;
 
+       if (!rzg2l_csi2_code_to_fmt(fse->code))
+               return -EINVAL;
+
        fse->min_width = RZG2L_CSI2_MIN_WIDTH;
        fse->min_height = RZG2L_CSI2_MIN_HEIGHT;
        fse->max_width = RZG2L_CSI2_MAX_WIDTH;

Reply via email to