On Sun, 15 Jan 2017, Hans de Goede wrote: > Do you want me to also send out a new version of the platform patches when > I send the next shot at the LED side of things, or shall I keep those > in my personal tree until the LED api is finalized ? If you don't sent the entire patch set here, please keep me CC'd. You guys seem to have some misconceptions about how the ThinkPad LEDs work. For one, they're LIFO, as in: whomever set their state last, wins. And you have several agents trying to command them: EC itself, SMBIOS (in SMM mode), ACPI DSDT, thinkpad-acpi. Most of the time, changes are event-driven, so you _could_ try to have the last word on those which all such events are visible... Anyway, the fact is that you can never be really ensure they're at the state you want. It is a best-effort kind of thing. Also, they're controlled by EC firmware, not GPIO, so you don't know their physical state (on/off/brightness level): you just know what high-level state the EC is set to (on/off/blink/ramp) and that information is not always available (the LEDs can be write-only, or worse, turn-on-only or turn-off-only at the ACPI DSDT interface level, which is what we typically use in thinkpad-acpi :p). I.e: some thinkpad LEDs are better behaved than others, and more controllable than others. And this changes with firmware model/version. -- Henrique Holschuh -- To unsubscribe from this list: send the line "unsubscribe linux-leds" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
