summaryrefslogtreecommitdiff
path: root/src/ec/google/chromeec
diff options
context:
space:
mode:
authorYou-Cheng Syu <youcheng@google.com>2019-03-12 13:02:18 +0800
committerPatrick Georgi <pgeorgi@google.com>2019-08-23 20:23:19 +0000
commit85bb874c9c4cd48b9aad050a4232f5e8d6a7bbad (patch)
tree7640f5b4f66a5cd6590e737d14feecdf2c14dccc /src/ec/google/chromeec
parent8d6ea6a491fb53460762a2f01c7e637a5c7cccc1 (diff)
google/chromeos: Support AP watchdog flag from Chrome EC
After ChromiumOS CL:1293132 and CL:1295890, Chrome EC can store the flag telling if the last reboot was triggered by AP watchdog for some boards (e.g., Kukui). This CL adds a new function google_chromeec_get_ap_watchdog_flag(), which reads the AP watchdog flag from Chrome EC, and updates the tables of reset causes and reset flags. A new Kconfig option CHROMEOS_USE_EC_WATCHDOG_FLAG is added for elog_handle_watchdog_tombstone() to determine if watchdog reset was triggered by the AP watchdog flag from EC instead of the tombstone in AP. BUG=b:109900671,b:118654976 BRANCH=none TEST=test with https://review.coreboot.org/c/coreboot/+/31843 Change-Id: I7a970666a8c6da32ac1c6af8280e808fe7fc106d Signed-off-by: You-Cheng Syu <youcheng@google.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/31834 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Hung-Te Lin <hungte@chromium.org>
Diffstat (limited to 'src/ec/google/chromeec')
-rw-r--r--src/ec/google/chromeec/ec.c39
-rw-r--r--src/ec/google/chromeec/ec.h1
2 files changed, 28 insertions, 12 deletions
diff --git a/src/ec/google/chromeec/ec.c b/src/ec/google/chromeec/ec.c
index c34faaaade..1b0f7eee03 100644
--- a/src/ec/google/chromeec/ec.c
+++ b/src/ec/google/chromeec/ec.c
@@ -704,6 +704,29 @@ retry:
return cec_cmd.cmd_code;
}
+static uint16_t google_chromeec_get_uptime_info(
+ struct ec_response_uptime_info *rsp)
+{
+ struct chromeec_command cmd = {
+ .cmd_code = EC_CMD_GET_UPTIME_INFO,
+ .cmd_version = 0,
+ .cmd_data_in = NULL,
+ .cmd_size_in = 0,
+ .cmd_data_out = rsp,
+ .cmd_size_out = sizeof(*rsp),
+ .cmd_dev_index = 0,
+ };
+ google_chromeec_command(&cmd);
+ return cmd.cmd_code;
+}
+
+bool google_chromeec_get_ap_watchdog_flag(void)
+{
+ struct ec_response_uptime_info rsp;
+ return (!google_chromeec_get_uptime_info(&rsp) &&
+ (rsp.ec_reset_flags & EC_RESET_FLAG_AP_WATCHDOG));
+}
+
int google_chromeec_i2c_xfer(uint8_t chip, uint8_t addr, int alen,
uint8_t *buffer, int len, int is_read)
{
@@ -944,6 +967,7 @@ static const char *reset_cause_to_str(uint16_t cause)
"reset: debug warm reboot",
"reset: at AP's request",
"reset: during EC initialization",
+ "reset: AP watchdog",
};
static const size_t shutdown_cause_begin = 1 << 15;
@@ -997,22 +1021,13 @@ static void google_chromeec_log_uptimeinfo(void)
"usb-resume",
"rdd",
"rbox",
- "security"
+ "security",
+ "ap-watchdog",
};
struct ec_response_uptime_info cmd_resp;
int i, flag, flag_count;
- struct chromeec_command get_uptime_cmd = {
- .cmd_code = EC_CMD_GET_UPTIME_INFO,
- .cmd_version = 0,
- .cmd_data_in = NULL,
- .cmd_size_in = 0,
- .cmd_data_out = &cmd_resp,
- .cmd_size_out = sizeof(cmd_resp),
- .cmd_dev_index = 0,
- };
- google_chromeec_command(&get_uptime_cmd);
- if (get_uptime_cmd.cmd_code) {
+ if (google_chromeec_get_uptime_info(&cmd_resp)) {
/*
* Deliberately say nothing for EC's that don't support this
* command
diff --git a/src/ec/google/chromeec/ec.h b/src/ec/google/chromeec/ec.h
index d7fe7337c3..019f9c1eb0 100644
--- a/src/ec/google/chromeec/ec.h
+++ b/src/ec/google/chromeec/ec.h
@@ -64,6 +64,7 @@ int google_chromeec_kbbacklight(int percent);
void google_chromeec_post(u8 postcode);
int google_chromeec_vbnv_context(int is_read, uint8_t *data, int len);
uint8_t google_chromeec_get_switches(void);
+bool google_chromeec_get_ap_watchdog_flag(void);
/* Temporary secure storage commands */
int google_chromeec_vstore_supported(void);