RE: keyboard backlight max_brightness bug on Dell Latitude E6410

[Gabriel M. Elder]

I've also been updating
https://bugzilla.kernel.org/show_bug.cgi?id=196913 btw.

What source file is responsible for setting this; led-class.c?

I would like to know, so I can tell it to use the correct value:
max_brightness = supported_brightness_levels-1;  //
!actual_max_brightness+1 :)


# smbios-keyboard-ctl --set-level=10; echo $?

Old Keyboard illumination level is: 9
	  Set Trigger Failed. Failed to write config
	 Error Return Code : cbRES1: 0x-2 cbRES2: 0x93512 cbRES3: 0xCFFFF 
0

# smbios-keyboard-ctl --get-status; echo $?
Helper function to print current status of keyboard illumination

 Current status of KeyBoard Illumination setting on your system: 
-------------------------------------------------------------------

 Configured mode state: 
	 Auto: ALS- and input-activity-based On; input-activity based Off

 Your Keyboard will illumination on: 
	Any Keystroke
	Touchpad activity

 Keyboard illumination timeout has bee set at: 10  Seconds

 Current setting of ALS value that turns the light on or off: 18
 Current ALS Reading : 45
 Current keyboard light level : 9
0


-------- Original Message --------
Subject: Re: keyboard backlight max_brightness bug on Dell Latitude
E6410
From: Pali RohÃr <pali.rohar@xxxxxxxxx>
Date: Thu, September 14, 2017 8:54 am
To: "Gabriel M. Elder" <gabriel@xxxxxxxxxxxxxx>, Gabriele Mazzotta
<gabriele.mzt@xxxxxxxxx>
Cc: dvhart@xxxxxxxxxxxxx, andy@xxxxxxxxxxxxx,
platform-driver-x86@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx

Adding Gabriele to thread, IIRC you have machine which uses
"supported keyboard light brightness levels"
Can you look at this bug, if your machine is affected by it too?

Important parts in ouptput:

> ... --info
> Supported Keyboard light brightness levels : 10

> ... --get-status
> Current keyboard light level : 9

Gabriel, can you play with this tool, which values can be set via
--set-level= parameter? Is 10 accepted? --get-status can be used to
check if value was accepted.

On Thursday 14 September 2017 06:35:15 Gabriel M. Elder wrote:
> 
> output from smbios-keyboard-ctl --info:
> 
> Libsmbios version : 2.3.0
> smbios-keyboard-ctl version : 2.3.0
> 
> Capabilities of KeyBoard Illumination on your system: 
> -------------------------------------------------------------------
> Supported USER Selectable Modes : 
> Always OFF
> Auto: ALS- and input-activity-based On; input-activity based Off
> Auto: Input-activity-based On; input-activity based Off
> 
> Supported Keyboard illumination type : Backlight
> 
> Supports Keyboard illumination on : 
> Any Keystroke
> Touchpad activity
> Pointing stick
> 
> Can configure Keyboard illumination timeout unit in : 
> Seconds
> Minutes
> Hours
> 
> Supported Keyboard light brightness levels : 10
> 
> Maximum acceptable seconds timeout value : 255
> 
> Maximum acceptable minutes timeout value : 255
> 
> Maximum acceptable hours timeout value : 12
> 
> Maximum acceptable days timeout value : 0
> 
> 
> output from smbios-keyboard-ctl --get-status:
> 
> Helper function to print current status of keyboard illumination
> 
> Current status of KeyBoard Illumination setting on your system: 
> -------------------------------------------------------------------
> 
> Configured mode state: 
> Auto: Input-activity-based On; input-activity based Off
> 
> Your Keyboard will illumination on: 
> Any Keystroke
> Touchpad activity
> Pointing stick
> 
> Keyboard illumination timeout has bee set at: 10 Seconds
> 
> Current setting of ALS value that turns the light on or off: 18
> Current ALS Reading : 16
> Current keyboard light level : 9
> 
> 
> -------- Original Message --------
> Subject: Re: keyboard backlight max_brightness bug on Dell Latitude
> E6410
> From: Pali RohÃr <pali.rohar@xxxxxxxxx>
> Date: Thu, September 14, 2017 3:06 am
> To: "Gabriel M. Elder" <gabriel@xxxxxxxxxxxxxx>
> Cc: dvhart@xxxxxxxxxxxxx, andy@xxxxxxxxxxxxx,
> platform-driver-x86@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
> 
> On Monday 11 September 2017 23:25:27 Gabriel M. Elder wrote:
> > Hi all,
> > Hans de Goede, one of the upower maintainers, suggested I alert you all to this bug:
> > 
> > https://bugs.freedesktop.org/show_bug.cgi?id=100041
> > 
> > and the new one I filed via the kernel bugzilla:
> > 
> > https://bugzilla.kernel.org/show_bug.cgi?id=196913
> > "keyboard backlight max_brightness value outside allowable range on Dell Latitude E6410 laptop"
> > 
> > Please check it out at your earliest convenience.
> > 
> > thanks,
> > - Gabriel</html>
> 
> Hi Gabriel, please avoid sending such html emails to mailing list as it
> is hard to read them and also you have a very big chance that email
> would be eaten by spam filter or other developers would completely
> ignore it...
> 
> To debug your problem, can you run smbios-keyboard-ctl tool from the
> libsmbios project? https://github.com/dell/libsmbios
> 
> We would need output from --info parameter and also from --get-status.
> 

-- 
Pali RohÃr
pali.rohar@xxxxxxxxx