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> I wanted to test it to be extra safe but discovered a regression that broke probing of the VI driver, so I am now investigating that. Luca -- Luca Ceresoli, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
