leds: flash: Fix the size of sysfs_groups array
authorJacek Anaszewski <j.anaszewski@samsung.com>
Thu, 12 Mar 2015 15:45:02 +0000 (08:45 -0700)
committerBryan Wu <cooloney@gmail.com>
Mon, 30 Mar 2015 18:55:48 +0000 (11:55 -0700)
LED_FLASH_MAX_SYSFS_GROUPS macro had value that was relevant for
previous version of the patches introducing LED Flash class. Currently
it is required to reserve the room for maximum 4 sysfs groups.
Since the last element of the struct attribute_group array passed to
the function device_create_with_groups has to be NULL, the size of the
array has to be greater by one than maximum allowed number of groups.
Therefore, the name of the macro is being changed to
LED_FLASH_SYSFS_GROUPS_SIZE, to make it more accurrate.

Signed-off-by: Jacek Anaszewski <j.anaszewski@samsung.com>
Acked-by: Kyungmin Park <kyungmin.park@samsung.com>
Cc: Bryan Wu <cooloney@gmail.com>
Cc: Richard Purdie <rpurdie@rpsys.net>
Signed-off-by: Bryan Wu <cooloney@gmail.com>
include/linux/led-class-flash.h

index 21ec91e13c47ba5cd12a12864b6cf7c4f740228a..e97966d1fb8daa30a41dc57c5bdcb6d2d398476c 100644 (file)
@@ -32,7 +32,7 @@ struct led_classdev_flash;
 #define LED_FAULT_LED_OVER_TEMPERATURE (1 << 8)
 #define LED_NUM_FLASH_FAULTS           9
 
-#define LED_FLASH_MAX_SYSFS_GROUPS 7
+#define LED_FLASH_SYSFS_GROUPS_SIZE    5
 
 struct led_flash_ops {
        /* set flash brightness */
@@ -80,7 +80,7 @@ struct led_classdev_flash {
        struct led_flash_setting timeout;
 
        /* LED Flash class sysfs groups */
-       const struct attribute_group *sysfs_groups[LED_FLASH_MAX_SYSFS_GROUPS];
+       const struct attribute_group *sysfs_groups[LED_FLASH_SYSFS_GROUPS_SIZE];
 };
 
 static inline struct led_classdev_flash *lcdev_to_flcdev(