From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935919AbdADUAQ (ORCPT ); Wed, 4 Jan 2017 15:00:16 -0500 Received: from mail-wj0-f195.google.com ([209.85.210.195]:34052 "EHLO mail-wj0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934405AbdADT7r (ORCPT ); Wed, 4 Jan 2017 14:59:47 -0500 Subject: Re: [PATCH] leds: add LED_ON brightness as boolean value To: Andi Shyti , Richard Purdie , Pavel Machek References: <20170104134107.24902-1-andi.shyti@samsung.com> Cc: linux-leds@vger.kernel.org, linux-kernel@vger.kernel.org, Andi Shyti From: Jacek Anaszewski X-Enigmail-Draft-Status: N1110 Message-ID: Date: Wed, 4 Jan 2017 20:59:00 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.4.0 MIME-Version: 1.0 In-Reply-To: <20170104134107.24902-1-andi.shyti@samsung.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Andi, Thanks for the patch. On 01/04/2017 02:41 PM, Andi Shyti wrote: > Some devices do not handle the led brightness or simply don't > care about it. Conceptually said devices want to just switch on > or off the led. It is useless in this case to have a 255 range > of brightness, while just having an LED_ON and LED_OFF improves > the boolean meaning of the led status. > > Signed-off-by: Andi Shyti > --- > include/linux/leds.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/include/linux/leds.h b/include/linux/leds.h > index 569cb531094c..0258f9c49034 100644 > --- a/include/linux/leds.h > +++ b/include/linux/leds.h > @@ -27,6 +27,7 @@ struct device; > > enum led_brightness { > LED_OFF = 0, > + LED_ON, Now lack of explicit value assignment to LED_ON looks weird since it is surrounded by other initializer values. I'd prefer to have "LED_ON = 1," in this line. > LED_HALF = 127, > LED_FULL = 255, > }; > -- Best regards, Jacek Anaszewski