lpc32xx: Kconfig: switch to CONFIG_CONS_INDEX

There's nothing special or unique to the lpc32xx that requires its own config
parameter for specifying the console uart index. Therefore instead of using
the lpc32xx-specific CONFIG_SYS_LPC32XX_UART include parameter, use the
already-available CONFIG_CONS_INDEX from Kconfig.

Signed-off-by: Trevor Woerner <twoerner@gmail.com>
Reviewed-by: Simon Glass <sjg@chromium.org>

diff --git a/arch/arm/include/asm/arch-lpc32xx/config.h b/arch/arm/include/asm/arch-lpc32xx/config.h
index 0836091..45e46f9 100644 (file)
--- a/arch/arm/include/asm/arch-lpc32xx/config.h
+++ b/arch/arm/include/asm/arch-lpc32xx/config.h
@@ -12,8 +12,8 @@
 /* Basic CPU architecture */
 
 /* UART configuration */
-#if    (CONFIG_SYS_LPC32XX_UART == 1) || (CONFIG_SYS_LPC32XX_UART == 2) || \
-       (CONFIG_SYS_LPC32XX_UART == 7)
+#if    (CONFIG_CONS_INDEX == 1) || (CONFIG_CONS_INDEX == 2) || \
+       (CONFIG_CONS_INDEX == 7)
 #if !defined(CONFIG_LPC32XX_HSUART)
 #define CONFIG_LPC32XX_HSUART
 #endif

diff --git a/arch/arm/mach-lpc32xx/devices.c b/arch/arm/mach-lpc32xx/devices.c
index e1e2e0d..0a4fef2 100644 (file)
--- a/arch/arm/mach-lpc32xx/devices.c
+++ b/arch/arm/mach-lpc32xx/devices.c
@@ -23,8 +23,7 @@ void lpc32xx_uart_init(unsigned int uart_id)
                return;
 
        /* Disable loopback mode, if it is set by S1L bootloader */
-       clrbits_le32(&ctrl->loop,
-                    UART_LOOPBACK(CONFIG_SYS_LPC32XX_UART));
+       clrbits_le32(&ctrl->loop, UART_LOOPBACK(uart_id));
 
        if (uart_id < 3 || uart_id > 6)
                return;

diff --git a/board/timll/devkit3250/devkit3250.c b/board/timll/devkit3250/devkit3250.c
index 3c744b9..9d4ffb0 100644 (file)
--- a/board/timll/devkit3250/devkit3250.c
+++ b/board/timll/devkit3250/devkit3250.c
@@ -38,7 +38,7 @@ void reset_periph(void)
 
 int board_early_init_f(void)
 {
-       lpc32xx_uart_init(CONFIG_SYS_LPC32XX_UART);
+       lpc32xx_uart_init(CONFIG_CONS_INDEX);
        lpc32xx_i2c_init(1);
        lpc32xx_i2c_init(2);
        lpc32xx_ssp_init();

diff --git a/board/timll/devkit3250/devkit3250_spl.c b/board/timll/devkit3250/devkit3250_spl.c
index 47af78a..12e8ae9 100644 (file)
--- a/board/timll/devkit3250/devkit3250_spl.c
+++ b/board/timll/devkit3250/devkit3250_spl.c
@@ -49,7 +49,7 @@ void spl_board_init(void)
        /* First of all silence buzzer controlled by GPO_20 */
        writel((1 << 20), &gpio->p3_outp_clr);
 
-       lpc32xx_uart_init(CONFIG_SYS_LPC32XX_UART);
+       lpc32xx_uart_init(CONFIG_CONS_INDEX);
        preloader_console_init();
 
        ddr_init(&dram_64mb);

diff --git a/board/work-microwave/work_92105/work_92105.c b/board/work-microwave/work_92105/work_92105.c
index bdcecff..5d12f84 100644 (file)
--- a/board/work-microwave/work_92105/work_92105.c
+++ b/board/work-microwave/work_92105/work_92105.c
@@ -37,7 +37,7 @@ void reset_periph(void)
 int board_early_init_f(void)
 {
        /* initialize serial port for console */
-       lpc32xx_uart_init(CONFIG_SYS_LPC32XX_UART);
+       lpc32xx_uart_init(CONFIG_CONS_INDEX);
        /* enable I2C, SSP, MAC, NAND */
        lpc32xx_i2c_init(1); /* only I2C1 has devices, I2C2 has none */
        lpc32xx_ssp_init();

diff --git a/board/work-microwave/work_92105/work_92105_spl.c b/board/work-microwave/work_92105/work_92105_spl.c
index a31553a..d940114 100644 (file)
--- a/board/work-microwave/work_92105/work_92105_spl.c
+++ b/board/work-microwave/work_92105/work_92105_spl.c
@@ -58,7 +58,7 @@ const struct emc_dram_settings dram_128mb = {
 void spl_board_init(void)
 {
        /* initialize serial port for console */
-       lpc32xx_uart_init(CONFIG_SYS_LPC32XX_UART);
+       lpc32xx_uart_init(CONFIG_CONS_INDEX);
        /* initialize console */
        preloader_console_init();
        /* init DDR and NAND to chainload U-Boot */

diff --git a/configs/devkit3250_defconfig b/configs/devkit3250_defconfig
index 93c048c..9ae70f7 100644 (file)
--- a/configs/devkit3250_defconfig
+++ b/configs/devkit3250_defconfig
@@ -51,6 +51,8 @@ CONFIG_PHYLIB=y
 CONFIG_PHY_ADDR_ENABLE=y
 CONFIG_PHY_ADDR=31
 CONFIG_PHY_SMSC=y
+CONFIG_SPECIFY_CONSOLE_INDEX=y
+CONFIG_CONS_INDEX=5
 CONFIG_SYS_NS16550=y
 CONFIG_SPI=y
 CONFIG_USB=y

diff --git a/configs/work_92105_defconfig b/configs/work_92105_defconfig
index c3f666d..e9605ad 100644 (file)
--- a/configs/work_92105_defconfig
+++ b/configs/work_92105_defconfig
@@ -48,5 +48,7 @@ CONFIG_MTD_RAW_NAND=y
 CONFIG_PHYLIB=y
 CONFIG_PHY_ADDR_ENABLE=y
 CONFIG_PHY_SMSC=y
+CONFIG_SPECIFY_CONSOLE_INDEX=y
+CONFIG_CONS_INDEX=5
 CONFIG_SYS_NS16550=y
 CONFIG_SPI=y

diff --git a/include/configs/devkit3250.h b/include/configs/devkit3250.h
index 5d2b77b..921a38c 100644 (file)
--- a/include/configs/devkit3250.h
+++ b/include/configs/devkit3250.h
@@ -31,11 +31,6 @@
                                         - GENERATED_GBL_DATA_SIZE)
 
 /*
- * Serial Driver
- */
-#define CONFIG_SYS_LPC32XX_UART                5   /* UART5 */
-
-/*
  * DMA
  */
 #if !defined(CONFIG_SPL_BUILD)

diff --git a/include/configs/work_92105.h b/include/configs/work_92105.h
index 7874b77..076a1b0 100644 (file)
--- a/include/configs/work_92105.h
+++ b/include/configs/work_92105.h
@@ -36,11 +36,6 @@
                                         - GENERATED_GBL_DATA_SIZE)
 
 /*
- * Serial Driver
- */
-#define CONFIG_SYS_LPC32XX_UART                5   /* UART5 - NS16550 */
-
-/*
  * Ethernet Driver
  */
 

diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
index e549a55..624d851 100644 (file)
--- a/scripts/config_whitelist.txt
+++ b/scripts/config_whitelist.txt
@@ -2734,7 +2734,6 @@ CONFIG_SYS_LOW
 CONFIG_SYS_LOWMEM_BASE
 CONFIG_SYS_LOW_RES_TIMER
 CONFIG_SYS_LPAE_SDRAM_BASE
-CONFIG_SYS_LPC32XX_UART
 CONFIG_SYS_LS1_DDR_BLOCK1_SIZE
 CONFIG_SYS_LS_MC_AIOP_IMG_MAX_LENGTH
 CONFIG_SYS_LS_MC_BOOT_TIMEOUT_MS