diff options
author | Ricardo Quesada <ricardoq@google.com> | 2021-07-28 17:15:56 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2021-07-30 20:19:42 +0000 |
commit | d92c21a2ae6544aac372ec9b98a97cab1db6d302 (patch) | |
tree | e33dc9708bde975b8b8249b7a1fe8cf022353920 /core/cortex-m0 | |
parent | 5f8e59c305bd8f9497a7409a0d27e034896e4038 (diff) | |
download | chrome-ec-d92c21a2ae6544aac372ec9b98a97cab1db6d302.tar.gz |
cortex-m*: make watchdog code more readable
This CL makes the code more readable by adding comment / defines
regarding the meaning of PSP[5] and PSP[6].
BUG=None
TEST=make buildall
BRANCH=None
Change-Id: I0cbe7384e20f0287096624a00b0ec19c0fb7f9bd
Signed-off-by: Ricardo Quesada <ricardoq@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3059814
Tested-by: Ricardo Quesada <ricardoq@chromium.org>
Reviewed-by: Aseda Aboagye <aaboagye@chromium.org>
Commit-Queue: Ricardo Quesada <ricardoq@chromium.org>
Auto-Submit: Ricardo Quesada <ricardoq@chromium.org>
Diffstat (limited to 'core/cortex-m0')
-rw-r--r-- | core/cortex-m0/watchdog.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/core/cortex-m0/watchdog.c b/core/cortex-m0/watchdog.c index 20fcc2ab35..9961922ee5 100644 --- a/core/cortex-m0/watchdog.c +++ b/core/cortex-m0/watchdog.c @@ -11,6 +11,14 @@ #include "timer.h" #include "watchdog.h" +/* + * As defined by Armv7-M Reference Manual B1.5.6 "Exception Entry Behavior", + * the structure of the saved context on the stack is: + * r0, r1, r2, r3, r12, lr, pc, psr, ... + */ +#define STACK_IDX_REG_LR 5 +#define STACK_IDX_REG_PC 6 + void watchdog_trace(uint32_t excep_lr, uint32_t excep_sp) { uint32_t psp; @@ -27,12 +35,12 @@ void watchdog_trace(uint32_t excep_lr, uint32_t excep_sp) /* Log PC. If we were in task context, log task id too. */ #ifdef CONFIG_SOFTWARE_PANIC - panic_set_reason(PANIC_SW_WATCHDOG, stack[6], + panic_set_reason(PANIC_SW_WATCHDOG, stack[STACK_IDX_REG_PC], (excep_lr & 0xf) == 1 ? 0xff : task_get_current()); #endif panic_printf("### WATCHDOG PC=%08x / LR=%08x / pSP=%08x ", - stack[6], stack[5], psp); + stack[STACK_IDX_REG_PC], stack[STACK_IDX_REG_LR], psp); if ((excep_lr & 0xf) == 1) panic_puts("(exc) ###\n"); else |