On 14/12/15 13:56, Rob Herring wrote:

On Wed, Dec 2, 2015 at 3:33 AM, Vladimir Murzin [off-list ref] wrote:

quoted

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 <redacted>
---
 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