[coreboot-gerrit] New patch to review for coreboot: SeaBIOS: Add Kconfig option to set verbosity level

Stefan Tauner (stefan.tauner@gmx.at) gerrit at coreboot.org
Sun Jan 8 04:09:44 CET 2017


Stefan Tauner (stefan.tauner at gmx.at) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/18051

-gerrit

commit 5b282665ab9d1f73a071e9b9263c333249e1d8ba
Author: Stefan Tauner <stefan.tauner at gmx.at>
Date:   Sun Jan 8 04:03:23 2017 +0100

    SeaBIOS: Add Kconfig option to set verbosity level
    
    Previously SeaBIOS's default was used (1). This patch defaults to
    coreboot's console level instead which is approximately the same
    verbosity as SeaBIOS and thus what a user would probably expect.
    
    Change-Id: If79e5f40c9380bb527f870eeb7d0cb43faf00beb
    Signed-off-by: Stefan Tauner <stefan.tauner at gmx.at>
---
 payloads/external/Makefile.inc     |  3 ++-
 payloads/external/SeaBIOS/Kconfig  | 10 ++++++++++
 payloads/external/SeaBIOS/Makefile |  7 +++++--
 3 files changed, 17 insertions(+), 3 deletions(-)

diff --git a/payloads/external/Makefile.inc b/payloads/external/Makefile.inc
index e4d3e12..a7b073a 100644
--- a/payloads/external/Makefile.inc
+++ b/payloads/external/Makefile.inc
@@ -81,7 +81,8 @@ payloads/external/SeaBIOS/seabios/out/bios.bin.elf seabios: $(DOTCONFIG)
 			CONFIG_SEABIOS_VGA_COREBOOT=$(CONFIG_SEABIOS_VGA_COREBOOT) \
 			CONFIG_DRIVERS_UART_8250IO=$(CONFIG_DRIVERS_UART_8250IO) \
 			CONFIG_CONSOLE_SERIAL=$(CONFIG_CONSOLE_SERIAL) \
-			CONFIG_TTYS0_BASE=$(CONFIG_TTYS0_BASE)
+			CONFIG_TTYS0_BASE=$(CONFIG_TTYS0_BASE) \
+			CONFIG_SEABIOS_DEBUG_LEVEL=$(CONFIG_SEABIOS_DEBUG_LEVEL)
 
 payloads/external/SeaBIOS/seabios/out/vgabios.bin: seabios
 payloads/external/SeaBIOS/seabios/.config: payloads/external/SeaBIOS/seabios/out/bios.bin.elf
diff --git a/payloads/external/SeaBIOS/Kconfig b/payloads/external/SeaBIOS/Kconfig
index ac4f922..9b5e28f 100644
--- a/payloads/external/SeaBIOS/Kconfig
+++ b/payloads/external/SeaBIOS/Kconfig
@@ -94,4 +94,14 @@ config PAYLOAD_VGABIOS_FILE
 	depends on SEABIOS_VGA_COREBOOT
 	default "payloads/external/SeaBIOS/seabios/out/vgabios.bin"
 
+config SEABIOS_DEBUG_LEVEL
+	prompt "Debug level (verbosity)"
+	default DEFAULT_CONSOLE_LOGLEVEL
+	int
+	help
+	  The higher the number, the more verbose SeaBIOS will be.
+	  The range is not well defined but the typical values range from about
+	  1 to 9 inclusive. The default is to use coreboot's loglevel.
+
+	  Set to zero to disable all output.
 endif
diff --git a/payloads/external/SeaBIOS/Makefile b/payloads/external/SeaBIOS/Makefile
index 4b108d5..47a40d3 100644
--- a/payloads/external/SeaBIOS/Makefile
+++ b/payloads/external/SeaBIOS/Makefile
@@ -60,8 +60,11 @@ else
 	false
 endif
 endif
-	# This shows how to force a previously set .config option *off*
-	#echo "# CONFIG_SMBIOS is not set" >> seabios/.config
+ifneq ($(CONFIG_SEABIOS_DEBUG_LEVEL),)
+	echo "CONFIG_DEBUG_LEVEL=$(CONFIG_SEABIOS_DEBUG_LEVEL)" >> seabios/.config
+endif
+#	This shows how to force a previously set .config option *off*
+#	echo "# CONFIG_SMBIOS is not set" >> seabios/.config
 	$(MAKE) -C seabios olddefconfig OUT=out/
 
 build: config



More information about the coreboot-gerrit mailing list