Re: [PATCH v2 5/9] clocksource/drivers/tcb_clksrc: move Kconfig option

[Daniel Lezcano]

Arnd can you ack this patch? I would like to take the series via my tree.


On 15/04/2019 17:08, Alexandre Belloni wrote:
> Move the ATMEL_TCB_CLKSRC option to drivers/clocksource and make it silent
> if COMPILE_TEST is not selected.
> 
> Signed-off-by: Alexandre Belloni <alexandre.belloni@xxxxxxxxxxx>
> ---
>  drivers/clocksource/Kconfig |  7 +++++++
>  drivers/misc/Kconfig        | 14 --------------
>  2 files changed, 7 insertions(+), 14 deletions(-)
> 
> diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig
> index 171502a356aa..e38ef4906e30 100644
> --- a/drivers/clocksource/Kconfig
> +++ b/drivers/clocksource/Kconfig
> @@ -410,6 +410,13 @@ config ATMEL_ST
>  	help
>  	  Support for the Atmel ST timer.
>  
> +config ATMEL_TCB_CLKSRC
> +	bool "Atmel TC Block timer driver" if COMPILE_TEST
> +	depends on HAS_IOMEM
> +	select TIMER_OF if OF
> +	help
> +	  Support for Timer Counter Blocks on Atmel SoCs.
> +
>  config CLKSRC_EXYNOS_MCT
>  	bool "Exynos multi core timer driver" if COMPILE_TEST
>  	depends on ARM || ARM64
> diff --git a/drivers/misc/Kconfig b/drivers/misc/Kconfig
> index 268a01d3d6f3..c84033909395 100644
> --- a/drivers/misc/Kconfig
> +++ b/drivers/misc/Kconfig
> @@ -59,20 +59,6 @@ config ATMEL_TCLIB
>  	  blocks found on many Atmel processors.  This facilitates using
>  	  these blocks by different drivers despite processor differences.
>  
> -config ATMEL_TCB_CLKSRC
> -	bool "TC Block Clocksource"
> -	depends on ARCH_AT91
> -	select TIMER_OF if OF
> -	default y
> -	help
> -	  Select this to get a high precision clocksource based on a
> -	  TC block with a 5+ MHz base clock rate.  Two timer channels
> -	  are combined to make a single 32-bit timer.
> -
> -	  When GENERIC_CLOCKEVENTS is defined, the third timer channel
> -	  may be used as a clock event device supporting oneshot mode
> -	  (delays of up to two seconds) based on the 32 KiHz clock.
> -
>  config DUMMY_IRQ
>  	tristate "Dummy IRQ handler"
>  	default n
> 


-- 
 <http://www.linaro.org/> Linaro.org â Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog