[coreboot-gerrit] New patch to review for coreboot: console: honor CONFIG_POSTCAR_CONSOLE

Aaron Durbin (adurbin@chromium.org) gerrit at coreboot.org
Fri Sep 16 23:33:07 CEST 2016


Aaron Durbin (adurbin at chromium.org) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/16617

-gerrit

commit ec784234d44bd3a818d6a6147d535f177c7987de
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Fri Sep 16 16:20:24 2016 -0500

    console: honor CONFIG_POSTCAR_CONSOLE
    
    The declarations for console_init() were unconditionally
    exposed even though there is a Kconfig option. Correct this
    by honoring the CONFIG_POSTCAR_CONSOLE condition.
    
    BUG=chrome-os-partner:57513
    
    Change-Id: Id45ae3d7c05a9f4ebcf85c446fc68a709513bb0f
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
---
 src/include/console/console.h | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/include/console/console.h b/src/include/console/console.h
index 2392ddf..013ef20 100644
--- a/src/include/console/console.h
+++ b/src/include/console/console.h
@@ -40,8 +40,9 @@ void mainboard_post(u8 value);
 void __attribute__ ((noreturn)) die(const char *msg);
 
 #define __CONSOLE_ENABLE__ \
-	((ENV_BOOTBLOCK && CONFIG_BOOTBLOCK_CONSOLE) || \
-	ENV_VERSTAGE || ENV_ROMSTAGE || ENV_RAMSTAGE || ENV_POSTCAR || \
+	((ENV_BOOTBLOCK && IS_ENABLED(CONFIG_BOOTBLOCK_CONSOLE)) || \
+	(ENV_POSTCAR && IS_ENABLED(CONFIG_POSTCAR_CONSOLE)) || \
+	ENV_VERSTAGE || ENV_ROMSTAGE || ENV_RAMSTAGE || \
 	(ENV_SMM && CONFIG_DEBUG_SMI))
 
 #if __CONSOLE_ENABLE__



More information about the coreboot-gerrit mailing list