Hi Niklas, Thank you for the patch. On Fri, Jun 06, 2025 at 08:26:03PM +0200, Niklas Söderlund wrote: > On Gen2 where sub-devices where not exposed to user-space the filed s/filed/field/ > TB/BT ordering was controlled by a hack in the VIN driver. Before > converting it to media device model where the subdevice is exposed > remove that hack. > > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx> > --- > * Changes since v4 > - Broken out from larger patch. > --- > drivers/media/platform/renesas/rcar-vin/rcar-dma.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/media/platform/renesas/rcar-vin/rcar-dma.c b/drivers/media/platform/renesas/rcar-vin/rcar-dma.c > index 5c08ee2c9807..4fb33359bb0f 100644 > --- a/drivers/media/platform/renesas/rcar-vin/rcar-dma.c > +++ b/drivers/media/platform/renesas/rcar-vin/rcar-dma.c > @@ -700,9 +700,6 @@ static int rvin_setup(struct rvin_dev *vin) > case V4L2_FIELD_INTERLACED: > /* Default to TB */ > vnmc = VNMC_IM_FULL; > - /* Use BT if video standard can be read and is 60 Hz format */ > - if (!vin->info->use_mc && vin->std & V4L2_STD_525_60) > - vnmc = VNMC_IM_FULL | VNMC_FOC; > break; > case V4L2_FIELD_INTERLACED_TB: > vnmc = VNMC_IM_FULL; -- Regards, Laurent Pinchart
