On 11/8/16 5:26 AM, Jacek Anaszewski wrote:
Hi David,


+struct uleds_device {
+    struct uleds_user_dev    user_dev;
+    struct led_classdev    led_cdev;
+    struct mutex        mutex;
+    enum uleds_state    state;
+    wait_queue_head_t    waitq;
+    unsigned char        brightness;

I've just noticed that this is wrong, since LED subsystem
brightness type is enum led_brightness, i.e. int.
LED_FULL (255) value is a legacy enum value that can be overridden
by max_brightness property.

Please submit a fix so that I could merge it with the original
patch before sending it upstream.

Thanks,
Jacek Anaszewski


The brightness should be a 32-bit integer then?

Reply via email to