Attention is currently required from: Raul Rangel. Hello Raul Rangel,
I'd like you to do a code review. Please visit
https://review.coreboot.org/c/coreboot/+/61692
to review the following change.
Change subject: console/post: Lower post code loglevel to BIOS_INFO ......................................................................
console/post: Lower post code loglevel to BIOS_INFO
Post codes don't signify an emergency error, so they shouldn't be classified as BIOS_EMERG. Now that loglevels are more visible, this misclassification looks pretty glaring. This patch changes them to BIOS_INFO which seems more appropriate for an informational code that is expected to occur in the normal boot flow.
Signed-off-by: Julius Werner jwerner@chromium.org Change-Id: I85c8768232ae0cbf65669a7ee6abd538a3b2d5e1 --- M src/console/post.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/92/61692/1
diff --git a/src/console/post.c b/src/console/post.c index 21ab000..050ed53 100644 --- a/src/console/post.c +++ b/src/console/post.c @@ -19,7 +19,7 @@ arch_post_code(value);
if (CONFIG(CONSOLE_POST)) - printk(BIOS_EMERG, "POST: 0x%02x\n", value); + printk(BIOS_NOTICE, "POST: 0x%02x\n", value);
mainboard_post(value); }