diff options
author | Furquan Shaikh <furquan@google.com> | 2016-07-25 17:00:07 -0700 |
---|---|---|
committer | Furquan Shaikh <furquan@google.com> | 2016-07-28 00:40:03 +0200 |
commit | 0d9cd92efb8b1bf2f9ef75d241040fb6955f4930 (patch) | |
tree | a3091130c04a9231bb0852cae48f5b8dfd3fc986 /src/vendorcode/google/chromeos/watchdog.c | |
parent | c66a02634c1d4b0f7929cb9f4c510fff56db0ca6 (diff) |
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 <furquan@google.com>
Reviewed-on: https://review.coreboot.org/15897
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Tested-by: build bot (Jenkins)
Diffstat (limited to 'src/vendorcode/google/chromeos/watchdog.c')
-rw-r--r-- | src/vendorcode/google/chromeos/watchdog.c | 15 |
1 files changed, 14 insertions, 1 deletions
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 <arch/io.h> +#include <assert.h> +#include <bootstate.h> #include <console/console.h> #include <elog.h> #include <reset.h> +#include <symbols.h> #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); } |