Re: [PATCH 0/3] ACPI / sleep: Support power button wakeup from S2I on recent Dell laptops
From: Jérôme de Bretagne <hidden>
Date: 2017-06-12 20:36:36
Also in:
linux-acpi
From: Jérôme de Bretagne <hidden>
Date: 2017-06-12 20:36:36
Also in:
linux-acpi
I don't see HEBC in the ASL for the 7275. It would be good to provide
this information across both of the BIOS versions (working and non-working).
As well as what
status = acpi_evaluate_integer(handle, "BTNC", NULL, &button_cap);
evaluates as on both too.I'll provide both BTNC and HEBC log results to compare, with both BIOS versions, a bit later today.
Since this platform shipped with Win 8.1 initially rather than Win10, It's possible 5 button array was not part of the INT33D5 spec at that time and wasn't used by the Windows Intel HID filter driver. My suspicion: Win 8.1 didn't support 5 button array definition yet and those events that are normally part of 5 button array (as detected by bit 17) were coming across as regular HID event codes (like you were seeing 0xCE and 0xCF unknown events from debug log when OS was running).
That was my assumption also, since my earlier 3-liner patch was doing this exactly: trying to wake up on a regular 0xCE event. And it did work. Jérome