Skip to content

Commit a0d8f7a

Browse files
Laurent PinchartHans Verkuil
authored andcommitted
Revert "media: renesas: vsp1: Initialize format on all pads"
This reverts commit 133ac42. The change to format initialization, along with the change to format propagation in the BRx in commit 937f3e6 ("media: renesas: vsp1: brx: Fix format propagation"), broke configuration of the DRM pipeline. Revert it to fix the regression. The original commit was meant to fix a v4l2-compliance failure, with no known userspace applications being affected beside test tools. Reverting is the simplest option, a more comprehensive fix can be developed (and tested more thoroughly) later. Fixes: 133ac42 ("media: renesas: vsp1: Initialize format on all pads") Tested-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> # On RZ/T2H Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com> Link: https://patch.msgid.link/20260506215650.1897177-2-laurent.pinchart+renesas@ideasonboard.com Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com> Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>
1 parent 646ebdd commit a0d8f7a

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

drivers/media/platform/renesas/vsp1/vsp1_entity.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -380,7 +380,7 @@ static int vsp1_entity_init_state(struct v4l2_subdev *subdev,
380380
unsigned int pad;
381381

382382
/* Initialize all pad formats with default values. */
383-
for (pad = 0; pad < subdev->entity.num_pads; ++pad) {
383+
for (pad = 0; pad < subdev->entity.num_pads - 1; ++pad) {
384384
struct v4l2_subdev_format format = {
385385
.pad = pad,
386386
.which = sd_state ? V4L2_SUBDEV_FORMAT_TRY

0 commit comments

Comments
 (0)