media: ov5693: count num_supplies using array_size
authorTommaso Merciai <tommaso.merciai@amarulasolutions.com>
Wed, 13 Jul 2022 15:05:01 +0000 (16:05 +0100)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Fri, 15 Jul 2022 14:33:08 +0000 (15:33 +0100)
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@amarulasolutions.com>
Reviewed-by: Jacopo Mondi <jacopo@jmondi.org>
Reviewed-by: Daniel Scally <djrscally@gmail.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/media/i2c/ov5693.c

index 117ff54..f410333 100644 (file)
 #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",