Re: [PATCH] clk: imx: Select MXC_CLK for CLK_IMX8QXP

[Randy Dunlap]

On 9/21/23 02:36, Abel Vesa wrote:
> If the i.MX8QXP clock provider is built-in but the MXC_CLK is
> built as module, build fails:
> 
> aarch64-linux-ld: drivers/clk/imx/clk-imx8-acm.o: in function `imx8_acm_clk_probe':
> clk-imx8-acm.c:(.text+0x3d0): undefined reference to `imx_check_clk_hws'
> 
> Fix that by selecting MXC_CLK in case of CLK_IMX8QXP.
> 
> Fixes: c2cccb6d0b33 ("clk: imx: add imx8qxp clk driver")
> Closes: https://lore.kernel.org/all/8b77219e-b59e-40f1-96f1-980a0b2debcf@xxxxxxxxxxxxx/
> Reported-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> Signed-off-by: Abel Vesa <abel.vesa@xxxxxxxxxx>

Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
Tested-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>

Thanks.

> ---
>  drivers/clk/imx/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/clk/imx/Kconfig b/drivers/clk/imx/Kconfig
> index f6b82e0b9703..db3bca5f4ec9 100644
> --- a/drivers/clk/imx/Kconfig
> +++ b/drivers/clk/imx/Kconfig
> @@ -96,6 +96,7 @@ config CLK_IMX8QXP
>  	depends on (ARCH_MXC && ARM64) || COMPILE_TEST
>  	depends on IMX_SCU && HAVE_ARM_SMCCC
>  	select MXC_CLK_SCU
> +	select MXC_CLK
>  	help
>  	  Build the driver for IMX8QXP SCU based clocks.
>  

-- 
~Randy