Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight

From: Darren Hart
Date: Tue Jan 12 2016 - 11:59:28 EST


On Mon, Jan 11, 2016 at 05:28:00PM -0200, Henrique de Moraes Holschuh wrote:
>
> > Henrique, so are you taking back your Ack from 10 minutes prior?
>
> Hmm? No, the ACK stands.
>
> Pavel was talking about another feature altogether, apparently: older
> thinkpads did not have "keyboard backlight" (as in light from below the
> keys). They had a "ThinkLight", which is an overhead light that shines
> down on the keyboard.
>
> The two features are not the same (and are handled differently by the
> firmware, for whatever reason), although they do serve the same purpose.
> I don't think a thinkpad will ever have both features at the same time,
> so I have no idea why they changed the firmware interface.
>
> The patch adds support to the "keyboard backlight" feature, which was
> previously NOT supported.
>
> > I've dropped this patch. Please let me know if I should pick it back up.
>
> Please pick it back up.
>

Gah, these two threads landed next to each other in my Inbox and I didn't pick
up the split. Apologies.

This is queued in testing (again).

--
Darren Hart
Intel Open Source Technology Center