On 14/12/15 13:56, Rob Herring wrote: > On Wed, Dec 2, 2015 at 3:33 AM, Vladimir Murzin <vladimir.murzin@xxxxxxx> 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 <vladimir.murzin@xxxxxxx> >> --- >> 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 > > > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html