Hi Tommaso On Mon, Jun 27, 2022 at 05:04:47PM +0200, Tommaso Merciai wrote: > Instead of hardcode OV5693_NUM_SUPPLIES in a define is better use > ARRAY_SIZE function to count the number of supplies from > ov5693_supply_names array > > Signed-off-by: Tommaso Merciai <tommaso.merciai@xxxxxxxxxxxxxxxxxxxx> Reviewed-by: Jacopo Mondi <jacopo@xxxxxxxxxx> thanks j > --- > drivers/media/i2c/ov5693.c | 15 +++++++-------- > 1 file changed, 7 insertions(+), 8 deletions(-) > > diff --git a/drivers/media/i2c/ov5693.c b/drivers/media/i2c/ov5693.c > index 117ff5403312..f410333c4c67 100644 > --- a/drivers/media/i2c/ov5693.c > +++ b/drivers/media/i2c/ov5693.c > @@ -127,11 +127,15 @@ > #define OV5693_LINK_FREQ_419_2MHZ 419200000 > #define OV5693_PIXEL_RATE 167680000 > > -/* Miscellaneous */ > -#define OV5693_NUM_SUPPLIES 2 > - > #define to_ov5693_sensor(x) container_of(x, struct ov5693_device, sd) > > +static const char * const ov5693_supply_names[] = { > + "avdd", /* Analog power */ > + "dovdd", /* Digital I/O power */ > +}; > + > +#define OV5693_NUM_SUPPLIES ARRAY_SIZE(ov5693_supply_names) > + > struct ov5693_reg { > u32 reg; > u8 val; > @@ -352,11 +356,6 @@ static const s64 link_freq_menu_items[] = { > OV5693_LINK_FREQ_419_2MHZ > }; > > -static const char * const ov5693_supply_names[] = { > - "avdd", > - "dovdd", > -}; > - > static const char * const ov5693_test_pattern_menu[] = { > "Disabled", > "Random Data", > -- > 2.25.1 >