From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756328AbaLWS7s (ORCPT ); Tue, 23 Dec 2014 13:59:48 -0500 Received: from mail-wi0-f180.google.com ([209.85.212.180]:49656 "EHLO mail-wi0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751403AbaLWS7r (ORCPT ); Tue, 23 Dec 2014 13:59:47 -0500 From: Gabriele Mazzotta To: cooloney@gmail.com, rpurdie@rpsys.net Cc: linux-leds@vger.kernel.org, linux-kernel@vger.kernel.org, j.anaszewski@samsung.com, kyungmin.park@samsung.com, Gabriele Mazzotta Subject: [PATCH] leds: Don't allow brightness values greater than max_brightness Date: Tue, 23 Dec 2014 19:59:23 +0100 Message-Id: <1419361163-4762-1-git-send-email-gabriele.mzt@gmail.com> X-Mailer: git-send-email 2.1.4 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 --- drivers/leds/leds.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/leds/leds.h b/drivers/leds/leds.h index 2348dbd..79efe57 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); -- 2.1.4