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

[Henrique de Moraes Holschuh <hmh@hmh.eng.br>]

On Tue, Jan 12, 2016, at 14:35, Johannes Stezenbach wrote:
> On Tue, Jan 12, 2016 at 02:07:10PM -0200, Henrique de Moraes Holschuh
> wrote:
> > Is the ACPI AML for fn+space readable enough, or is it trapping directly
> > into SMM?  Because if it interacts with the traditional higher-level
> > ACPI AML we already use to talk to the thinklight (and now to the
> > backlight), it should be relatively easy to fix the driver to better
> > support the x230.
> 
> I have no clue about ACPI, do you have some hints
> how to get the info you want?  I found this:
> 
>   sudo acpidump > acpidata.dat
>   acpixtract -sSSDT acpidata.dat
>   acpixtract -sDSDT acpidata.dat
>   iasl -d DSDT.dat SSDT*.dat
> 
>   ..and then look at the .dsl files that contain the AML assembler. 
> 
> But what to look out for?

Well, you should familiarize yourself with ACPI AML a bit. Then, look
for the ACPI methods that thinkpad-acpi (and the patch to add backlight
support) calls, and try to work out how they work in your thinkpad.

Typical thinkpad firmware behavior is to sometimes have a lower level
implementation, a higher level ACPI API that the O.S. is supposed to
call (and which will call into the lower level implementation, or to the
work itself).

We want to stick to the higher level API, as it is more stable.  Use of
lower-level APIs or direct EC access must be whitelist-restricted.

> PS: failed to mention my x230 kernel was still 4.2.x in case it matters

It doesn't.

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh