Julius Werner has submitted this change. ( https://review.coreboot.org/c/coreboot/+/61692 )
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 Reviewed-on: https://review.coreboot.org/c/coreboot/+/61692 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Raul Rangel rrangel@chromium.org --- M src/console/post.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Raul Rangel: Looks good to me, approved
diff --git a/src/console/post.c b/src/console/post.c index 21ab000..1d99a2e 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_INFO, "POST: 0x%02x\n", value);
mainboard_post(value); }