[coreboot-gerrit] Patch set updated for coreboot: a391f98 uart: Redefine Kconfig options

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Mon Apr 7 09:31:20 CEST 2014


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5311

-gerrit

commit a391f98722c7051fbf3ec29027ce5300d81a2763
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Thu Feb 13 17:16:22 2014 +0200

    uart: Redefine Kconfig options
    
    Option DRIVERS_UART builds with support for UART hardware.
    Option CONSOLE_SERIAL enables the console output for UART.
    
    Those x86 boards that do not have serial port on SuperIO should select
    NO_UART_ON_SUPERIO to disable 8250 UART for the default configuration.
    
    Removes:
      CONSOLE_SERIAL_UART
      HAVE_UART_IO_MAPPED
      HAVE_UART_MEMORY_MAPPED
    
    Renames:
      CONSOLE_SERIAL8250     ->  DRIVERS_UART_8250IO
      CONSOLE_SERIAL8250MEM  ->  DRIVERS_UART_8250MEM
    
    Change-Id: Id3afa05f85c0d6849746886db8b6c2ed6c846b61
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/arch/armv7/early_console.c             |  4 +--
 src/arch/x86/lib/romcc_console.c           |  6 ++---
 src/arch/x86/lib/romstage_console.c        | 10 ++-----
 src/console/Kconfig                        | 43 +++++++++++++-----------------
 src/console/uart_console.c                 |  2 +-
 src/cpu/allwinner/a10/Kconfig              |  2 +-
 src/cpu/allwinner/a10/Makefile.inc         | 12 ++++-----
 src/cpu/samsung/exynos5250/Kconfig         |  4 +--
 src/cpu/samsung/exynos5250/Makefile.inc    |  6 ++---
 src/cpu/samsung/exynos5420/Kconfig         |  4 +--
 src/cpu/samsung/exynos5420/Makefile.inc    |  6 ++---
 src/cpu/ti/am335x/Makefile.inc             |  7 +++--
 src/cpu/x86/smm/smiutil.c                  | 10 ++-----
 src/drivers/Makefile.inc                   |  2 +-
 src/drivers/oxford/oxpcie/Kconfig          |  2 +-
 src/drivers/oxford/oxpcie/Makefile.inc     |  2 +-
 src/drivers/uart/Kconfig                   | 15 ++++++++---
 src/drivers/uart/Makefile.inc              | 13 ++++-----
 src/lib/coreboot_table.c                   |  2 +-
 src/mainboard/emulation/qemu-armv7/Kconfig |  2 +-
 src/mainboard/samsung/lumpy/romstage.c     |  4 +--
 src/mainboard/samsung/stumpy/romstage.c    |  4 +--
 src/mainboard/ti/beaglebone/Kconfig        |  4 +--
 src/superio/via/vt1211/vt1211.c            |  2 +-
 24 files changed, 79 insertions(+), 89 deletions(-)

diff --git a/src/arch/armv7/early_console.c b/src/arch/armv7/early_console.c
index 93ea38d..7614531 100644
--- a/src/arch/armv7/early_console.c
+++ b/src/arch/armv7/early_console.c
@@ -28,7 +28,7 @@ void console_tx_byte(unsigned char byte)
 	if (byte == '\n')
 		console_tx_byte('\r');
 
-#if CONFIG_CONSOLE_SERIAL_UART
+#if CONFIG_CONSOLE_SERIAL
 	uart_tx_byte(byte);
 #endif
 #if CONFIG_CONSOLE_CBMEM && !defined(__BOOT_BLOCK__)
@@ -38,7 +38,7 @@ void console_tx_byte(unsigned char byte)
 
 static void _console_tx_flush(void)
 {
-#if CONFIG_CONSOLE_SERIAL_UART
+#if CONFIG_CONSOLE_SERIAL
 	uart_tx_flush();
 #endif
 }
diff --git a/src/arch/x86/lib/romcc_console.c b/src/arch/x86/lib/romcc_console.c
index 66c81db..4bd3ad4 100644
--- a/src/arch/x86/lib/romcc_console.c
+++ b/src/arch/x86/lib/romcc_console.c
@@ -30,7 +30,7 @@
  */
 #define console_loglevel CONFIG_DEFAULT_CONSOLE_LOGLEVEL
 
-#if CONFIG_CONSOLE_SERIAL8250
+#if CONFIG_CONSOLE_SERIAL && CONFIG_DRIVERS_UART_8250IO
 #include "drivers/uart/util.c"
 #include "drivers/uart/uart8250io.c"
 #endif
@@ -40,7 +40,7 @@
 
 static void __console_tx_byte(unsigned char byte)
 {
-#if CONFIG_CONSOLE_SERIAL8250
+#if CONFIG_CONSOLE_SERIAL
 	uart_tx_byte(byte);
 #endif
 #if CONFIG_CONSOLE_NE2K
@@ -61,7 +61,7 @@ static void __console_tx_nibble(unsigned nibble)
 static void __console_tx_char(int loglevel, unsigned char byte)
 {
 	if (console_loglevel >= loglevel) {
-#if CONFIG_CONSOLE_SERIAL8250
+#if CONFIG_CONSOLE_SERIAL
 		uart_tx_byte(byte);
 #endif
 #if CONFIG_CONSOLE_NE2K
diff --git a/src/arch/x86/lib/romstage_console.c b/src/arch/x86/lib/romstage_console.c
index 5e7923e..0337245 100644
--- a/src/arch/x86/lib/romstage_console.c
+++ b/src/arch/x86/lib/romstage_console.c
@@ -31,10 +31,7 @@ void console_tx_byte(unsigned char byte)
 	if (byte == '\n')
 		console_tx_byte('\r');
 
-#if CONFIG_CONSOLE_SERIAL8250MEM
-	uart_tx_byte(byte);
-#endif
-#if CONFIG_CONSOLE_SERIAL8250
+#if CONFIG_CONSOLE_SERIAL
 	uart_tx_byte(byte);
 #endif
 #if CONFIG_CONSOLE_USB && (CONFIG_USBDEBUG_IN_ROMSTAGE || !defined(__PRE_RAM__))
@@ -53,10 +50,7 @@ void console_tx_byte(unsigned char byte)
 
 void console_tx_flush(void)
 {
-#if CONFIG_CONSOLE_SERIAL8250MEM
-	uart_tx_flush();
-#endif
-#if CONFIG_CONSOLE_SERIAL8250
+#if CONFIG_CONSOLE_SERIAL
 	uart_tx_flush();
 #endif
 #if CONFIG_CONSOLE_NE2K
diff --git a/src/console/Kconfig b/src/console/Kconfig
index d77bb9e..65fc1aa 100644
--- a/src/console/Kconfig
+++ b/src/console/Kconfig
@@ -29,37 +29,29 @@ config SQUELCH_EARLY_SMP
 config CONSOLE_SERIAL
 	bool "Serial port console output"
 	default y
+	depends on DRIVERS_UART_8250IO || DRIVERS_UART_8250MEM || HAVE_UART_SPECIAL
 	help
-	  Send coreboot debug output to a serial port (should be one or more of
-	  CONSOLE_SERIAL8250, CONSOLE_SERIAL8250MEM, CONSOLE_SERIAL_UART)
+	  Send coreboot debug output to a serial port.
 
-config CONSOLE_SERIAL8250
-	bool "Serial port console output (I/O mapped, 8250-compatible)"
-	depends on CONSOLE_SERIAL
-	depends on HAVE_UART_IO_MAPPED
-	default y
-	help
-	  Send coreboot debug output to an I/O mapped serial port console.
+	  The type of serial port driver selected based on your configuration is
+	  shown on the following menu line. Supporting multiple different types
+	  of UARTs in one build is not supported.
 
-config CONSOLE_SERIAL8250MEM
-	bool "Serial port console output (memory mapped, 8250-compatible)"
-	depends on CONSOLE_SERIAL
-	depends on HAVE_UART_MEMORY_MAPPED
-	help
-	  Send coreboot debug output to a memory mapped serial port console.
+if CONSOLE_SERIAL
+
+	comment "I/O mapped, 8250-compatible"
+	depends on DRIVERS_UART_8250IO
 
-config CONSOLE_SERIAL_UART
-	bool "Serial port console output (device-specific UART)"
-	depends on CONSOLE_SERIAL
+	comment "memory mapped, 8250-compatible"
+	depends on DRIVERS_UART_8250MEM
+
+	comment "device-specific UART"
 	depends on HAVE_UART_SPECIAL
-	default y
-	help
-	  Send coreboot debug output to a device-specific serial port console.
 
 choice
 	prompt "Serial port for 8250"
 	default CONSOLE_SERIAL_COM1
-	depends on CONSOLE_SERIAL8250
+	depends on DRIVERS_UART_8250IO
 
 config CONSOLE_SERIAL_COM1
 	bool "COM1/ttyS0, I/O port 0x3f8"
@@ -82,7 +74,7 @@ endchoice
 
 config TTYS0_BASE
 	hex
-	depends on CONSOLE_SERIAL8250
+	depends on DRIVERS_UART_8250IO
 	default 0x3f8 if CONSOLE_SERIAL_COM1
 	default 0x2f8 if CONSOLE_SERIAL_COM2
 	default 0x3e8 if CONSOLE_SERIAL_COM3
@@ -93,7 +85,6 @@ config TTYS0_BASE
 choice
 	prompt "Baud rate"
 	default CONSOLE_SERIAL_115200
-	depends on CONSOLE_SERIAL
 
 config CONSOLE_SERIAL_115200
 	bool "115200"
@@ -133,7 +124,9 @@ config TTYS0_BAUD
 config TTYS0_LCS
 	int
 	default 3
-	depends on CONSOLE_SERIAL8250 || CONSOLE_SERIAL8250MEM
+	depends on DRIVERS_UART_8250IO || DRIVERS_UART_8250MEM
+
+endif # CONSOLE_SERIAL
 
 config SPKMODEM
 	bool "spkmodem (console on speaker) console output"
diff --git a/src/console/uart_console.c b/src/console/uart_console.c
index e2cd6a8..f5535d7 100644
--- a/src/console/uart_console.c
+++ b/src/console/uart_console.c
@@ -43,7 +43,7 @@ static unsigned char uartmem_rx_byte(void)
 /* This only relevant with x86 with GDB_STUB enabled.*/
 static int uartmem_tst_byte(void)
 {
-#if CONFIG_CONSOLE_SERIAL8250 || CONFIG_CONSOLE_SERIAL8250MEM
+#if CONFIG_DRIVERS_UART_8250IO || CONFIG_DRIVERS_UART_8250MEM
 	return uart_can_rx_byte();
 #else
 	return 0;
diff --git a/src/cpu/allwinner/a10/Kconfig b/src/cpu/allwinner/a10/Kconfig
index 72e3630..8512c38 100644
--- a/src/cpu/allwinner/a10/Kconfig
+++ b/src/cpu/allwinner/a10/Kconfig
@@ -76,7 +76,7 @@ config SYS_SDRAM_BASE
 choice CONSOLE_SERIAL_UART_CHOICES
 	prompt "Serial Console UART"
 	default CONSOLE_SERIAL_UART0
-	depends on CONSOLE_SERIAL_UART
+	depends on CONSOLE_SERIAL
 
 config CONSOLE_SERIAL_UART0
 	bool "UART0"
diff --git a/src/cpu/allwinner/a10/Makefile.inc b/src/cpu/allwinner/a10/Makefile.inc
index a86d719..716e463 100644
--- a/src/cpu/allwinner/a10/Makefile.inc
+++ b/src/cpu/allwinner/a10/Makefile.inc
@@ -4,8 +4,6 @@ bootblock-y	+= gpio.c
 bootblock-y	+= pinmux.c
 bootblock-y	+= raminit.c
 bootblock-y	+= timer.c
-bootblock-$(CONFIG_BOOTBLOCK_CONSOLE) += uart.c
-bootblock-$(CONFIG_BOOTBLOCK_CONSOLE) += uart_console.c
 
 romstage-y	+= bootblock_media.c
 romstage-y	+= cbmem.c
@@ -13,8 +11,6 @@ romstage-y	+= clock.c
 romstage-y	+= pinmux.c
 romstage-y	+= timer.c
 romstage-y	+= twi.c
-romstage-y	+= uart.c
-romstage-y	+= uart_console.c
 
 ramstage-y	+= bootblock_media.c
 ramstage-y	+= cbmem.c
@@ -23,8 +19,12 @@ ramstage-y	+= cpu.c
 ramstage-y	+= monotonic_timer.c
 ramstage-y	+= timer.c
 ramstage-y	+= twi.c
-ramstage-y	+= uart.c
-ramstage-y	+= uart_console.c
+
+ifeq ($(CONFIG_DRIVERS_UART),y)
+bootblock-$(CONFIG_BOOTBLOCK_CONSOLE) += uart.c uart_console.c
+romstage-y	+= uart.c uart_console.c
+ramstage-y	+= uart.c uart_console.c
+endif
 
 real-target: $(obj)/BOOT0
 
diff --git a/src/cpu/samsung/exynos5250/Kconfig b/src/cpu/samsung/exynos5250/Kconfig
index 63038c3..9320184 100644
--- a/src/cpu/samsung/exynos5250/Kconfig
+++ b/src/cpu/samsung/exynos5250/Kconfig
@@ -91,7 +91,7 @@ config SYS_SDRAM_BASE
 choice CONSOLE_SERIAL_UART_CHOICES
 	prompt "Serial Console UART"
 	default CONSOLE_SERIAL_UART3
-	depends on CONSOLE_SERIAL_UART
+	depends on CONSOLE_SERIAL
 
 config CONSOLE_SERIAL_UART0
 	bool "UART0"
@@ -117,7 +117,7 @@ endchoice
 
 config CONSOLE_SERIAL_UART_ADDRESS
 	hex
-	depends on CONSOLE_SERIAL_UART
+	depends on CONSOLE_SERIAL
 	default 0x12c00000 if CONSOLE_SERIAL_UART0
 	default 0x12c10000 if CONSOLE_SERIAL_UART1
 	default 0x12c20000 if CONSOLE_SERIAL_UART2
diff --git a/src/cpu/samsung/exynos5250/Makefile.inc b/src/cpu/samsung/exynos5250/Makefile.inc
index 6d7ae16..3d5b5b4 100644
--- a/src/cpu/samsung/exynos5250/Makefile.inc
+++ b/src/cpu/samsung/exynos5250/Makefile.inc
@@ -9,7 +9,7 @@ bootblock-y += pinmux.c mct.c power.c
 bootblock-$(CONFIG_BOOTBLOCK_CONSOLE) += clock_init.c
 bootblock-$(CONFIG_BOOTBLOCK_CONSOLE) += clock.c
 bootblock-$(CONFIG_BOOTBLOCK_CONSOLE) += monotonic_timer.c
-ifeq ($(CONFIG_CONSOLE_SERIAL_UART),y)
+ifeq ($(CONFIG_DRIVERS_UART),y)
 bootblock-$(CONFIG_BOOTBLOCK_CONSOLE) += uart.c
 endif
 bootblock-y += wakeup.c
@@ -25,7 +25,7 @@ romstage-y += dmc_init_ddr3.c
 romstage-y += power.c
 romstage-y += mct.c
 romstage-y += monotonic_timer.c
-ifeq ($(CONFIG_CONSOLE_SERIAL_UART),y)
+ifeq ($(CONFIG_DRIVERS_UART),y)
 romstage-$(CONFIG_EARLY_CONSOLE) += uart.c
 endif
 romstage-y += wakeup.c
@@ -40,7 +40,7 @@ ramstage-y += clock.c
 ramstage-y += clock_init.c
 ramstage-y += pinmux.c
 ramstage-y += power.c
-ramstage-$(CONFIG_CONSOLE_SERIAL_UART) += uart.c
+ramstage-$(CONFIG_DRIVERS_UART) += uart.c
 ramstage-y += cpu.c
 ramstage-y += tmu.c
 ramstage-y += mct.c
diff --git a/src/cpu/samsung/exynos5420/Kconfig b/src/cpu/samsung/exynos5420/Kconfig
index dd35c35..5f78cc5 100644
--- a/src/cpu/samsung/exynos5420/Kconfig
+++ b/src/cpu/samsung/exynos5420/Kconfig
@@ -93,7 +93,7 @@ config SYS_SDRAM_BASE
 choice CONSOLE_SERIAL_UART_CHOICES
 	prompt "Serial Console UART"
 	default CONSOLE_SERIAL_UART3
-	depends on CONSOLE_SERIAL_UART
+	depends on CONSOLE_SERIAL
 
 config CONSOLE_SERIAL_UART0
 	bool "UART0"
@@ -119,7 +119,7 @@ endchoice
 
 config CONSOLE_SERIAL_UART_ADDRESS
 	hex
-	depends on CONSOLE_SERIAL_UART
+	depends on CONSOLE_SERIAL
 	default 0x12c00000 if CONSOLE_SERIAL_UART0
 	default 0x12c10000 if CONSOLE_SERIAL_UART1
 	default 0x12c20000 if CONSOLE_SERIAL_UART2
diff --git a/src/cpu/samsung/exynos5420/Makefile.inc b/src/cpu/samsung/exynos5420/Makefile.inc
index 4169a2e..844b835 100644
--- a/src/cpu/samsung/exynos5420/Makefile.inc
+++ b/src/cpu/samsung/exynos5420/Makefile.inc
@@ -9,7 +9,7 @@ bootblock-y += pinmux.c mct.c power.c
 bootblock-$(CONFIG_BOOTBLOCK_CONSOLE) += clock_init.c
 bootblock-$(CONFIG_BOOTBLOCK_CONSOLE) += clock.c
 bootblock-$(CONFIG_BOOTBLOCK_CONSOLE) += monotonic_timer.c
-ifeq ($(CONFIG_CONSOLE_SERIAL_UART),y)
+ifeq ($(CONFIG_DRIVERS_UART),y)
 bootblock-$(CONFIG_BOOTBLOCK_CONSOLE) += uart.c
 endif
 bootblock-y += wakeup.c
@@ -25,7 +25,7 @@ romstage-y += dmc_init_ddr3.c
 romstage-y += power.c
 romstage-y += mct.c
 romstage-y += monotonic_timer.c
-ifeq ($(CONFIG_CONSOLE_SERIAL_UART),y)
+ifeq ($(CONFIG_DRIVERS_UART),y)
 romstage-$(CONFIG_EARLY_CONSOLE) += uart.c
 endif
 romstage-y += wakeup.c
@@ -40,7 +40,7 @@ ramstage-y += clock.c
 ramstage-y += clock_init.c
 ramstage-y += pinmux.c
 ramstage-y += power.c
-ramstage-$(CONFIG_CONSOLE_SERIAL_UART) += uart.c
+ramstage-$(CONFIG_DRIVERS_UART) += uart.c
 ramstage-y += cpu.c
 ramstage-y += tmu.c
 ramstage-y += mct.c
diff --git a/src/cpu/ti/am335x/Makefile.inc b/src/cpu/ti/am335x/Makefile.inc
index ff00733..5b10bd8 100644
--- a/src/cpu/ti/am335x/Makefile.inc
+++ b/src/cpu/ti/am335x/Makefile.inc
@@ -1,16 +1,19 @@
 bootblock-y	+= dmtimer.c
 bootblock-y	+= bootblock_media.c
 bootblock-y	+= pinmux.c
-bootblock-$(CONFIG_EARLY_CONSOLE) += uart.c
 
 romstage-y	+= nand.c
-romstage-y	+= uart.c
 
 ramstage-y	+= dmtimer.c
 ramstage-y	+= monotonic_timer.c
 ramstage-y	+= nand.c
 ramstage-y	+= timer.c
+
+ifeq ($(CONFIG_DRIVERS_UART),y)
+bootblock-$(CONFIG_BOOTBLOCK_CONSOLE) += uart.c
+romstage-y	+= uart.c
 ramstage-y	+= uart.c
+endif
 
 $(call add-class,omap-header)
 
diff --git a/src/cpu/x86/smm/smiutil.c b/src/cpu/x86/smm/smiutil.c
index 3325c32..6e9822e 100644
--- a/src/cpu/x86/smm/smiutil.c
+++ b/src/cpu/x86/smm/smiutil.c
@@ -32,10 +32,7 @@ void console_tx_byte(unsigned char byte)
 	if (byte == '\n')
 		console_tx_byte('\r');
 
-#if CONFIG_CONSOLE_SERIAL8250MEM
-	uart_tx_byte(byte);
-#endif
-#if CONFIG_CONSOLE_SERIAL8250
+#if CONFIG_CONSOLE_SERIAL
 	uart_tx_byte(byte);
 #endif
 }
@@ -45,10 +42,7 @@ void console_init(void)
 {
 #if CONFIG_DEBUG_SMI
 	console_loglevel = CONFIG_DEFAULT_CONSOLE_LOGLEVEL;
-#if CONFIG_CONSOLE_SERIAL8250
-	uart_init();
-#endif
-#if CONFIG_CONSOLE_SERIAL8250MEM
+#if CONFIG_CONSOLE_SERIAL
 	uart_init();
 #endif
 #endif
diff --git a/src/drivers/Makefile.inc b/src/drivers/Makefile.inc
index 197a900..148e257 100644
--- a/src/drivers/Makefile.inc
+++ b/src/drivers/Makefile.inc
@@ -30,7 +30,7 @@ subdirs-y += parade
 subdirs-y += realtek
 subdirs-y += sil
 subdirs-y += trident
-subdirs-y += uart
+subdirs-$(CONFIG_DRIVERS_UART) += uart
 subdirs-y += usb
 subdirs-y += ics
 subdirs-y += spi
diff --git a/src/drivers/oxford/oxpcie/Kconfig b/src/drivers/oxford/oxpcie/Kconfig
index 899a153..5ad42aa 100644
--- a/src/drivers/oxford/oxpcie/Kconfig
+++ b/src/drivers/oxford/oxpcie/Kconfig
@@ -1,7 +1,7 @@
 config DRIVERS_OXFORD_OXPCIE
 	bool "Oxford OXPCIe952"
 	default n
-	select HAVE_UART_MEMORY_MAPPED
+	select DRIVERS_UART_8250MEM
 	help
 	  Support for Oxford OXPCIe952 serial port PCIe cards.
 	  Currently only devices with the vendor ID 0x1415 and device ID
diff --git a/src/drivers/oxford/oxpcie/Makefile.inc b/src/drivers/oxford/oxpcie/Makefile.inc
index 1922425..5e3e6e5 100644
--- a/src/drivers/oxford/oxpcie/Makefile.inc
+++ b/src/drivers/oxford/oxpcie/Makefile.inc
@@ -1,4 +1,4 @@
-ifeq ($(CONFIG_CONSOLE_SERIAL8250MEM),y)
+ifeq ($(CONFIG_CONSOLE_SERIAL),y)
 ramstage-y += oxpcie_early.c oxpcie.c
 romstage-y += oxpcie_early.c
 endif
diff --git a/src/drivers/uart/Kconfig b/src/drivers/uart/Kconfig
index 5c9a9d1..889c69a 100644
--- a/src/drivers/uart/Kconfig
+++ b/src/drivers/uart/Kconfig
@@ -1,10 +1,19 @@
-# This should default to N and be set by SuperI/O drivers that have an UART
-config HAVE_UART_IO_MAPPED
+config DRIVERS_UART
 	bool
+	default y if CONSOLE_SERIAL
+	default n
+
+config DRIVERS_UART_8250IO
+	bool "Serial port on SuperIO"
 	depends on !ARCH_ARMV7
+	default n if NO_UART_ON_SUPERIO
 	default y if ARCH_X86
 
-config HAVE_UART_MEMORY_MAPPED
+# Select this for mainboard without SuperIO serial port.
+config NO_UART_ON_SUPERIO
+	def_bool n
+
+config DRIVERS_UART_8250MEM
 	bool
 	default n
 
diff --git a/src/drivers/uart/Makefile.inc b/src/drivers/uart/Makefile.inc
index 415c5bb..286bff2 100644
--- a/src/drivers/uart/Makefile.inc
+++ b/src/drivers/uart/Makefile.inc
@@ -1,28 +1,25 @@
-ifeq ($(CONFIG_CONSOLE_SERIAL),y)
 romstage-y += util.c
 ramstage-y += util.c
 bootblock-y += util.c
 smm-$(CONFIG_DEBUG_SMI) += util.c
-endif
 
-ifeq ($(CONFIG_CONSOLE_SERIAL8250),y)
+# Add the driver, only one can be enabled. The driver files may
+# be located in the soc/ or cpu/ directories instead of here.
+
+ifeq ($(CONFIG_DRIVERS_UART_8250IO),y)
 romstage-y += uart8250io.c
 ramstage-y += uart8250io.c
 smm-$(CONFIG_DEBUG_SMI) += uart8250io.c
 endif
 
-ifeq ($(CONFIG_CONSOLE_SERIAL8250MEM),y)
+ifeq ($(CONFIG_DRIVERS_UART_8250MEM),y)
 romstage-y += uart8250mem.c
 ramstage-y += uart8250mem.c
 smm-$(CONFIG_DEBUG_SMI) += uart8250mem.c
 endif
 
-ifeq ($(CONFIG_CONSOLE_SERIAL_UART),y)
-
 ifeq ($(CONFIG_DRIVERS_UART_PL011),y)
 bootblock-$(CONFIG_BOOTBLOCK_CONSOLE) += pl011.c
 romstage-$(CONFIG_EARLY_CONSOLE) += pl011.c
 ramstage-y += pl011.c
 endif
-
-endif
diff --git a/src/lib/coreboot_table.c b/src/lib/coreboot_table.c
index e4554d7..8e7f85d 100644
--- a/src/lib/coreboot_table.c
+++ b/src/lib/coreboot_table.c
@@ -390,7 +390,7 @@ unsigned long write_coreboot_table(
 	lb_mainboard(head);
 
 	/* Record the serial ports and consoles */
-#if CONFIG_CONSOLE_SERIAL8250 || CONFIG_CONSOLE_SERIAL8250MEM || CONFIG_CONSOLE_SERIAL_UART
+#if CONFIG_CONSOLE_SERIAL
 	uart_fill_lb(head);
 #endif
 #if CONFIG_CONSOLE_USB
diff --git a/src/mainboard/emulation/qemu-armv7/Kconfig b/src/mainboard/emulation/qemu-armv7/Kconfig
index edba627..040ed9c 100644
--- a/src/mainboard/emulation/qemu-armv7/Kconfig
+++ b/src/mainboard/emulation/qemu-armv7/Kconfig
@@ -50,7 +50,7 @@ config DRAM_SIZE_MB
 
 config CONSOLE_SERIAL_UART_ADDRESS
 	hex
-	depends on CONSOLE_SERIAL_UART
+	depends on CONSOLE_SERIAL
 	default 0x10009000
 
 # Memory map for qemu vexpress-a9:
diff --git a/src/mainboard/samsung/lumpy/romstage.c b/src/mainboard/samsung/lumpy/romstage.c
index 9933459..55c1f0a 100644
--- a/src/mainboard/samsung/lumpy/romstage.c
+++ b/src/mainboard/samsung/lumpy/romstage.c
@@ -40,7 +40,7 @@
 #include <cpu/x86/msr.h>
 #include "option_table.h"
 #include "gpio.h"
-#if CONFIG_CONSOLE_SERIAL8250
+#if CONFIG_DRIVERS_UART_8250IO
 #include "superio/smsc/lpc47n207/lpc47n207.h"
 #include "superio/smsc/lpc47n207/early_serial.c"
 #endif
@@ -53,7 +53,7 @@ static void pch_enable_lpc(void)
 	/* Set COM1/COM2 decode range */
 	pci_write_config16(PCH_LPC_DEV, LPC_IO_DEC, 0x0010);
 
-#if CONFIG_CONSOLE_SERIAL8250
+#if CONFIG_DRIVERS_UART_8250IO
 	/* Enable SuperIO + EC + KBC + COM1 + lpc47n207 config*/
 	pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | MC_LPC_EN |
 		KBC_LPC_EN | CNF2_LPC_EN | COMA_LPC_EN);
diff --git a/src/mainboard/samsung/stumpy/romstage.c b/src/mainboard/samsung/stumpy/romstage.c
index e5b55b5..3e27a6a 100644
--- a/src/mainboard/samsung/stumpy/romstage.c
+++ b/src/mainboard/samsung/stumpy/romstage.c
@@ -40,7 +40,7 @@
 #include <cpu/x86/bist.h>
 #include <cpu/x86/msr.h>
 #include "gpio.h"
-#if CONFIG_CONSOLE_SERIAL8250
+#if CONFIG_DRIVERS_UART_8250IO
 #include "superio/smsc/lpc47n207/lpc47n207.h"
 #include "superio/smsc/lpc47n207/early_serial.c"
 #endif
@@ -62,7 +62,7 @@ static void pch_enable_lpc(void)
 	/* Set COM1/COM2 decode range */
 	pci_write_config16(PCH_LPC_DEV, LPC_IO_DEC, 0x0010);
 
-#if CONFIG_CONSOLE_SERIAL8250
+#if CONFIG_DRIVERS_UART_8250IO
 	/* Enable SuperIO + PS/2 Keyboard/Mouse + COM1 + lpc47n207 config*/
 	pci_write_config16(PCH_LPC_DEV, LPC_EN, CNF1_LPC_EN | KBC_LPC_EN |\
 			   CNF2_LPC_EN | COMA_LPC_EN);
diff --git a/src/mainboard/ti/beaglebone/Kconfig b/src/mainboard/ti/beaglebone/Kconfig
index df7ebeb..c842a5d 100644
--- a/src/mainboard/ti/beaglebone/Kconfig
+++ b/src/mainboard/ti/beaglebone/Kconfig
@@ -64,7 +64,7 @@ config CBFS_HEADER_ROM_OFFSET
 choice CONSOLE_SERIAL_UART_CHOICES
 	prompt "Serial Console UART"
 	default CONSOLE_SERIAL_UART0
-	depends on CONSOLE_SERIAL_UART
+	depends on CONSOLE_SERIAL
 
 config CONSOLE_SERIAL_UART0
 	bool "UART0"
@@ -100,7 +100,7 @@ endchoice
 
 config CONSOLE_SERIAL_UART_ADDRESS
 	hex
-	depends on CONSOLE_SERIAL_UART
+	depends on CONSOLE_SERIAL
 	default 0x44e09000 if CONSOLE_SERIAL_UART0
 	default 0x48022000 if CONSOLE_SERIAL_UART1
 	default 0x48024000 if CONSOLE_SERIAL_UART2
diff --git a/src/superio/via/vt1211/vt1211.c b/src/superio/via/vt1211/vt1211.c
index 551f6f0..f7a5cd9 100644
--- a/src/superio/via/vt1211/vt1211.c
+++ b/src/superio/via/vt1211/vt1211.c
@@ -117,7 +117,7 @@ static void vt1211_pnp_set_resources(struct device *dev)
 {
 	struct resource *res;
 
-#if CONFIG_CONSOLE_SERIAL8250
+#if CONFIG_CONSOLE_SERIAL && CONFIG_DRIVERS_UART_8250IO
 	/* TODO: Do the same for SP2? */
 	if (dev->path.pnp.device == VT1211_SP1) {
 		for (res = dev->resource_list; res; res = res->next) {



More information about the coreboot-gerrit mailing list