[U-Boot] [PATCH 1/3] serial: Migrate CONFIG_ARC_SERIAL to Kconfig
Signed-off-by: Tuomas Tynkkynen tuomas@tuxera.com --- arch/arc/Kconfig | 1 + drivers/serial/Kconfig | 7 +++++++ include/configs/nsim.h | 10 ---------- scripts/config_whitelist.txt | 1 - 4 files changed, 8 insertions(+), 11 deletions(-)
diff --git a/arch/arc/Kconfig b/arch/arc/Kconfig index aee15d5353..fb91412150 100644 --- a/arch/arc/Kconfig +++ b/arch/arc/Kconfig @@ -143,6 +143,7 @@ config TARGET_TB100
config TARGET_NSIM bool "Support standalone nSIM & Free nSIM" + imply ARC_SERIAL
config TARGET_AXS101 bool "Support Synopsys Designware SDP board AXS101" diff --git a/drivers/serial/Kconfig b/drivers/serial/Kconfig index 3d5b2bf15f..0a01399961 100644 --- a/drivers/serial/Kconfig +++ b/drivers/serial/Kconfig @@ -416,6 +416,13 @@ config ALTERA_UART Select this to enable an UART for Altera devices. Please find details on the "Embedded Peripherals IP User Guide" of Altera.
+config ARC_SERIAL + bool "ARC UART support" + depends on DM_SERIAL && ARC + help + Select this to enable the support for UART on some ARC (Synopsys) + FPGA-based boards. + config AR933X_UART bool "QCA/Atheros ar933x UART support" depends on DM_SERIAL && SOC_AR933X diff --git a/include/configs/nsim.h b/include/configs/nsim.h index 58d3e5f7de..e7fb35fe3b 100644 --- a/include/configs/nsim.h +++ b/include/configs/nsim.h @@ -25,16 +25,6 @@ #define CONFIG_SYS_BOOTM_LEN SZ_32M #define CONFIG_SYS_LOAD_ADDR 0x82000000
-/* - * UART configuration - * - */ -#define CONFIG_ARC_SERIAL - -/* - * Command line configuration - */ - /* * Environment settings */ diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt index 997ef771c7..831b4add54 100644 --- a/scripts/config_whitelist.txt +++ b/scripts/config_whitelist.txt @@ -48,7 +48,6 @@ CONFIG_ARCH_RMOBILE_EXTRAM_BOOT CONFIG_ARCH_TEGRA CONFIG_ARCH_USE_BUILTIN_BSWAP CONFIG_ARC_MMU_VER -CONFIG_ARC_SERIAL CONFIG_ARIES_M28_V10 CONFIG_ARMADA100 CONFIG_ARMADA100_FEC
Signed-off-by: Tuomas Tynkkynen tuomas@tuxera.com --- arch/Kconfig | 1 + drivers/serial/Kconfig | 5 +++++ include/configs/M5208EVBE.h | 1 - include/configs/M52277EVB.h | 1 - include/configs/M5235EVB.h | 1 - include/configs/M5249EVB.h | 1 - include/configs/M5253DEMO.h | 1 - include/configs/M5253EVBE.h | 1 - include/configs/M5272C3.h | 1 - include/configs/M5275EVB.h | 1 - include/configs/M5282EVB.h | 1 - include/configs/M53017EVB.h | 1 - include/configs/M5329EVB.h | 1 - include/configs/M5373EVB.h | 1 - include/configs/M54418TWR.h | 1 - include/configs/M54451EVB.h | 1 - include/configs/M54455EVB.h | 1 - include/configs/M5475EVB.h | 1 - include/configs/M5485EVB.h | 1 - include/configs/amcore.h | 1 - include/configs/astro_mcf5373l.h | 1 - include/configs/cobra5272.h | 1 - include/configs/eb_cpu5282.h | 1 - include/configs/stmark2.h | 1 - scripts/config_whitelist.txt | 1 - 25 files changed, 6 insertions(+), 23 deletions(-)
diff --git a/arch/Kconfig b/arch/Kconfig index e599e7a39c..c9310406c7 100644 --- a/arch/Kconfig +++ b/arch/Kconfig @@ -28,6 +28,7 @@ config M68K select HAVE_PRIVATE_LIBGCC select SYS_BOOT_GET_CMDLINE select SYS_BOOT_GET_KBD + imply MCFUART
config MICROBLAZE bool "MicroBlaze architecture" diff --git a/drivers/serial/Kconfig b/drivers/serial/Kconfig index 0a01399961..9387b642b3 100644 --- a/drivers/serial/Kconfig +++ b/drivers/serial/Kconfig @@ -467,6 +467,11 @@ config FSL_LPUART Select this to enable a Low Power UART for Freescale VF610 and QorIQ Layerscape devices.
+config MCFUART + bool "ColdFire UART support" + help + Select this to enable UART support on some ColdFire m68k SoCs. + config MVEBU_A3700_UART bool "UART support for Armada 3700" default n diff --git a/include/configs/M5208EVBE.h b/include/configs/M5208EVBE.h index 3385dcb5b7..d7d21b5708 100644 --- a/include/configs/M5208EVBE.h +++ b/include/configs/M5208EVBE.h @@ -14,7 +14,6 @@ * High Level Configuration Options * (easy to change) */ -#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_WATCHDOG diff --git a/include/configs/M52277EVB.h b/include/configs/M52277EVB.h index 7798b06625..16005f7ef5 100644 --- a/include/configs/M52277EVB.h +++ b/include/configs/M52277EVB.h @@ -19,7 +19,6 @@ * (easy to change) */
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_WATCHDOG diff --git a/include/configs/M5235EVB.h b/include/configs/M5235EVB.h index d221f718f0..3600418907 100644 --- a/include/configs/M5235EVB.h +++ b/include/configs/M5235EVB.h @@ -19,7 +19,6 @@ * (easy to change) */
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_WATCHDOG diff --git a/include/configs/M5249EVB.h b/include/configs/M5249EVB.h index 6b37e46c56..fa78918197 100644 --- a/include/configs/M5249EVB.h +++ b/include/configs/M5249EVB.h @@ -20,7 +20,6 @@ */ #define CONFIG_MCFTMR
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_WATCHDOG diff --git a/include/configs/M5253DEMO.h b/include/configs/M5253DEMO.h index cf10f306c7..7b21e68dd1 100644 --- a/include/configs/M5253DEMO.h +++ b/include/configs/M5253DEMO.h @@ -9,7 +9,6 @@
#define CONFIG_MCFTMR
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_WATCHDOG /* disable watchdog */ diff --git a/include/configs/M5253EVBE.h b/include/configs/M5253EVBE.h index daa44b315b..4a23379be7 100644 --- a/include/configs/M5253EVBE.h +++ b/include/configs/M5253EVBE.h @@ -10,7 +10,6 @@
#define CONFIG_MCFTMR
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_WATCHDOG /* disable watchdog */ diff --git a/include/configs/M5272C3.h b/include/configs/M5272C3.h index 936a91e27d..bf02282e97 100644 --- a/include/configs/M5272C3.h +++ b/include/configs/M5272C3.h @@ -19,7 +19,6 @@ */ #define CONFIG_MCFTMR
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_WATCHDOG diff --git a/include/configs/M5275EVB.h b/include/configs/M5275EVB.h index 1babb72fd0..12836140c4 100644 --- a/include/configs/M5275EVB.h +++ b/include/configs/M5275EVB.h @@ -24,7 +24,6 @@
#define CONFIG_MCFTMR
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
/* Configuration for environment diff --git a/include/configs/M5282EVB.h b/include/configs/M5282EVB.h index b29515077a..5a31d3d860 100644 --- a/include/configs/M5282EVB.h +++ b/include/configs/M5282EVB.h @@ -19,7 +19,6 @@ */ #define CONFIG_MCFTMR
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_MONITOR_IS_IN_RAM /* define if monitor is started from a pre-loader */ diff --git a/include/configs/M53017EVB.h b/include/configs/M53017EVB.h index 96fb87bc7c..a33c91d0e4 100644 --- a/include/configs/M53017EVB.h +++ b/include/configs/M53017EVB.h @@ -19,7 +19,6 @@ * (easy to change) */
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_WATCHDOG diff --git a/include/configs/M5329EVB.h b/include/configs/M5329EVB.h index 5a994859f1..1e0eb0c657 100644 --- a/include/configs/M5329EVB.h +++ b/include/configs/M5329EVB.h @@ -19,7 +19,6 @@ * (easy to change) */
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_WATCHDOG diff --git a/include/configs/M5373EVB.h b/include/configs/M5373EVB.h index 7b2d1bb398..92b1f31b30 100644 --- a/include/configs/M5373EVB.h +++ b/include/configs/M5373EVB.h @@ -19,7 +19,6 @@ * (easy to change) */
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_WATCHDOG diff --git a/include/configs/M54418TWR.h b/include/configs/M54418TWR.h index ba9ba00698..a3ba39d063 100644 --- a/include/configs/M54418TWR.h +++ b/include/configs/M54418TWR.h @@ -19,7 +19,6 @@ * (easy to change) */
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0) #define CONFIG_SYS_BAUDRATE_TABLE { 9600 , 19200 , 38400 , 57600, 115200 }
diff --git a/include/configs/M54451EVB.h b/include/configs/M54451EVB.h index b144332b22..273fac319d 100644 --- a/include/configs/M54451EVB.h +++ b/include/configs/M54451EVB.h @@ -20,7 +20,6 @@ */ #define CONFIG_M54451EVB /* M54451EVB board */
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#define LDS_BOARD_TEXT board/freescale/m54451evb/sbf_dram_init.o (.text*) diff --git a/include/configs/M54455EVB.h b/include/configs/M54455EVB.h index 14dd2a516e..274a83126c 100644 --- a/include/configs/M54455EVB.h +++ b/include/configs/M54455EVB.h @@ -20,7 +20,6 @@ */ #define CONFIG_M54455EVB /* M54455EVB board */
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#define LDS_BOARD_TEXT board/freescale/m54455evb/sbf_dram_init.o (.text*) diff --git a/include/configs/M5475EVB.h b/include/configs/M5475EVB.h index 566f24e3b1..a04d2bc883 100644 --- a/include/configs/M5475EVB.h +++ b/include/configs/M5475EVB.h @@ -19,7 +19,6 @@ * (easy to change) */
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_HW_WATCHDOG diff --git a/include/configs/M5485EVB.h b/include/configs/M5485EVB.h index 31564748bc..cbe29102af 100644 --- a/include/configs/M5485EVB.h +++ b/include/configs/M5485EVB.h @@ -19,7 +19,6 @@ * (easy to change) */
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_HW_WATCHDOG diff --git a/include/configs/amcore.h b/include/configs/amcore.h index 569fd982b6..865d3fee48 100644 --- a/include/configs/amcore.h +++ b/include/configs/amcore.h @@ -12,7 +12,6 @@ #define CONFIG_HOSTNAME AMCORE
#define CONFIG_MCFTMR -#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT 0 #define CONFIG_SYS_BAUDRATE_TABLE { 9600, 19200, 38400, 57600, 115200 }
diff --git a/include/configs/astro_mcf5373l.h b/include/configs/astro_mcf5373l.h index 4a62bce40f..1f3b457e33 100644 --- a/include/configs/astro_mcf5373l.h +++ b/include/configs/astro_mcf5373l.h @@ -89,7 +89,6 @@ * in u-boot command interface */
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (2) #define CONFIG_SYS_UART2_ALT3_GPIO
diff --git a/include/configs/cobra5272.h b/include/configs/cobra5272.h index ee870f916f..e8bba609ee 100644 --- a/include/configs/cobra5272.h +++ b/include/configs/cobra5272.h @@ -50,7 +50,6 @@ * --- */
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
/* --- diff --git a/include/configs/eb_cpu5282.h b/include/configs/eb_cpu5282.h index d79cbf7791..786458fdf7 100644 --- a/include/configs/eb_cpu5282.h +++ b/include/configs/eb_cpu5282.h @@ -17,7 +17,6 @@
#define CONFIG_MISC_INIT_R
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT (0)
#undef CONFIG_MONITOR_IS_IN_RAM /* starts uboot direct */ diff --git a/include/configs/stmark2.h b/include/configs/stmark2.h index d20dd9c3c5..d79d5390e1 100644 --- a/include/configs/stmark2.h +++ b/include/configs/stmark2.h @@ -11,7 +11,6 @@
#define CONFIG_HOSTNAME stmark2
-#define CONFIG_MCFUART #define CONFIG_SYS_UART_PORT 0 #define CONFIG_SYS_BAUDRATE_TABLE { 9600 , 19200 , 38400 , 57600, 115200 }
diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt index 831b4add54..30da0122f0 100644 --- a/scripts/config_whitelist.txt +++ b/scripts/config_whitelist.txt @@ -1250,7 +1250,6 @@ CONFIG_MCFFEC CONFIG_MCFPIT CONFIG_MCFRTC CONFIG_MCFTMR -CONFIG_MCFUART CONFIG_MCLK_DIS CONFIG_MDIO_TIMEOUT CONFIG_MEMSIZE
Signed-off-by: Tuomas Tynkkynen tuomas@tuxera.com --- arch/arm/Kconfig | 2 ++ drivers/serial/Kconfig | 5 +++++ include/configs/xilinx_zynqmp.h | 1 - include/configs/zynq-common.h | 2 -- scripts/config_whitelist.txt | 1 - 5 files changed, 7 insertions(+), 4 deletions(-)
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index 190f883aa8..44a4471b47 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -776,6 +776,7 @@ config ARCH_ZYNQ select CLK select SPL_CLK if SPL select CLK_ZYNQ + imply ARM_DCC imply CMD_CLK imply FAT_WRITE imply CMD_SPL @@ -792,6 +793,7 @@ config ARCH_ZYNQMP select SPL_BOARD_INIT if SPL select SPL_CLK if SPL select DM_USB if USB + imply ARM_DCC imply FAT_WRITE
config TEGRA diff --git a/drivers/serial/Kconfig b/drivers/serial/Kconfig index 9387b642b3..41ce6dec8c 100644 --- a/drivers/serial/Kconfig +++ b/drivers/serial/Kconfig @@ -432,6 +432,11 @@ config AR933X_UART tree binding to operate, please refer to the document at doc/device-tree-bindings/serial/qca,ar9330-uart.txt.
+config ARM_DCC + bool "ARM DCC (Debug Communications Channel) serial console support" + help + Select this to enable a serial port using the ARM JTAG DCC port. + config ATMEL_USART bool "Atmel USART support" help diff --git a/include/configs/xilinx_zynqmp.h b/include/configs/xilinx_zynqmp.h index 8c0b5d9c06..312a2a83ff 100644 --- a/include/configs/xilinx_zynqmp.h +++ b/include/configs/xilinx_zynqmp.h @@ -42,7 +42,6 @@ #define CONFIG_SYS_MALLOC_LEN (CONFIG_ENV_SIZE + 0x2000000)
/* Serial setup */ -#define CONFIG_ARM_DCC #define CONFIG_CPU_ARMV8
#define CONFIG_SYS_BAUDRATE_TABLE \ diff --git a/include/configs/zynq-common.h b/include/configs/zynq-common.h index 554fb66634..699c91d3c5 100644 --- a/include/configs/zynq-common.h +++ b/include/configs/zynq-common.h @@ -32,8 +32,6 @@ #define CONFIG_SYS_BAUDRATE_TABLE \ {300, 600, 1200, 2400, 4800, 9600, 19200, 38400, 57600, 115200, 230400}
-#define CONFIG_ARM_DCC - /* Ethernet driver */ #if defined(CONFIG_ZYNQ_GEM) # define CONFIG_MII diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt index 30da0122f0..417a978f0a 100644 --- a/scripts/config_whitelist.txt +++ b/scripts/config_whitelist.txt @@ -59,7 +59,6 @@ CONFIG_ARMV7_SECURE_MAX_SIZE CONFIG_ARMV7_SECURE_RESERVE_SIZE CONFIG_ARMV8_SWITCH_TO_EL1 CONFIG_ARM_ARCH_CP15_ERRATA -CONFIG_ARM_DCC CONFIG_ARM_FREQ CONFIG_ARM_GIC_BASE_ADDRESS CONFIG_ARM_PL180_MMCI_BASE
Hi gents,
On Sun, 2018-04-08 at 00:27 +0300, Tuomas Tynkkynen wrote:
Signed-off-by: Tuomas Tynkkynen tuomas@tuxera.com
arch/arm/Kconfig | 2 ++ drivers/serial/Kconfig | 5 +++++ include/configs/xilinx_zynqmp.h | 1 - include/configs/zynq-common.h | 2 -- scripts/config_whitelist.txt | 1 - 5 files changed, 7 insertions(+), 4 deletions(-)
Not sure who's the right person to review and accept this one. I'm not the one for sure :)
-Alexey
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index 190f883aa8..44a4471b47 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -776,6 +776,7 @@ config ARCH_ZYNQ select CLK select SPL_CLK if SPL select CLK_ZYNQ
- imply ARM_DCC imply CMD_CLK imply FAT_WRITE imply CMD_SPL
@@ -792,6 +793,7 @@ config ARCH_ZYNQMP select SPL_BOARD_INIT if SPL select SPL_CLK if SPL select DM_USB if USB
- imply ARM_DCC imply FAT_WRITE
config TEGRA diff --git a/drivers/serial/Kconfig b/drivers/serial/Kconfig index 9387b642b3..41ce6dec8c 100644 --- a/drivers/serial/Kconfig +++ b/drivers/serial/Kconfig @@ -432,6 +432,11 @@ config AR933X_UART tree binding to operate, please refer to the document at doc/device-tree-bindings/serial/qca,ar9330-uart.txt.
+config ARM_DCC
- bool "ARM DCC (Debug Communications Channel) serial console support"
- help
Select this to enable a serial port using the ARM JTAG DCC port.config ATMEL_USART bool "Atmel USART support" help diff --git a/include/configs/xilinx_zynqmp.h b/include/configs/xilinx_zynqmp.h index 8c0b5d9c06..312a2a83ff 100644 --- a/include/configs/xilinx_zynqmp.h +++ b/include/configs/xilinx_zynqmp.h @@ -42,7 +42,6 @@ #define CONFIG_SYS_MALLOC_LEN (CONFIG_ENV_SIZE + 0x2000000)
/* Serial setup */ -#define CONFIG_ARM_DCC #define CONFIG_CPU_ARMV8
#define CONFIG_SYS_BAUDRATE_TABLE \ diff --git a/include/configs/zynq-common.h b/include/configs/zynq-common.h index 554fb66634..699c91d3c5 100644 --- a/include/configs/zynq-common.h +++ b/include/configs/zynq-common.h @@ -32,8 +32,6 @@ #define CONFIG_SYS_BAUDRATE_TABLE \ {300, 600, 1200, 2400, 4800, 9600, 19200, 38400, 57600, 115200, 230400}
-#define CONFIG_ARM_DCC
/* Ethernet driver */ #if defined(CONFIG_ZYNQ_GEM) # define CONFIG_MII diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt index 30da0122f0..417a978f0a 100644 --- a/scripts/config_whitelist.txt +++ b/scripts/config_whitelist.txt @@ -59,7 +59,6 @@ CONFIG_ARMV7_SECURE_MAX_SIZE CONFIG_ARMV7_SECURE_RESERVE_SIZE CONFIG_ARMV8_SWITCH_TO_EL1 CONFIG_ARM_ARCH_CP15_ERRATA -CONFIG_ARM_DCC CONFIG_ARM_FREQ CONFIG_ARM_GIC_BASE_ADDRESS CONFIG_ARM_PL180_MMCI_BASE
Ooops I meant https://patchwork.ozlabs.org/patch/895969/ here.
This one I guess is for Albert or Michal.
-Alexey
On Mon, 2018-05-28 at 12:33 +0300, Alexey Brodkin wrote:
Hi gents,
On Sun, 2018-04-08 at 00:27 +0300, Tuomas Tynkkynen wrote:
Signed-off-by: Tuomas Tynkkynen tuomas@tuxera.com
arch/arm/Kconfig | 2 ++ drivers/serial/Kconfig | 5 +++++ include/configs/xilinx_zynqmp.h | 1 - include/configs/zynq-common.h | 2 -- scripts/config_whitelist.txt | 1 - 5 files changed, 7 insertions(+), 4 deletions(-)
Not sure who's the right person to review and accept this one. I'm not the one for sure :)
-Alexey
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index 190f883aa8..44a4471b47 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -776,6 +776,7 @@ config ARCH_ZYNQ select CLK select SPL_CLK if SPL select CLK_ZYNQ
- imply ARM_DCC imply CMD_CLK imply FAT_WRITE imply CMD_SPL
@@ -792,6 +793,7 @@ config ARCH_ZYNQMP select SPL_BOARD_INIT if SPL select SPL_CLK if SPL select DM_USB if USB
- imply ARM_DCC imply FAT_WRITE
config TEGRA diff --git a/drivers/serial/Kconfig b/drivers/serial/Kconfig index 9387b642b3..41ce6dec8c 100644 --- a/drivers/serial/Kconfig +++ b/drivers/serial/Kconfig @@ -432,6 +432,11 @@ config AR933X_UART tree binding to operate, please refer to the document at doc/device-tree-bindings/serial/qca,ar9330-uart.txt.
+config ARM_DCC
- bool "ARM DCC (Debug Communications Channel) serial console support"
- help
Select this to enable a serial port using the ARM JTAG DCC port.config ATMEL_USART bool "Atmel USART support" help diff --git a/include/configs/xilinx_zynqmp.h b/include/configs/xilinx_zynqmp.h index 8c0b5d9c06..312a2a83ff 100644 --- a/include/configs/xilinx_zynqmp.h +++ b/include/configs/xilinx_zynqmp.h @@ -42,7 +42,6 @@ #define CONFIG_SYS_MALLOC_LEN (CONFIG_ENV_SIZE + 0x2000000)
/* Serial setup */ -#define CONFIG_ARM_DCC #define CONFIG_CPU_ARMV8
#define CONFIG_SYS_BAUDRATE_TABLE \ diff --git a/include/configs/zynq-common.h b/include/configs/zynq-common.h index 554fb66634..699c91d3c5 100644 --- a/include/configs/zynq-common.h +++ b/include/configs/zynq-common.h @@ -32,8 +32,6 @@ #define CONFIG_SYS_BAUDRATE_TABLE \ {300, 600, 1200, 2400, 4800, 9600, 19200, 38400, 57600, 115200, 230400}
-#define CONFIG_ARM_DCC
/* Ethernet driver */ #if defined(CONFIG_ZYNQ_GEM) # define CONFIG_MII diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt index 30da0122f0..417a978f0a 100644 --- a/scripts/config_whitelist.txt +++ b/scripts/config_whitelist.txt @@ -59,7 +59,6 @@ CONFIG_ARMV7_SECURE_MAX_SIZE CONFIG_ARMV7_SECURE_RESERVE_SIZE CONFIG_ARMV8_SWITCH_TO_EL1 CONFIG_ARM_ARCH_CP15_ERRATA -CONFIG_ARM_DCC CONFIG_ARM_FREQ CONFIG_ARM_GIC_BASE_ADDRESS CONFIG_ARM_PL180_MMCI_BASE
Hi,
On 7.4.2018 23:27, Tuomas Tynkkynen wrote:
Signed-off-by: Tuomas Tynkkynen tuomas@tuxera.com
arch/arm/Kconfig | 2 ++ drivers/serial/Kconfig | 5 +++++ include/configs/xilinx_zynqmp.h | 1 - include/configs/zynq-common.h | 2 -- scripts/config_whitelist.txt | 1 - 5 files changed, 7 insertions(+), 4 deletions(-)
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index 190f883aa8..44a4471b47 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -776,6 +776,7 @@ config ARCH_ZYNQ select CLK select SPL_CLK if SPL select CLK_ZYNQ
- imply ARM_DCC imply CMD_CLK imply FAT_WRITE imply CMD_SPL
@@ -792,6 +793,7 @@ config ARCH_ZYNQMP select SPL_BOARD_INIT if SPL select SPL_CLK if SPL select DM_USB if USB
- imply ARM_DCC imply FAT_WRITE
config TEGRA diff --git a/drivers/serial/Kconfig b/drivers/serial/Kconfig index 9387b642b3..41ce6dec8c 100644 --- a/drivers/serial/Kconfig +++ b/drivers/serial/Kconfig @@ -432,6 +432,11 @@ config AR933X_UART tree binding to operate, please refer to the document at doc/device-tree-bindings/serial/qca,ar9330-uart.txt.
+config ARM_DCC
- bool "ARM DCC (Debug Communications Channel) serial console support"
- help
Select this to enable a serial port using the ARM JTAG DCC port.
I am missing adding dependency to DEBUG_UART_ARM_DCC symbol which depends on this new one.
config ATMEL_USART bool "Atmel USART support" help diff --git a/include/configs/xilinx_zynqmp.h b/include/configs/xilinx_zynqmp.h index 8c0b5d9c06..312a2a83ff 100644 --- a/include/configs/xilinx_zynqmp.h +++ b/include/configs/xilinx_zynqmp.h @@ -42,7 +42,6 @@ #define CONFIG_SYS_MALLOC_LEN (CONFIG_ENV_SIZE + 0x2000000)
/* Serial setup */ -#define CONFIG_ARM_DCC #define CONFIG_CPU_ARMV8
#define CONFIG_SYS_BAUDRATE_TABLE \ diff --git a/include/configs/zynq-common.h b/include/configs/zynq-common.h index 554fb66634..699c91d3c5 100644 --- a/include/configs/zynq-common.h +++ b/include/configs/zynq-common.h @@ -32,8 +32,6 @@ #define CONFIG_SYS_BAUDRATE_TABLE \ {300, 600, 1200, 2400, 4800, 9600, 19200, 38400, 57600, 115200, 230400}
-#define CONFIG_ARM_DCC
/* Ethernet driver */ #if defined(CONFIG_ZYNQ_GEM) # define CONFIG_MII diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt index 30da0122f0..417a978f0a 100644 --- a/scripts/config_whitelist.txt +++ b/scripts/config_whitelist.txt @@ -59,7 +59,6 @@ CONFIG_ARMV7_SECURE_MAX_SIZE CONFIG_ARMV7_SECURE_RESERVE_SIZE CONFIG_ARMV8_SWITCH_TO_EL1 CONFIG_ARM_ARCH_CP15_ERRATA -CONFIG_ARM_DCC
Nit: Tom is syncing this time to time that you don't need to remove it but it is ok to keep it here too.
CONFIG_ARM_FREQ CONFIG_ARM_GIC_BASE_ADDRESS CONFIG_ARM_PL180_MMCI_BASE
Please rebase it on the top of current master branch and I am happy to apply this one via my tree. You can send this one separately.
Thanks, Michal
Hi Tuomas,
On Sun, 2018-04-08 at 00:27 +0300, Tuomas Tynkkynen wrote:
Signed-off-by: Tuomas Tynkkynen tuomas@tuxera.com
arch/arc/Kconfig | 1 + drivers/serial/Kconfig | 7 +++++++ include/configs/nsim.h | 10 ---------- scripts/config_whitelist.txt | 1 - 4 files changed, 8 insertions(+), 11 deletions(-)
Sorry I didn't notice your patch and already made a similar change, see http://git.denx.de/?p=u-boot.git;a=commit;h=d7ac185fd840d1406fcafc1b41248213....
Next time please add maintainers of modified code in a copy of your email (either via "Cc" tag right in patch or explicitly on execution of "git send-email").
Maintainer names you may get easily with "get_maintainer.pl" script like that: ---------------------------->8------------------------------ ./scripts/get_maintainer.pl 0001-serial-Migrate-CONFIG_ARC_SERIAL-to-Kconfig.patch Alexey Brodkin alexey.brodkin@synopsys.com (maintainer:ARC) Eugeniy Paltsev Eugeniy.Paltsev@synopsys.com (maintainer:ARC) Alexander Graf agraf@suse.de (commit_signer:6/24=25%,authored:6/24=25%,added_lines:46/228=20%) Stefan Roese sr@denx.de (commit_signer:4/24=17%,authored:3/24=12%,added_lines:25/228=11%,removed_lines:10/30=33%) Simon Glass sjg@chromium.org (commit_signer:4/24=17%,commit_signer:80/232=34%,authored:49/232=21%) Patrice Chotard patrice.chotard@st.com (commit_signer:4/24=17%,authored:4/24=17%,removed_lines:6/30=20%) Bin Meng bmeng.cn@gmail.com (commit_signer:2/24=8%,commit_signer:54/232=23%,authored:12/232=5%,removed_lines:128/1669=8%) Tuomas Tynkkynen tuomas@tuxera.com (authored:2/24=8%,added_lines:14/228=6%,commit_signer:34/232=15%,authored:33/232=14%,removed_lines:127/1669=8%) Christophe Leroy christophe.leroy@c-s.fr (authored:2/24=8%,added_lines:54/228=24%,added_lines:22/32=69%) Patrick Delaunay patrick.delaunay@st.com (removed_lines:3/30=10%) Heiko Schocher hs@denx.de (removed_lines:2/30=7%,removed_lines:931/1669=56%) Philipp Tomsich philipp.tomsich@theobroma-systems.com (commit_signer:42/232=18%,added_lines:4/32=12%) Adam Ford aford173@gmail.com (commit_signer:24/232=10%,authored:24/232=10%) Chris Packham judge.packham@gmail.com (added_lines:3/32=9%) uboot-snps-arc@synopsys.com (open list:ARC) u-boot@lists.denx.de (open list) ---------------------------->8------------------------------
From the list above you should definitely Cc all mentioned lists as well as maintainers to make sure right people will be notified :)
Also it seems like you sent a series without so-called cover-letter and so entire series became named after the first patch and so all 3 patches (2 of which have nothing to do with ARC) we assigned to me in Patchwork and again right people never realized there's some work for them.
-Alexey
participants (3)
-
Alexey Brodkin -
Michal Simek -
Tuomas Tynkkynen