[SRU] [Xenial] [PATCH 0/1] Fix keyboard led sysfs cannot control its brightness

Stefan Bader stefan.bader at canonical.com
Tue May 30 12:33:18 UTC 2017


On 25.05.2017 05:29, Kai-Heng Feng wrote:
> BugLink: https://bugs.launchpad.net/bugs/1693126
> 
> Impact:
> On some new dell laptops, the keyboard led backlight sysfs cannot control its brightness:
>  dell_laptop: Setting old previous keyboard state failed
>  leds dell::kbd_backlight: Setting an LED's brightness failed (-6)
> 
> Fix:
> Commit 9216e0dcb5533a999d544d0af8661118e0588e1d can fix the issue:
>  This patch adds support for retrieving current timeout AC values and also
>  updating them. Current timeout value in sysfs is displayed based on current
>  AC status, like current display brightness value.
> 
> The patch uses dell_smbios_find_token() which is a refactored version of
> find_token_location(). I use the latter one to avoid unnecessary patch series.
> 
> Test:
> Keyboad backlight now can be correctly controlled by sysfs
> "dell:kbd_backlight/brightness" on Dell Precision 3520.
> 
> Pali Rohár (1):
>   platform/x86: dell-laptop: Add keyboard backlight timeout AC settings
> 
>  drivers/platform/x86/dell-laptop.c | 59 ++++++++++++++++++++++++++++++++++----
>  1 file changed, 53 insertions(+), 6 deletions(-)
> 

This change is relate to "Support new chassis type for dell-laptop", so it would
be better to group both patches together. It is also a bit weird that this one
is only asked for Xenial and the other one for Yakkety and Zesty, too.

-Stefan

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20170530/4f1a41c3/attachment.sig>


More information about the kernel-team mailing list