https://bugzilla.kernel.org/show_bug.cgi?id=195651
Bug ID: 195651
Summary: Boot EC - Asus BIOS contain broken DSDT EC but working
ECDT EC
Product: ACPI
Version: 2.5
Kernel Version: 4.11
Hardware: All
OS: Linux
Tree: Mainline
Status: NEW
Severity: normal
Priority: P1
Component: EC
Assignee: [email protected]
Reporter: [email protected]
Regression: No
Created attachment 256185
--> https://bugzilla.kernel.org/attachment.cgi?id=256185&action=edit
acpidump output
The Asus laptops X550VXK/FX502VD/FX502VE have a BIOS bug where the
ECDT (correctly) states that EC events trigger GPE 0x23, but the
DSDT _GPE method (incorrectly) says GPE 0x33.
A cursory glance of the code suggests that this should work regardless
(because it looks like Linux would treat the ECDT and DSDT as two separate
ECs supported simultaneously), but in practice it is not working since the
ECDT is ultimately ignored in favour of the DSDT EC. The sequence of
events is:
1. acpi_ec_ecdt_probe() is called in early boot, and calls
acpi_config_boot_ec(is_ecdt=true) for the ECDT EC.
acpi_config_boot_ec() sets boot_ec to this new EC, and
boot_ec_is_ecdt = true.
2. Later, acpi_ec_dsdt_probe() is called and creates a new ec to represent
the DSDT EC. Then:
/*
* When the DSDT EC is available, always re-configure boot EC to
* have _REG evaluated. _REG can only be evaluated after the
* namespace initialization.
* At this point, the GPE is not fully initialized, so do not to
* handle the events.
*/
ret = acpi_config_boot_ec(ec, ec->handle, false, false);
So the DSDT EC is passed to acpi_config_boot_ec() which does:
/* Unset old boot EC */
if (boot_ec != ec)
acpi_ec_free(boot_ec);
acpi_ec_free() sets boot_ec to NULL.
Further down in acpi_config_boot_ec() we reach:
/* Set new boot EC */
if (!boot_ec) {
boot_ec = ec;
boot_ec_is_ecdt = is_ecdt;
}
So now boot_ec points to the DSDT EC and boot_ec_is_ecdt is false.
3. Later, ecpi_ec_ecdt_start() is called and this looks like it would
enable GPEs on our ECDT, but it bails out because of:
if (!boot_ec_is_ecdt)
return -ENODEV;
The comment I pasted above from acpi_ec_dsdt_probe() says that it is going
to reconfigure the boot EC, but it instead calls acpi_config_boot_ec() on
the newly probed DSDT EC. It seems like the code is not following the
comment here.
Adjusting that code to work with boot_ec adjusts the sequence of events so
that both boot EC and DSDT are treated independently and as a result, we
get EC interrupts firing correctly.
This fixes media keys on the mentioned laptop models.
Thanks to Chris Chiu for diagnosis.
--
You are receiving this mail because:
You are watching the assignee of the bug.
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
acpi-bugzilla mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla
