Johnny Lin uploaded patch set #9 to this change.

View Change

console: Unify overriding console log level with get_option()

1. Integrate get_console_loglevel() into get_option(), if both
USE_OPTION_TABLE and CONSOLE_OVERRIDE_LOGLEVEL are selected,
log level will be set by USE_OPTION_TABLE.

2. Remove CONSOLE_LEVEL_CONST and let init_log_level() set
log level for all stages. get_log_level() is called by printk
so avoid calling an overridden function in case it impacts
performance.

3.CONFIG_DEFAULT_CONSOLE_LOGLEVEL can still be used when
CONSOLE_OVERRIDE_LOGLEVEL is selected.

Tested=On OCP Delta Lake, coreboot log level can be overridden.

Change-Id: I278e392bed178df7a8cdb90685963c1fedf0bcc4
Signed-off-by: Johnny Lin <johnny_lin@wiwynn.com>
---
M src/console/Kconfig
M src/console/Makefile.inc
M src/console/init.c
A src/console/option.c
M src/include/console/console.h
M src/include/option.h
6 files changed, 24 insertions(+), 23 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/34/45134/9

To view, visit change 45134. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I278e392bed178df7a8cdb90685963c1fedf0bcc4
Gerrit-Change-Number: 45134
Gerrit-PatchSet: 9
Gerrit-Owner: Johnny Lin <Johnny_Lin@wiwynn.com>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Jingle Hsu <jingle_hsu@wiwynn.com>
Gerrit-Reviewer: Jonathan Zhang <jonzhang@fb.com>
Gerrit-Reviewer: Marc Jones <marc@marcjonesconsulting.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Morgan Jang <Morgan_Jang@wiwynn.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Rocky Phagura
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Reviewer: insomniac <insomniac@slackware.it>
Gerrit-CC: Nico Huber <nico.h@gmx.de>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-MessageType: newpatchset