ACPI: PM: Set up EC GPE for system wakeup from drivers that need it
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 30 Jul 2019 09:55:59 +0000 (11:55 +0200)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 30 Jul 2019 10:26:21 +0000 (12:26 +0200)
commit10a08fd65ec1a68ccd86b19ec822ed5f2e50113f
tree2f3385d230b870c37e2f9067193aaeed73d1adeb
parentb605c44c30b59990e806f930c37bd288b9d901a5
ACPI: PM: Set up EC GPE for system wakeup from drivers that need it

The EC GPE needs to be set up for system wakeup only if there is a
driver depending on it, either intel-hid or intel-vbtn, bound to a
button device that is expected to wake up the system from sleep (such
as the power button on some Dell systems, like the XPS13 9360).  It
doesn't need to be set up for waking up the system from sleep in any
other cases and whether or not it is expected to wake up the system
from sleep doesn't depend on whether or not the LPS0 device is
present in the ACPI namespace.

For this reason, rearrange the ACPI suspend-to-idle code to make the
drivers depending on the EC GPE wakeup take care of setting it up and
decouple that from the LPS0 device handling.

While at it, make intel-hid and intel-vbtn prepare for system wakeup
only if they are allowed to wake up the system from sleep by user
space (via sysfs).

[Note that acpi_ec_mark_gpe_for_wake() and acpi_ec_set_gpe_wake_mask()
 are there to prevent the EC GPE from being disabled by the
 acpi_enable_all_wakeup_gpes() call in acpi_s2idle_prepare(), so on
 systems with either intel-hid or intel-vbtn this change doesn't
 affect any interactions with the hardware or platform firmware.]

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
drivers/acpi/ec.c
drivers/acpi/internal.h
drivers/acpi/sleep.c
drivers/platform/x86/intel-hid.c
drivers/platform/x86/intel-vbtn.c
include/linux/acpi.h
include/linux/suspend.h