[coreboot-gerrit] Patch set updated for coreboot: 6a51478 console/Kconfig: Enable CBMEM console on x86 by default

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Sat Mar 8 13:46:54 CET 2014


Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5350

-gerrit

commit 6a5147833c74bcbeb3fa8f694bb6379d5bb32616
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Sat Mar 8 10:46:52 2014 +0100

    console/Kconfig: Enable CBMEM console on x86 by default
    
    Thanks to Kyösti Mälkki’s Google Summer of Code 2013 work, console
    output to CBMEM (CBMEM console) for ramstage messages works on all x86
    boards. For romstage messages it works for Intel boards from the i945
    chipset on, but it does not work yet for AMD boards.
    
    Enabling CBMEM console on Google Pit and Snow results in compile errors,
    so it is just enabled for non-ARM systems.
    
    The board status script under `util/board_status/` reads the coreboot
    messages from CBMEM, which are then uploaded to the board status
    repository. With CBMEM console disabled by default, currently no
    coreboot console messages are uploaded to the board status repository.
    Enabling CBMEM console by default improves this situation, so that at
    least ramstage messages are stored in the board status repository.
    
    Change-Id: I8d5a58c078325c43a0317bcfaafc722d039aab0b
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 src/console/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/console/Kconfig b/src/console/Kconfig
index bb64f29..86c7cbb 100644
--- a/src/console/Kconfig
+++ b/src/console/Kconfig
@@ -202,7 +202,7 @@ config CONSOLE_NE2K_IO_PORT
 
 config CONSOLE_CBMEM
 	bool "Send console output to a CBMEM buffer"
-	default n
+	default y
 	help
 	  Enable this to save the console output in a CBMEM buffer. This would
 	  allow to see coreboot console output from Linux space.



More information about the coreboot-gerrit mailing list