Stefan Ott has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40620 )
Change subject: mb/lenovo/x200: Add support for ThinkLight ......................................................................
Patch Set 8:
Patch Set 8:
Patch Set 3:
Patch Set 3:
ThinkLight is that light in the display frame, replaced by the keyboard LED light on newer ThinkPads, right? Does that make any difference?
Are there ThinkPads without ThinkLight? Then we could add a Kconfig for that
Yes, that's the light in the display frame. I don't know about newer ThinkPads, but it's possible that they don't have it anymore.
Anyway, I will be reorganizing this change a split it into various small changes for the different models.
I checked now on my x260. /proc/acpi/ibm/light does nothing, while /proc/acpi/ibm/kbdlight is the keyboard background light.
Thank you. That would be consistent with https://www.thinkwiki.org/wiki/ThinkLight which doesn't list the X260.