From patchwork Fri Jan 30 02:01:17 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Rothwell X-Patchwork-Id: 5746311 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 92E039F36E for ; Fri, 30 Jan 2015 02:04:13 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 56E222022D for ; Fri, 30 Jan 2015 02:04:12 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.9]) (using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 2865320222 for ; Fri, 30 Jan 2015 02:04:11 +0000 (UTC) Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.80.1 #2 (Red Hat Linux)) id 1YH0u6-0006PA-FW; Fri, 30 Jan 2015 02:01:50 +0000 Received: from ozlabs.org ([2401:3900:2:1::2]) by bombadil.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1YH0u2-0006Ky-Hm for linux-arm-kernel@lists.infradead.org; Fri, 30 Jan 2015 02:01:47 +0000 Received: from authenticated.ozlabs.org (localhost [127.0.0.1]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPSA id CEE851402B0; Fri, 30 Jan 2015 13:01:22 +1100 (AEDT) Date: Fri, 30 Jan 2015 13:01:17 +1100 From: Stephen Rothwell To: Olof Johansson , Arnd Bergmann , , Russell King Subject: [PATCH] arm: reduce conflicts in Kcinfig.debug Message-ID: <20150130130117.4d9fbd70@canb.auug.org.au> X-Mailer: Claws Mail 3.11.1 (GTK+ 2.24.25; i586-pc-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20150129_180146_862685_92A18479 X-CRM114-Status: GOOD ( 15.61 ) X-Spam-Score: 0.0 (/) Cc: linux-next@vger.kernel.org, linux-kernel@vger.kernel.org X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org X-Spam-Status: No, score=-4.2 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_MED, T_RP_MATCHES_RCVD, UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP I have seen several conflicts in this file (and other Kconfig files) over time, so I suggest a solution like below. Create new config symbols that can be selected by other configs and use them as single dependencies rather than having long lists of other config symbols as dependencies. Signed-off-by: Stephen Rothwell --- arch/arm/Kconfig | 1 + arch/arm/Kconfig.debug | 55 +++++++++++++++++++++++++++++++++++--------------- 2 files changed, 40 insertions(+), 16 deletions(-) Not actually tested, but it was a very mechanical transformation. This is against Russell's current arm tree in linux-next, so there would be some adjustments required for the arm-soc tree. There are probably some other places that this technique would be useful as well. diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index 1f5eb9aee750..1c9ad3832f88 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -432,6 +432,7 @@ config ARCH_EP93XX select ARM_VIC select CLKDEV_LOOKUP select CPU_ARM920T + select DEBUG_UART_PHYS_ENABLE help This enables support for the Cirrus EP93xx series of CPUs. diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug index a324ecdfeb21..fc991aba8944 100644 --- a/arch/arm/Kconfig.debug +++ b/arch/arm/Kconfig.debug @@ -96,6 +96,8 @@ choice config DEBUG_ASM9260_UART bool "Kernel low-level debugging via asm9260 UART" depends on MACH_ASM9260 + select DEBUG_UART_PHYS_ENABLE + select DEBUG_UART_VIRT_ENABLE help Say Y here if you want the debug print routines to direct their output to an UART or USART port on asm9260 based @@ -407,6 +409,8 @@ choice config DEBUG_MESON_UARTAO bool "Kernel low-level debugging via Meson6 UARTAO" depends on ARCH_MESON + select DEBUG_UART_PHYS_ENABLE + select DEBUG_UART_VIRT_ENABLE help Say Y here if you want kernel low-lever debugging support on Amlogic Meson6 based platforms on the UARTAO. @@ -430,6 +434,8 @@ choice config DEBUG_MSM_UART bool "Kernel low-level debugging messages via MSM UART" depends on ARCH_MSM + select DEBUG_UART_PHYS_ENABLE + select DEBUG_UART_VIRT_ENABLE help Say Y here if you want the debug print routines to direct their output to the serial port on MSM devices. @@ -449,6 +455,8 @@ choice config DEBUG_QCOM_UARTDM bool "Kernel low-level debugging messages via QCOM UARTDM" depends on ARCH_QCOM + select DEBUG_UART_PHYS_ENABLE + select DEBUG_UART_VIRT_ENABLE help Say Y here if you want the debug print routines to direct their output to the serial port on Qualcomm devices. @@ -506,6 +514,8 @@ choice config DEBUG_NETX_UART bool "Kernel low-level debugging messages via NetX UART" depends on ARCH_NETX + select DEBUG_UART_PHYS_ENABLE + select DEBUG_UART_VIRT_ENABLE help Say Y here if you want kernel low-level debugging support on Hilscher NetX based platforms. @@ -750,6 +760,7 @@ choice config DEBUG_R7S72100_SCIF2 bool "Kernel low-level debugging messages via SCIF2 on R7S72100" depends on ARCH_R7S72100 + select DEBUG_UART_PHYS_ENABLE help Say Y here if you want kernel low-level debugging support via SCIF2 on Renesas RZ/A1H (R7S72100). @@ -757,6 +768,7 @@ choice config DEBUG_RCAR_GEN1_SCIF0 bool "Kernel low-level debugging messages via SCIF0 on R8A7778" depends on ARCH_R8A7778 + select DEBUG_UART_PHYS_ENABLE help Say Y here if you want kernel low-level debugging support via SCIF0 on Renesas R-Car M1A (R8A7778). @@ -764,6 +776,7 @@ choice config DEBUG_RCAR_GEN1_SCIF2 bool "Kernel low-level debugging messages via SCIF2 on R8A7779" depends on ARCH_R8A7779 + select DEBUG_UART_PHYS_ENABLE help Say Y here if you want kernel low-level debugging support via SCIF2 on Renesas R-Car H1 (R8A7779). @@ -771,6 +784,7 @@ choice config DEBUG_RCAR_GEN2_SCIF0 bool "Kernel low-level debugging messages via SCIF0 on R8A7790/R8A7791/R8A7793)" depends on ARCH_R8A7790 || ARCH_R8A7791 || ARCH_R8A7793 + select DEBUG_UART_PHYS_ENABLE help Say Y here if you want kernel low-level debugging support via SCIF0 on Renesas R-Car H2 (R8A7790), M2-W (R8A7791), or @@ -779,6 +793,7 @@ choice config DEBUG_RCAR_GEN2_SCIF2 bool "Kernel low-level debugging messages via SCIF2 on R8A7794" depends on ARCH_R8A7794 + select DEBUG_UART_PHYS_ENABLE help Say Y here if you want kernel low-level debugging support via SCIF2 on Renesas R-Car E2 (R8A7794). @@ -786,6 +801,7 @@ choice config DEBUG_RMOBILE_SCIFA0 bool "Kernel low-level debugging messages via SCIFA0 on R8A73A4/SH7372" depends on ARCH_R8A73A4 || ARCH_SH7372 + select DEBUG_UART_PHYS_ENABLE help Say Y here if you want kernel low-level debugging support via SCIFA0 on Renesas R-Mobile APE6 (R8A73A4) or SH-Mobile @@ -794,6 +810,7 @@ choice config DEBUG_RMOBILE_SCIFA1 bool "Kernel low-level debugging messages via SCIFA1 on R8A7740" depends on ARCH_R8A7740 + select DEBUG_UART_PHYS_ENABLE help Say Y here if you want kernel low-level debugging support via SCIFA1 on Renesas R-Mobile A1 (R8A7740). @@ -801,6 +818,7 @@ choice config DEBUG_RMOBILE_SCIFA4 bool "Kernel low-level debugging messages via SCIFA4 on SH73A0" depends on ARCH_SH73A0 + select DEBUG_UART_PHYS_ENABLE help Say Y here if you want kernel low-level debugging support via SCIFA4 on Renesas SH-Mobile AG5 (SH73A0). @@ -1122,6 +1140,8 @@ choice config DEBUG_LL_UART_8250 bool "Kernel low-level debugging via 8250 UART" + select DEBUG_UART_PHYS_ENABLE + select DEBUG_UART_VIRT_ENABLE help Say Y here if you wish the debug print routes to direct their output to an 8250 UART. You can use this option @@ -1136,6 +1156,7 @@ choice config DEBUG_LL_UART_EFM32 bool "Kernel low-level debugging via efm32 UART" depends on ARCH_EFM32 + select DEBUG_UART_PHYS_ENABLE help Say Y here if you want the debug print routines to direct their output to an UART or USART port on efm32 based @@ -1149,6 +1170,8 @@ choice config DEBUG_LL_UART_PL01X bool "Kernel low-level debugging via ARM Ltd PL01x Primecell UART" + select DEBUG_UART_PHYS_ENABLE + select DEBUG_UART_VIRT_ENABLE help Say Y here if you wish the debug print routes to direct their output to a PL01x Primecell UART. You can use @@ -1171,6 +1194,8 @@ config DEBUG_S3C2410_UART config DEBUG_S3C24XX_UART bool + select DEBUG_UART_PHYS_ENABLE + select DEBUG_UART_VIRT_ENABLE config DEBUG_S5PV210_UART bool @@ -1264,6 +1289,8 @@ config DEBUG_LL_INCLUDE # Compatibility options for PL01x config DEBUG_UART_PL01X bool + select DEBUG_UART_PHYS_ENABLE + select DEBUG_UART_VIRT_ENABLE # Compatibility options for 8250 config DEBUG_UART_8250 @@ -1272,13 +1299,21 @@ config DEBUG_UART_8250 ARCH_GEMINI || ARCH_IOP13XX || ARCH_IOP32X || \ ARCH_IOP33X || ARCH_IXP4XX || \ ARCH_LPC32XX || ARCH_MV78XX0 || ARCH_ORION5X || ARCH_RPC + select DEBUG_UART_PHYS_ENABLE + select DEBUG_UART_VIRT_ENABLE # Compatibility options for BCM63xx config DEBUG_UART_BCM63XX def_bool ARCH_BCM_63XX + select DEBUG_UART_PHYS_ENABLE + select DEBUG_UART_VIRT_ENABLE + +config DEBUG_UART_PHYS_ENABLE + bool config DEBUG_UART_PHYS hex "Physical base address of debug UART" + depends on DEBUG_UART_PHYS_ENABLE default 0x00100a00 if DEBUG_NETX_UART default 0x01c20000 if DEBUG_DAVINCI_DMx_UART0 default 0x01c28000 if DEBUG_SUNXI_UART0 @@ -1365,20 +1400,13 @@ config DEBUG_UART_PHYS default 0xfffb9800 if DEBUG_OMAP1UART3 || DEBUG_OMAP7XXUART3 default 0xfffe8600 if DEBUG_UART_BCM63XX default 0xfffff700 if ARCH_IOP33X - depends on ARCH_EP93XX || \ - DEBUG_LL_UART_8250 || DEBUG_LL_UART_PL01X || \ - DEBUG_LL_UART_EFM32 || \ - DEBUG_UART_8250 || DEBUG_UART_PL01X || DEBUG_MESON_UARTAO || \ - DEBUG_MSM_UART || DEBUG_NETX_UART || \ - DEBUG_QCOM_UARTDM || DEBUG_R7S72100_SCIF2 || \ - DEBUG_RCAR_GEN1_SCIF0 || DEBUG_RCAR_GEN1_SCIF2 || \ - DEBUG_RCAR_GEN2_SCIF0 || DEBUG_RCAR_GEN2_SCIF2 || \ - DEBUG_RMOBILE_SCIFA0 || DEBUG_RMOBILE_SCIFA1 || \ - DEBUG_RMOBILE_SCIFA4 || DEBUG_S3C24XX_UART || \ - DEBUG_UART_BCM63XX || DEBUG_ASM9260_UART + +config DEBUG_UART_VIRT_ENABLE + bool config DEBUG_UART_VIRT hex "Virtual base address of debug UART" + depends on DEBUG_UART_VIRT_ENABLE default 0xe0000a00 if DEBUG_NETX_UART default 0xe0010fe0 if ARCH_RPC default 0xe1000000 if DEBUG_MSM_UART @@ -1454,11 +1482,6 @@ config DEBUG_UART_VIRT default 0xfefff700 if ARCH_IOP33X default 0xff003000 if DEBUG_U300_UART default DEBUG_UART_PHYS if !MMU - depends on DEBUG_LL_UART_8250 || DEBUG_LL_UART_PL01X || \ - DEBUG_UART_8250 || DEBUG_UART_PL01X || DEBUG_MESON_UARTAO || \ - DEBUG_MSM_UART || DEBUG_NETX_UART || \ - DEBUG_QCOM_UARTDM || DEBUG_S3C24XX_UART || \ - DEBUG_UART_BCM63XX || DEBUG_ASM9260_UART config DEBUG_UART_8250_SHIFT int "Register offset shift for the 8250 debug UART"