Re: [PATCH] platform: x86: dell-laptop: Add support for keyboard backlight

["Pali Rohár" <pali.rohar@gmail.com>]

[-- Attachment #1: Type: Text/Plain, Size: 1632 bytes --]

On Thursday 19 February 2015 11:58:29 Gabriele Mazzotta wrote:
> This patch adds the support for the configuration of the
> keyboard backlight on supported Dell laptops.
> 
> With this patch it is possible to set:
> * keyboard backlight level
> * timeout after which the backlight will be automatically
> turned off * input activity triggers (keyboard, touchpad,
> mouse) that enable the backlight * ambient light settings
> 
> The settings are exposed via
> /sys/class/leds/dell::kbd_backlight/
> 
> The code is based on the newly released documentation by Dell
> in the libsmbios project.
> 
> Signed-off-by: Pali Rohár <pali.rohar@gmail.com>
> Signed-off-by: Gabriele Mazzotta <gabriele.mzt@gmail.com>
> Cc: Dan Carpenter <dan.carpenter@oracle.com>
> ---
>  .../ABI/testing/sysfs-platform-dell-laptop         |   69 ++
>  drivers/platform/x86/dell-laptop.c                 | 1089
> +++++++++++++++++++- 2 files changed, 1152 insertions(+), 6
> deletions(-)
>  create mode 100644
> Documentation/ABI/testing/sysfs-platform-dell-laptop
> 

This patch is same as combination of previous, just contains all 
changes in single patch file. Difference between v3.19-rc5 and 
current version is just als code was moved to separate functions 
and als backlight can be enabled via new sysfs attribute (like in 
second patch which was sent).

I tested this patch on top of linus tree (commit 79513d0) and 
keyboard backlight works fine.

Darren, what do you think, can be this patch which brings 
keyboard backlight support finally moved to mainline kernel?

-- 
Pali Rohár
pali.rohar@gmail.com

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]