diff options
| author | Daniel Kurtz <djkurtz@chromium.org> | 2018-05-24 15:52:45 -0600 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2018-05-25 16:37:07 +0000 |
| commit | b6fdd224569f3ff0073bfe1ecfd110d9d30ec536 (patch) | |
| tree | 3020d53fe8fa52f213403d0749e47dc3abf49337 | |
| parent | 9b26127a45503bf8f048b417df7e40184346ef18 (diff) | |
soc/amd/stoneyridge: Record ACPI Wake events in ELOG
We are already reporting the Wake source, but we must also report the
ACPI wake itself.
BUG=b:79865267
TEST=firmware_EventLog
Change-Id: Id26dff46379800a63ab9b77f135d23c6382f77e6
Signed-off-by: Daniel Kurtz <djkurtz@chromium.org>
Reviewed-on: https://review.coreboot.org/26522
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | src/soc/amd/stoneyridge/southbridge.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/soc/amd/stoneyridge/southbridge.c b/src/soc/amd/stoneyridge/southbridge.c index f01f5344d7..5a3a442415 100644 --- a/src/soc/amd/stoneyridge/southbridge.c +++ b/src/soc/amd/stoneyridge/southbridge.c @@ -669,6 +669,10 @@ static void sb_log_pm1_status(uint16_t pm1_sts) if (!IS_ENABLED(CONFIG_ELOG)) return; + if (pm1_sts & WAK_STS) + elog_add_event_byte(ELOG_TYPE_ACPI_WAKE, + acpi_is_wakeup_s3() ? ACPI_S3 : ACPI_S5); + if (pm1_sts & PWRBTN_STS) elog_add_event_wake(ELOG_WAKE_SOURCE_PWRBTN, 0); |