On Mon, 25 Sep 2023 18:51:57 +0200 Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx> wrote: > Hi Hans, > > On Sat, 23 Sep 2023 17:20:57 +0200 > Hans Verkuil <hverkuil-cisco@xxxxxxxxx> wrote: > > > Don't rely on a define, let the compiler use the actual > > field size. > > > > Remove all uses of the V4L2_SUBDEV_NAME_SIZE define and also > > drop the define itself. > > > > Signed-off-by: Hans Verkuil <hverkuil-cisco@xxxxxxxxx> > > Cc: Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx> > > Cc: Maxime Ripard <mripard@xxxxxxxxxx> > > Cc: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx> > > Cc: Sowjanya Komatineni <skomatineni@xxxxxxxxxx> > > Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > > --- > > drivers/media/i2c/msp3400-driver.c | 2 +- > > drivers/media/platform/cadence/cdns-csi2rx.c | 4 ++-- > > drivers/media/platform/cadence/cdns-csi2tx.c | 4 ++-- > > drivers/media/platform/renesas/rcar-isp.c | 2 +- > > drivers/media/platform/renesas/rcar-vin/rcar-csi2.c | 2 +- > > drivers/media/platform/ti/omap3isp/ispstat.c | 2 +- > > drivers/staging/media/tegra-video/csi.c | 4 ++-- > > drivers/staging/media/tegra-video/vip.c | 2 +- > > For tegra-video: > Reviewed-by: Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx> And now also (for patches 11 and 12 as well): Tested-by: Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx> # tegra-video VIP Luca -- Luca Ceresoli, Bootlin Embedded Linux and Kernel engineering https://bootlin.com