On Fri, Jun 07, 2013 at 01:22:39PM +0000, Matthew Garrett wrote: > On Fri, 2013-06-07 at 16:20 +0800, Adam Lee wrote: > > Some new Lenovo or ThinkPad laptops don't have EC controllable LEDs. > > This patch adds their quirks. > > Can you detect this case? If this is a new way of doing things then > you're going to have to add new entries to the quirk list every time > Lenovo release a new machine, which isn't really going to work. Thanks. The difference is their LED handle expects only one argument, not two. Does kernel have a method detecting acpi_handle's arguments number? (checking...) About the new way: Lenovo is going to assign a ACPI method to each operation. Like the volume_toggle button and LED, hardware toggle is "SHDA", getting state is "GSMS", setting state is "SSMS". About the problem: Some models only support the new way but containing internal EC LED interfaces which makes our detecting failed at the same time. The new interfaces accept only one argument(on, off, or blink), we couldn't control LEDs by them because can't specify the LED number. If you give them two arguments, it returns ACPI warning, so I wanna add these "0" quirks. -- Regards, Adam Lee Hardware Enablement -- To unsubscribe from this list: send the line "unsubscribe platform-driver-x86" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
