Hello build bot (Jenkins), Marc Jones, Martin Roth, Jonathan Zhang, Marc Jones, Johnny Lin, Paul Menzel, insomniac, Angel Pons, Julius Werner, Kyösti Mälkki, Aaron Durbin,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/45765
to look at the new patch set (#7).
Change subject: console/init: Drop CONSOLE_LEVEL_CONST ......................................................................
console/init: Drop CONSOLE_LEVEL_CONST
We limited the configurability of the debug level to stages that have a `.data` section. This is not really a requirement, because a `.bss` section should suffice and we always have that now.
We want to make the debug level configurable early but also want to avoid calling get_option() early, as an error therein could result in no console output at all. Hence, we compromise and start using get_option() from the second console init on.
TEST=Booted QEMU once with `debug_level=Debug` and once with `debug_level=Notice`. On the second boot, most messages vanished for all stages but the bootblock.
Change-Id: I11484fc32dcbba8d31772bd0b82785f17b2fba11 Signed-off-by: Nico Huber nico.h@gmx.de --- M src/console/init.c 1 file changed, 5 insertions(+), 20 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/65/45765/7