Re: [PATCHv2 13/23] media: use sizeof() instead of V4L2_SUBDEV_NAME_SIZE

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux