Re: [PATCH] clk: davinci: add Kconfig symbol

[Brian Masney]

Hi Rosen,

On Sun, Apr 26, 2026 at 10:23:54PM -0700, Rosen Penev wrote:
> Add COMPILE_TEST support as a result so that the buildbots can run
> through this driver. There's nothing architecture specific about it.
> 
> Signed-off-by: Rosen Penev <rosenp@xxxxxxxxx>

It'd be nice if the commit title and/or the commit description mentioned
that this is for an existing driver in the tree.

> ---
>  drivers/clk/Kconfig          | 7 +++++++
>  drivers/clk/Makefile         | 2 +-
>  drivers/clk/davinci/Makefile | 6 +++---
>  3 files changed, 11 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/clk/Kconfig b/drivers/clk/Kconfig
> index 06a7ac971edf..42d84c0ed5fc 100644
> --- a/drivers/clk/Kconfig
> +++ b/drivers/clk/Kconfig
> @@ -317,6 +317,13 @@ config COMMON_CLK_AXI_CLKGEN
>  	  Support for the Analog Devices axi-clkgen pcore clock generator for Xilinx
>  	  FPGAs. It is commonly used in Analog Devices' reference designs.
>  
> +config CLK_DA850
> +	bool "DaVinci DA850 CLK support"
> +	depends on ARCH_DAVINCI_DA850 || COMPILE_TEST

The clk driver is needed for this platform, so you can make it easier
for someone to configure their kernel like this:

	depends on ARCH_ARM || COMPILE_TEST
	default ARCH_DAVINCI_DA850
	
Brian