linux-kernel - Re: [PATCH] fujitsu-laptop: Rework brightness of eco led

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20160706183108.GA66341@f23x64.localdomain>
Date:	Wed, 6 Jul 2016 11:31:08 -0700
From:	Darren Hart <dvhart@...radead.org>
To:	Jonathan Woithe <jwoithe@...t42.net>
Cc:	Matej Groma <matejgroma@...il.com>,
	platform-driver-x86@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] fujitsu-laptop: Rework brightness of eco led

On Mon, Jul 04, 2016 at 08:59:14PM +0930, Jonathan Woithe wrote:
> On Mon, Jul 04, 2016 at 12:04:12PM +0200, Matej Groma wrote:
> > For the sake of internal consistency, unset maximum brightness of eco
> > led and make it activatable only on values >= LED_FULL.
> > 
> > Signed-off-by: Matej Groma <matejgroma@...il.com>
> 
> Acked-by: Jonathan Woithe <jwoithe@...t42.net>

Queued, thank you.

> 
> > ---
> > Here is a small statistics covering max brightness of platform-x86 leds:
> > 1	other	unset
> > 8	6	7
> > 3/7 leds that have unset max_brightness are activated on nonzero value.
> > The other 4 are fujitsu-laptop leds. Maybe someone from led subsystem
> > would be willing to look at this and standardise guidelines on how and
> > when to use different brightness levels. Hopefully, this could then be
> > reverted.
> > 
> >  drivers/platform/x86/fujitsu-laptop.c | 5 ++---
> >  1 file changed, 2 insertions(+), 3 deletions(-)
> > 
> > diff --git a/drivers/platform/x86/fujitsu-laptop.c b/drivers/platform/x86/fujitsu-laptop.c
> > index 6ce8e78..61f39ab 100644
> > --- a/drivers/platform/x86/fujitsu-laptop.c
> > +++ b/drivers/platform/x86/fujitsu-laptop.c
> > @@ -212,7 +212,6 @@ static void eco_led_set(struct led_classdev *cdev,
> > 
> >  static struct led_classdev eco_led = {
> >   .name = "fujitsu::eco_led",
> > - .max_brightness = 1,
> >   .brightness_get = eco_led_get,
> >   .brightness_set = eco_led_set
> >  };
> > @@ -306,7 +305,7 @@ static void eco_led_set(struct led_classdev *cdev,
> >  	int curr;
> > 
> >  	curr = call_fext_func(FUNC_LEDS, 0x2, ECO_LED, 0x0);
> > -	if (brightness)
> > +	if (brightness >= LED_FULL)
> >  		call_fext_func(FUNC_LEDS, 0x1, ECO_LED, curr | ECO_LED_ON);
> >  	else
> >  		call_fext_func(FUNC_LEDS, 0x1, ECO_LED, curr & ~ECO_LED_ON);
> > @@ -352,7 +351,7 @@ static enum led_brightness eco_led_get(struct led_classdev *cdev)
> >  	enum led_brightness brightness = LED_OFF;
> > 
> >  	if (call_fext_func(FUNC_LEDS, 0x2, ECO_LED, 0x0) & ECO_LED_ON)
> > -		brightness = cdev->max_brightness;
> > +		brightness = LED_FULL;
> > 
> >  	return brightness;
> >  }
> 

-- 
Darren Hart
Intel Open Source Technology Center