From 56d06fdee534124f79b51ff92232373b783cddc2 Mon Sep 17 00:00:00 2001 From: Gabriele Mazzotta Date: Tue, 23 Dec 2014 10:59:23 -0800 Subject: leds: Don't allow brightness values greater than max_brightness Since commit 4d71a4a12b13 ("leds: Add support for setting brightness in a synchronous way") the value passed to brightness_set() is no longer limited to max_brightness and can be different from the internally saved brightness value. Signed-off-by: Gabriele Mazzotta Signed-off-by: Bryan Wu --- drivers/leds/leds.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'drivers/leds/leds.h') diff --git a/drivers/leds/leds.h b/drivers/leds/leds.h index 2348dbda5269..79efe57c7405 100644 --- a/drivers/leds/leds.h +++ b/drivers/leds/leds.h @@ -20,7 +20,8 @@ static inline void led_set_brightness_async(struct led_classdev *led_cdev, enum led_brightness value) { - led_cdev->brightness = min(value, led_cdev->max_brightness); + value = min(value, led_cdev->max_brightness); + led_cdev->brightness = value; if (!(led_cdev->flags & LED_SUSPENDED)) led_cdev->brightness_set(led_cdev, value); -- cgit v1.2.3