diff options
author | Randall Spangler <rspangler@chromium.org> | 2012-10-26 09:29:40 -0700 |
---|---|---|
committer | Gerrit <chrome-bot@google.com> | 2012-10-29 10:36:05 -0700 |
commit | fc6b412589742976db12de4aa64137c03bfbf311 (patch) | |
tree | b32b72aeef1284e9c2d8354d68cc8ef0694712e6 /include/panic.h | |
parent | bff5a49e6d06c13e67b1a72470dc1d9a2326eb16 (diff) | |
download | chrome-ec-fc6b412589742976db12de4aa64137c03bfbf311.tar.gz |
Consolidate emergency debug output
This removes the duplicate uart_emergency_printf() vs. panic_printf()
/ uart_emergency_puts() vs. panic_puts() implementation and saves
~0.5kb of code size.
The other significant change is that uart_flush_output() is now smart
enough to determine if it's in an interrupt; if so, it will spin-flush
the output buffer instead of waiting on the uart interrupt. This
removes the need for a separate panic_flush().
BUG=chrome-os-partner:15579
BRANCH=none
TEST=crash unaligned; should print well-formatted crash dump
Change-Id: Ifae756203dd1881806be563308077c1d68302e1f
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/36695
Diffstat (limited to 'include/panic.h')
-rw-r--r-- | include/panic.h | 27 |
1 files changed, 0 insertions, 27 deletions
diff --git a/include/panic.h b/include/panic.h index bb3e853bd2..b003f188c8 100644 --- a/include/panic.h +++ b/include/panic.h @@ -48,16 +48,6 @@ struct panic_data { #define PANIC_DATA_FLAG_OLD_HOSTCMD (1 << 2) /** - * Write a character to the panic reporting device - * - * This function will not return until the character has left the UART - * data register. Any previously queued UART traffic is displayed first. - * - * @param ch Character to write - */ -void panic_putc(int ch); - -/** * Write a string to the panic reporting device * * This function will not return until the string has left the UART @@ -68,23 +58,6 @@ void panic_putc(int ch); void panic_puts(const char *s); /** - * Very basic vprintf() for use in panic situations - * - * We only support %s and %nx where n is the number of hex digits to display. - * Currently we don't even support %d, and it is aimed at small code size. - * - * TODO(sjg@chromium.org): Really what we need is a vsnprintf() that is - * shared between the console UART and panic (and is also available as an - * snprintf()). The only downside is that we would then require a large - * printf() implementation to be always present, whereas presumably now we - * can turn it off. - * - * @param format printf-style format string - * @param args List of arguments to process - */ -void panic_vprintf(const char *format, va_list args); - -/** * Very basic printf() for use in panic situations * * See panic_vprintf() for full details |