Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755922Ab3IYXgs (ORCPT ); Wed, 25 Sep 2013 19:36:48 -0400 Received: from mail-pd0-f173.google.com ([209.85.192.173]:33075 "EHLO mail-pd0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755836Ab3IYXgq (ORCPT ); Wed, 25 Sep 2013 19:36:46 -0400 From: Magnus Damm To: linux-sh@vger.kernel.org Cc: khilman@linaro.org, arnd@arndb.de, daniel.lezcano@linaro.org, linux-kernel@vger.kernel.org, horms@verge.net.au, olof@lixom.net, Magnus Damm , tglx@linutronix.de Date: Thu, 26 Sep 2013 08:37:09 +0900 Message-Id: <20130925233709.16540.66903.sendpatchset@w520> Subject: [PATCH/RFC] clocksource: Consolidate SH and ARM mach-shmobile Kconfig bits Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 7129 Lines: 280 From: Magnus Damm Based on request from Olof Johansson, move ARM mach-shmobile clocksource entries to drivers/clocksource/Kconfig, and at the same time make the SH architecture use these entries. Kconfig entries for CMT, MTU2, TMU and STI are with this patch now kept in one place. The SH SYS_SUPPORTS_nnn bits are dropped to simplye let the user select the appropriate ones using Kconfig instead of trying to filter available timers based on SoC. Known issues: 1) Not yet wrappend in menu/endmenu, populates the root driver menu -> proposal: Create separate patch to wrap all the clocksource entries? 2) Should the entries be sorted? -> proposal: Ignore cosmetic bits 3) What is correct dependency for multiplatform and single platform? -> proposal: Perhaps we should simply default to no? 4) Is single patch across 2 architectures and drivers/clocksource OK? -> proposal: Keep it like this if possible Not-yet-signed-off-by: Magnus Damm --- arch/arm/mach-shmobile/Kconfig | 19 -------------- arch/sh/Kconfig | 53 ---------------------------------------- drivers/clocksource/Kconfig | 25 ++++++++++++++++++ 3 files changed, 25 insertions(+), 72 deletions(-) --- 0004/arch/arm/mach-shmobile/Kconfig +++ work/arch/arm/mach-shmobile/Kconfig 2013-09-26 07:51:06.000000000 +0900 @@ -307,25 +307,6 @@ config SHMOBILE_TIMER_HZ SH-Mobile systems using a 32768 Hz RCLK for clock events may want to select a HZ value such as 128 that can evenly divide RCLK. A HZ value that does not divide evenly may cause timer drift. - -config SH_TIMER_CMT - bool "CMT timer driver" - default y - help - This enables build of the CMT timer driver. - -config SH_TIMER_TMU - bool "TMU timer driver" - default y - help - This enables build of the TMU timer driver. - -config EM_TIMER_STI - bool "STI timer driver" - default y - help - This enables build of the STI timer driver. - endmenu endif --- 0001/arch/sh/Kconfig +++ work/arch/sh/Kconfig 2013-09-26 08:00:49.000000000 +0900 @@ -121,15 +121,6 @@ config SYS_SUPPORTS_NUMA config SYS_SUPPORTS_PCI bool -config SYS_SUPPORTS_CMT - bool - -config SYS_SUPPORTS_MTU2 - bool - -config SYS_SUPPORTS_TMU - bool - config STACKTRACE_SUPPORT def_bool y @@ -189,14 +180,12 @@ config CPU_SH3 bool select CPU_HAS_INTEVT select CPU_HAS_SR_RB - select SYS_SUPPORTS_TMU config CPU_SH4 bool select CPU_HAS_INTEVT select CPU_HAS_SR_RB select CPU_HAS_FPU if !CPU_SH4AL_DSP - select SYS_SUPPORTS_TMU select SYS_SUPPORTS_HUGETLBFS if MMU config CPU_SH4A @@ -211,7 +200,6 @@ config CPU_SH4AL_DSP config CPU_SH5 bool select CPU_HAS_FPU - select SYS_SUPPORTS_TMU select SYS_SUPPORTS_HUGETLBFS if MMU config CPU_SHX2 @@ -248,7 +236,6 @@ choice config CPU_SUBTYPE_SH7619 bool "Support SH7619 processor" select CPU_SH2 - select SYS_SUPPORTS_CMT # SH-2A Processor Support @@ -256,50 +243,38 @@ config CPU_SUBTYPE_SH7201 bool "Support SH7201 processor" select CPU_SH2A select CPU_HAS_FPU - select SYS_SUPPORTS_MTU2 config CPU_SUBTYPE_SH7203 bool "Support SH7203 processor" select CPU_SH2A select CPU_HAS_FPU - select SYS_SUPPORTS_CMT - select SYS_SUPPORTS_MTU2 select ARCH_WANT_OPTIONAL_GPIOLIB select PINCTRL config CPU_SUBTYPE_SH7206 bool "Support SH7206 processor" select CPU_SH2A - select SYS_SUPPORTS_CMT - select SYS_SUPPORTS_MTU2 config CPU_SUBTYPE_SH7263 bool "Support SH7263 processor" select CPU_SH2A select CPU_HAS_FPU - select SYS_SUPPORTS_CMT - select SYS_SUPPORTS_MTU2 config CPU_SUBTYPE_SH7264 bool "Support SH7264 processor" select CPU_SH2A select CPU_HAS_FPU - select SYS_SUPPORTS_CMT - select SYS_SUPPORTS_MTU2 select PINCTRL config CPU_SUBTYPE_SH7269 bool "Support SH7269 processor" select CPU_SH2A select CPU_HAS_FPU - select SYS_SUPPORTS_CMT - select SYS_SUPPORTS_MTU2 select PINCTRL config CPU_SUBTYPE_MXG bool "Support MX-G processor" select CPU_SH2A - select SYS_SUPPORTS_MTU2 help Select MX-G if running on an R8A03022BG part. @@ -352,7 +327,6 @@ config CPU_SUBTYPE_SH7720 bool "Support SH7720 processor" select CPU_SH3 select CPU_HAS_DSP - select SYS_SUPPORTS_CMT select ARCH_WANT_OPTIONAL_GPIOLIB select USB_ARCH_HAS_OHCI select USB_OHCI_SH if USB_OHCI_HCD @@ -364,7 +338,6 @@ config CPU_SUBTYPE_SH7721 bool "Support SH7721 processor" select CPU_SH3 select CPU_HAS_DSP - select SYS_SUPPORTS_CMT select USB_ARCH_HAS_OHCI select USB_OHCI_SH if USB_OHCI_HCD help @@ -420,7 +393,6 @@ config CPU_SUBTYPE_SH7723 select CPU_SHX2 select ARCH_SHMOBILE select ARCH_SPARSEMEM_ENABLE - select SYS_SUPPORTS_CMT select ARCH_WANT_OPTIONAL_GPIOLIB select PINCTRL help @@ -432,7 +404,6 @@ config CPU_SUBTYPE_SH7724 select CPU_SHX2 select ARCH_SHMOBILE select ARCH_SPARSEMEM_ENABLE - select SYS_SUPPORTS_CMT select ARCH_WANT_OPTIONAL_GPIOLIB select PINCTRL help @@ -512,7 +483,6 @@ config CPU_SUBTYPE_SH7343 bool "Support SH7343 processor" select CPU_SH4AL_DSP select ARCH_SHMOBILE - select SYS_SUPPORTS_CMT config CPU_SUBTYPE_SH7722 bool "Support SH7722 processor" @@ -521,7 +491,6 @@ config CPU_SUBTYPE_SH7722 select ARCH_SHMOBILE select ARCH_SPARSEMEM_ENABLE select SYS_SUPPORTS_NUMA - select SYS_SUPPORTS_CMT select ARCH_WANT_OPTIONAL_GPIOLIB select PINCTRL @@ -532,7 +501,6 @@ config CPU_SUBTYPE_SH7366 select ARCH_SHMOBILE select ARCH_SPARSEMEM_ENABLE select SYS_SUPPORTS_NUMA - select SYS_SUPPORTS_CMT endchoice @@ -565,27 +533,6 @@ source "arch/sh/boards/Kconfig" menu "Timer and clock configuration" -config SH_TIMER_TMU - bool "TMU timer driver" - depends on SYS_SUPPORTS_TMU - default y - help - This enables the build of the TMU timer driver. - -config SH_TIMER_CMT - bool "CMT timer driver" - depends on SYS_SUPPORTS_CMT - default y - help - This enables build of the CMT timer driver. - -config SH_TIMER_MTU2 - bool "MTU2 timer driver" - depends on SYS_SUPPORTS_MTU2 - default y - help - This enables build of the MTU2 timer driver. - config SH_PCLK_FREQ int "Peripheral clock frequency (in Hz)" depends on SH_CLK_CPG_LEGACY --- 0001/drivers/clocksource/Kconfig +++ work/drivers/clocksource/Kconfig 2013-09-26 07:59:19.000000000 +0900 @@ -109,3 +109,28 @@ config VF_PIT_TIMER bool help Support for Period Interrupt Timer on Freescale Vybrid Family SoCs. + +config SH_TIMER_CMT + bool "CMT timer driver" + default y + help + This enables build of the CMT timer driver. + +config SH_TIMER_MTU2 + bool "MTU2 timer driver" + default y + help + This enables build of the MTU2 timer driver. + +config SH_TIMER_TMU + bool "TMU timer driver" + default y + help + This enables build of the TMU timer driver. + +config EM_TIMER_STI + bool "STI timer driver" + default y + help + This enables build of the STI timer driver. + -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/