diff options
author | Jack Rosenthal <jrosenth@chromium.org> | 2019-05-13 14:47:38 -0600 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2019-05-16 09:04:10 -0700 |
commit | 822c33d6ff03ee4343c4b2edce8aa70663be6a96 (patch) | |
tree | 77b18c390d967ca91691733ce9e303a94143b628 /chip | |
parent | ebf10d6bd404a7ffe30580b223160926ed1ff036 (diff) | |
download | chrome-ec-822c33d6ff03ee4343c4b2edce8aa70663be6a96.tar.gz |
ish: move panic data into AON ROM
Currently the panic data is placed in a region that conflicts with the
aontaskfw stack. With the extended ROM space, we can put the panic
data in a safer location.
BUG=b:132457636
BRANCH=none
TEST=ran 'crash divzero', observed the panic data copy across reset
without any issues
Change-Id: I876f3f071e000017c8f2ee744838711da928857c
Signed-off-by: Jack Rosenthal <jrosenth@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/1610998
Reviewed-by: Jett Rink <jettrink@chromium.org>
Diffstat (limited to 'chip')
-rw-r--r-- | chip/ish/config_chip.h | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/chip/ish/config_chip.h b/chip/ish/config_chip.h index 9d42f7a0a8..cf0bd75c1a 100644 --- a/chip/ish/config_chip.h +++ b/chip/ish/config_chip.h @@ -54,10 +54,13 @@ + CONFIG_AON_RAM_SIZE \ - CONFIG_AON_ROM_SIZE) -/* Store persistent panic data in AON memory */ -#define CONFIG_PANIC_DATA_SIZE 0xFF -#define CONFIG_PANIC_DATA_BASE (CONFIG_AON_ROM_BASE \ - - CONFIG_PANIC_DATA_SIZE) +/* + * Store persistent panic data in AON memory. There are 256 bytes + * available for ECOS use, and we need two software-defined REG32's at + * the end. + */ +#define CONFIG_PANIC_DATA_BASE CONFIG_AON_ROM_BASE +#define CONFIG_PANIC_DATA_SIZE (256 - (2 * sizeof(uint32_t))) /* System stack size */ #define CONFIG_STACK_SIZE 1024 |