From mboxrd@z Thu Jan 1 00:00:00 1970 From: vladimir.murzin@arm.com (Vladimir Murzin) Date: Tue, 15 Dec 2015 13:16:11 +0000 Subject: [PATCH v1 02/10] clockevents/drivers: add MPS2 Timer driver In-Reply-To: References: <1449048790-25859-1-git-send-email-vladimir.murzin@arm.com> <1449048790-25859-3-git-send-email-vladimir.murzin@arm.com> Message-ID: <5670129B.700@arm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On 14/12/15 13:56, Rob Herring wrote: > On Wed, Dec 2, 2015 at 3:33 AM, Vladimir Murzin wrote: >> MPS2 platform has simple 32 bits general purpose countdown timers. >> >> The driver uses the first detected timer as a clocksource and the rest >> of the timers as a clockevent >> >> Signed-off-by: Vladimir Murzin >> --- >> drivers/clocksource/Kconfig | 5 + >> drivers/clocksource/Makefile | 1 + >> drivers/clocksource/mps2-timer.c | 277 ++++++++++++++++++++++++++++++++++++++ >> 3 files changed, 283 insertions(+) >> create mode 100644 drivers/clocksource/mps2-timer.c >> >> diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig >> index 2eb5f0e..8bca09c 100644 >> --- a/drivers/clocksource/Kconfig >> +++ b/drivers/clocksource/Kconfig >> @@ -137,6 +137,11 @@ config CLKSRC_STM32 >> depends on OF && ARM && (ARCH_STM32 || COMPILE_TEST) >> select CLKSRC_MMIO >> >> +config CLKSRC_MPS2 >> + bool "Clocksource for MPS2 SoCs" if COMPILE_TEST >> + depends on OF && ARM > > Does this really depend on both of these for COMPILE_TEST? Not really... > > You need to select CLKSRC_OF rather than CLKSRC_MMIO as well. > Does it looks better? config CLKSRC_MPS2 bool "Clocksource for MPS2 SoCs" if COMPILE_TEST depends on GENERIC_SCHED_CLOCK select CLKSRC_MMIO select CLKSRC_OF Cheers Vladimir > Rob > > >