summaryrefslogtreecommitdiff
path: root/src/ec/google
diff options
context:
space:
mode:
authorDuncan Laurie <dlaurie@chromium.org>2014-09-29 13:04:06 -0700
committerPatrick Georgi <pgeorgi@google.com>2015-04-02 17:28:12 +0200
commitb12e9becfe47f8f02046274181ce8bae122ab58a (patch)
tree5db0bf92c6931cf6cf6b0e8765db405f2e65d295 /src/ec/google
parent3b6a88ed7e49a71be5399755aa2640985547d34b (diff)
downloadcoreboot-b12e9becfe47f8f02046274181ce8bae122ab58a.tar.xz
chromeec: Fix logging of EC wake events
The EC behavior for reading events from the ACPI interface was broken with this commit: d899fda lpc: ACPI query-next-event drops masked events https://chromium-review.googlesource.com/194935 This is causing no EC wake events to be logged. To make sure they are logged once again set the wake mask before querying for events. Also remove the check for port80 event logging since this is no longer used as we now store the port80 code in CMOS and this is unnecessary commands to do for the resume path. BUG=chrome-os-partner:32462 BRANCH=samus,auron TEST=build and boot on samus, check for EC wake events for keyboard and lid in the event log. Change-Id: Ib46fc00006ff0e5777941fc3ab1d81607359c4cb Signed-off-by: Stefan Reinauer <reinauer@chromium.org> Original-Commit-Id: b4dccc03bdded8411cc1429521579ea006ec58a7 Original-Change-Id: Icdd0c1a37a94e0cbd9fd256172324bf989e6d0dc Original-Signed-off-by: Duncan Laurie <dlaurie@chromium.org> Original-Reviewed-on: https://chromium-review.googlesource.com/220373 Original-Reviewed-by: Aaron Durbin <adurbin@chromium.org> Reviewed-on: http://review.coreboot.org/9215 Tested-by: build bot (Jenkins) Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Diffstat (limited to 'src/ec/google')
-rw-r--r--src/ec/google/chromeec/ec.c33
1 files changed, 6 insertions, 27 deletions
diff --git a/src/ec/google/chromeec/ec.c b/src/ec/google/chromeec/ec.c
index a329b5d937..83c22d3a35 100644
--- a/src/ec/google/chromeec/ec.c
+++ b/src/ec/google/chromeec/ec.c
@@ -379,43 +379,22 @@ u32 google_chromeec_get_wake_mask(void)
EC_CMD_HOST_EVENT_GET_WAKE_MASK);
}
-#if CONFIG_ELOG
-/* Find the last port80 code from the previous boot */
-static u16 google_chromeec_get_port80_last_boot(void)
-{
- struct ec_response_port80_last_boot rsp;
- struct chromeec_command cmd = {
- .cmd_code = EC_CMD_PORT80_LAST_BOOT,
- .cmd_data_out = &rsp,
- .cmd_size_out = sizeof(rsp),
- };
-
- /* Get last port80 code */
- if (google_chromeec_command(&cmd) == 0)
- return rsp.code;
-
- return 0;
-}
-#endif
-
void google_chromeec_log_events(u32 mask)
{
#if CONFIG_ELOG
u8 event;
- u16 code;
-
- /* Find the last port80 code */
- code = google_chromeec_get_port80_last_boot();
+ u32 wake_mask;
- /* Log the last post code only if it is abornmal */
- if (code > 0 && code != POST_OS_BOOT && code != POST_OS_RESUME)
- printk(BIOS_DEBUG, "Chrome EC: Last POST code was 0x%02x\n",
- code);
+ /* Set wake mask so events will be read from ACPI interface */
+ wake_mask = google_chromeec_get_wake_mask();
+ google_chromeec_set_wake_mask(mask);
while ((event = google_chromeec_get_event()) != 0) {
if (EC_HOST_EVENT_MASK(event) & mask)
elog_add_event_byte(ELOG_TYPE_EC_EVENT, event);
}
+
+ google_chromeec_set_wake_mask(wake_mask);
#endif
}