On Thu Mar 19 01:59:07 2026 +0200, Laurent Pinchart wrote:
> The state initialization function vsp1_entity_init_state() incorrectly
> leaves the last entity pad out when initializing formats due to an off
> by one error. Fix it.
> 
> Tested-by: Lad Prabhakar <[email protected]> # HiHope 
> RZ/G2M
> Link: 
> https://patch.msgid.link/20260318235907.831556-14-laurent.pinchart+rene...@ideasonboard.com
> Signed-off-by: Laurent Pinchart <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/renesas/vsp1/vsp1_entity.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

---

diff --git a/drivers/media/platform/renesas/vsp1/vsp1_entity.c 
b/drivers/media/platform/renesas/vsp1/vsp1_entity.c
index 839b75b62ceb..1dad9589768c 100644
--- a/drivers/media/platform/renesas/vsp1/vsp1_entity.c
+++ b/drivers/media/platform/renesas/vsp1/vsp1_entity.c
@@ -380,7 +380,7 @@ static int vsp1_entity_init_state(struct v4l2_subdev 
*subdev,
        unsigned int pad;
 
        /* Initialize all pad formats with default values. */
-       for (pad = 0; pad < subdev->entity.num_pads - 1; ++pad) {
+       for (pad = 0; pad < subdev->entity.num_pads; ++pad) {
                struct v4l2_subdev_format format = {
                        .pad = pad,
                        .which = sd_state ? V4L2_SUBDEV_FORMAT_TRY
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to