GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

intel/skylake: Filter suspend well power failure event for Deep Sx 

If Deep Sx is enabled the event log will get entries added on every
power sequence transition indicating that the suspend well has failed.

When a board is using Deep Sx by design this is intended behavior and
just fills the logs with extraneous events.

To make this work the device init state has to be executed first so it
actually enables the Deep Sx policies in the SOC since this code does
not have any hooks back into the devicetree to read the intended setting
from there.

BUG=b:36042662
BRANCH=none
TEST=Perform suspend/resume on Eve device with Deep S3 enabled, and
then check the event log to be sure that it does not contain the
"SUS Power Fail" event.

Change-Id: I3c8242baa63685232025e1dfef5595ec0ec6d14a
Signed-off-by: Duncan Laurie <dlaurie@chromium.org>
Reviewed-on: https://review.coreboot.org/18664
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins)
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
This commit is contained in:
Duncan Laurie 2017-03-07 19:22:24 -08:00
parent cb76d50f0d
commit ac2cbd0ffb
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/soc/intel/skylake/elog.c
View File

@ -79,8 +79,12 @@ static void pch_log_power_and_resets(struct chipset_power_state *ps)
elog_add_event(ELOG_TYPE_POWER_FAIL); elog_add_event(ELOG_TYPE_POWER_FAIL);
/* SUS Well Power Failure */ /* SUS Well Power Failure */
if (ps->gen_pmcon_b & SUS_PWR_FLR) if (ps->gen_pmcon_b & SUS_PWR_FLR) {
elog_add_event(ELOG_TYPE_SUS_POWER_FAIL); /* Do not log SUS_PWR_FLR if waking from deep Sx */
if (!(ps->prev_sleep_state == ACPI_S3 && deep_s3_enabled()) &&
!(ps->prev_sleep_state == ACPI_S5 && deep_s5_enabled()))
elog_add_event(ELOG_TYPE_SUS_POWER_FAIL);
}
/* TCO Timeout */ /* TCO Timeout */
if (ps->prev_sleep_state != ACPI_S3 && if (ps->prev_sleep_state != ACPI_S3 &&
@ -122,4 +126,4 @@ static void pch_log_state(void *unused)
pch_log_wake_source(ps); pch_log_wake_source(ps);
} }
BOOT_STATE_INIT_ENTRY(BS_DEV_INIT, BS_ON_ENTRY, pch_log_state, NULL); BOOT_STATE_INIT_ENTRY(BS_DEV_INIT, BS_ON_EXIT, pch_log_state, NULL);