From 0d9cd92efb8b1bf2f9ef75d241040fb6955f4930 Mon Sep 17 00:00:00 2001 From: Furquan Shaikh Date: Mon, 25 Jul 2016 17:00:07 -0700 Subject: chromeos: Clean up elog handling 1. Currenty, boot reason is being added to elog only for some ARM32/ARM64 platforms. Change this so that boot reason is logged by default in elog for all devices which have CHROMEOS selected. 2. Add a new option to select ELOG_WATCHDOG_RESET for the devices that want to add details about watchdog reset in elog. This requires a special region WATCHDOG to be present in the memlayout. 3. Remove calls to elog add boot reason and watchdog reset from mainboards. BUG=chrome-os-partner:55639 Change-Id: I91ff5b158cfd2a0749e7fefc498d8659f7e6aa91 Signed-off-by: Furquan Shaikh Reviewed-on: https://review.coreboot.org/15897 Reviewed-by: Aaron Durbin Tested-by: build bot (Jenkins) --- src/vendorcode/google/chromeos/watchdog.c | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) (limited to 'src/vendorcode/google/chromeos/watchdog.c') diff --git a/src/vendorcode/google/chromeos/watchdog.c b/src/vendorcode/google/chromeos/watchdog.c index a2b18b75e5..fdaa17761a 100644 --- a/src/vendorcode/google/chromeos/watchdog.c +++ b/src/vendorcode/google/chromeos/watchdog.c @@ -14,24 +14,37 @@ */ #include +#include +#include #include #include #include +#include #include "chromeos.h" #include "symbols.h" #define WATCHDOG_TOMBSTONE_MAGIC 0x9d2f41a7 -void elog_add_watchdog_reset(void) +DECLARE_OPTIONAL_REGION(watchdog_tombstone); + +static void elog_handle_watchdog_tombstone(void *unused) { + if (!_watchdog_tombstone_size) + return; + if (read32(_watchdog_tombstone) == WATCHDOG_TOMBSTONE_MAGIC) elog_add_event(ELOG_TYPE_ASYNC_HW_TIMER_EXPIRED); + write32(_watchdog_tombstone, 0); } +BOOT_STATE_INIT_ENTRY(BS_POST_DEVICE, BS_ON_ENTRY, + elog_handle_watchdog_tombstone, NULL); + void mark_watchdog_tombstone(void) { + assert(_watchdog_tombstone_size); write32(_watchdog_tombstone, WATCHDOG_TOMBSTONE_MAGIC); } -- cgit v1.2.3