diff options
author | Jacob Garber <jgarber1@ualberta.ca> | 2019-06-03 15:19:47 -0600 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2019-06-11 17:29:02 +0000 |
commit | deb99af8a1bff0ca66cf5764570d5f8e2033039b (patch) | |
tree | 3cb48f05a9cb79d3c60c9b5f65fcacd10aa7f6da /src/console | |
parent | 913437e8a268130b7fbfb9d6d366c42de4f681be (diff) |
console: Allow using vprintk() with disabled console
The prototype of vprintk() is currently declared unconditionally, which
prevents it from being used in situations where the console is disabled.
The code will compile correctly, but not link, since the definition in
console.c isn't being provided. This adds a shim around the declaration
so that, like printk(), a call to vprintk() in this situation will expand
to a no-op function instead.
Change-Id: Ib4a9aa96a5b9dbb9b937ff45854bf6a407938b37
Signed-off-by: Jacob Garber <jgarber1@ualberta.ca>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/33181
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Martin Roth <martinroth@google.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/console')
-rw-r--r-- | src/console/printk.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/console/printk.c b/src/console/printk.c index 09522158a7..8606bbb5c9 100644 --- a/src/console/printk.c +++ b/src/console/printk.c @@ -45,7 +45,7 @@ static void wrap_putchar_cbmemc(unsigned char byte, void *data) __cbmemc_tx_byte(byte); } -int vprintk(int msg_level, const char *fmt, va_list args) +int do_vprintk(int msg_level, const char *fmt, va_list args) { int i, log_this; @@ -91,7 +91,7 @@ int do_printk(int msg_level, const char *fmt, ...) int i; va_start(args, fmt); - i = vprintk(msg_level, fmt, args); + i = do_vprintk(msg_level, fmt, args); va_end(args); return i; |