Shaunak Saha (shaunak.saha@intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16299
-gerrit
commit 79a7e60b730397b4b7ae0fc88fe08df075cbf58a Author: Shaunak Saha shaunak.saha@intel.com Date: Mon Aug 22 22:05:35 2016 -0700
intel/common: Clear wake status bits before sleep
Call power management utility function clear_wake_sts from southbridge_smi_sleep before going to sleep. This is needed to clear the wake status bits in ACPI registers GPE0.
BUG=chrome-os-partner:55583 BRANCH=None TEST=Verified that system goes to sleep on lidclose and powerd_dbus_suspend command issued from inbuild keyboard.
Change-Id: I204a59f8a19137d6a192ea2d89939eefcd5d41ce Signed-off-by: Shaunak Saha shaunak.saha@intel.com --- src/soc/intel/common/smihandler.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/soc/intel/common/smihandler.c b/src/soc/intel/common/smihandler.c index e27752b..ceb73e0 100644 --- a/src/soc/intel/common/smihandler.c +++ b/src/soc/intel/common/smihandler.c @@ -176,7 +176,8 @@ void southbridge_smi_sleep(const struct smm_save_state_ops *save_state_ops) printk(BIOS_DEBUG, "SMI#: ERROR: SLP_TYP reserved\n"); break; } - /* Clear pending wake status bit to avoid immediate wake */ + /* Clear set wake status bit in ACPI registers */ + clear_wake_sts();
/* Tri-state specific GPIOS to avoid leakage during S3/S5 */