Hi Dave On Tue, May 30, 2023 at 06:29:40PM +0100, Dave Stevenson wrote: > The IMX258_FLL_* defines are unused. Remove them. > > Signed-off-by: Dave Stevenson <dave.stevenson@xxxxxxxxxxxxxxx> Reviewed-by: Jacopo Mondi <jacopo.mondi@xxxxxxxxxxxxxxxx> > --- > drivers/media/i2c/imx258.c | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/drivers/media/i2c/imx258.c b/drivers/media/i2c/imx258.c > index 85d73b186111..6028579393b5 100644 > --- a/drivers/media/i2c/imx258.c > +++ b/drivers/media/i2c/imx258.c > @@ -29,12 +29,6 @@ > #define IMX258_VTS_30FPS_VGA 0x034c > #define IMX258_VTS_MAX 0xffff > > -/*Frame Length Line*/ > -#define IMX258_FLL_MIN 0x08a6 > -#define IMX258_FLL_MAX 0xffff > -#define IMX258_FLL_STEP 1 > -#define IMX258_FLL_DEFAULT 0x0c98 > - > /* HBLANK control - read only */ > #define IMX258_PPL_DEFAULT 5352 > > -- > 2.25.1 >