diff options
| author | Vic Yang <victoryang@chromium.org> | 2014-09-24 11:46:24 +0800 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-10-01 06:53:14 +0000 |
| commit | baf3bac6d818e0ea96b6cefdf6088e22315fff5a (patch) | |
| tree | 05071029ae60e922877b7b87be1903c24a10a019 /common/panic_output.c | |
| parent | 28891eaaab07e873ce5ff21006713ad4f8faf975 (diff) | |
| download | chrome-ec-baf3bac6d818e0ea96b6cefdf6088e22315fff5a.tar.gz | |
ryu: Use brief assertion failure message
Currently, when an assertion fails, the error message is like:
ASSERTION FAILURE '1 + 1 == 3' in command_apreset() at common/chipset.c:24
To save flash space, let's add an option to remove the failed
expression and function name. The error message becomes:
ASSERTION FAILURE at common/chipset.c:24
BUG=chrome-os-partner:32203
TEST=make buildall
TEST=Add an assertion and triggers it. Check error message.
BRANCH=None
Change-Id: Ie323d5b43cbff2cd8f6cd5bb46c1f34ecd16bd5e
Signed-off-by: Vic Yang <victoryang@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/219670
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Diffstat (limited to 'common/panic_output.c')
| -rw-r--r-- | common/panic_output.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/common/panic_output.c b/common/panic_output.c index f77277ab33..4f0f22d00d 100644 --- a/common/panic_output.c +++ b/common/panic_output.c @@ -78,6 +78,14 @@ void panic_reboot(void) } #ifdef CONFIG_DEBUG_ASSERT_REBOOTS +#ifdef CONFIG_DEBUG_ASSERT_BRIEF +void panic_assert_fail(const char *fname, int linenum) +{ + panic_printf("\nASSERTION FAILURE at %s:%d\n", fname, linenum); + + panic_reboot(); +} +#else void panic_assert_fail(const char *msg, const char *func, const char *fname, int linenum) { @@ -87,6 +95,7 @@ void panic_assert_fail(const char *msg, const char *func, const char *fname, panic_reboot(); } #endif +#endif void panic(const char *msg) { |