Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/52019 )
Change subject: include/cbfsglue.h: Use BIOS_INFO for LOG macro ......................................................................
include/cbfsglue.h: Use BIOS_INFO for LOG macro
The ERROR() and LOG() macros both used BIOS_ERR as the error level. The messages generated by the LOG() macro are informational items. Change to BIOS_INFO to reflect that.
BUG=N/A TEST=tested on facebook monolith
Change-Id: I3827a7d65a9d70045a36fb8db4b2c129e1045122 Signed-off-by: Wim Vervoorn wvervoorn@eltan.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/52019 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Frans Hendriks fhendriks@eltan.com Reviewed-by: Julius Werner jwerner@chromium.org Reviewed-by: Paul Menzel paulepanter@users.sourceforge.net --- M src/include/cbfs_glue.h 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Paul Menzel: Looks good to me, but someone else must approve Julius Werner: Looks good to me, approved Frans Hendriks: Looks good to me, approved
diff --git a/src/include/cbfs_glue.h b/src/include/cbfs_glue.h index ffca83e..3170c37 100644 --- a/src/include/cbfs_glue.h +++ b/src/include/cbfs_glue.h @@ -20,7 +20,7 @@ (CONFIG(TOCTOU_SAFETY) || ENV_INITIAL_STAGE))
#define ERROR(...) printk(BIOS_ERR, "CBFS ERROR: " __VA_ARGS__) -#define LOG(...) printk(BIOS_ERR, "CBFS: " __VA_ARGS__) +#define LOG(...) printk(BIOS_INFO, "CBFS: " __VA_ARGS__) #define DEBUG(...) do { \ if (CONFIG(DEBUG_CBFS)) \ printk(BIOS_SPEW, "CBFS DEBUG: " __VA_ARGS__); \