[PATCH v2 1/1] clk: imx: scu: fix MXC_CLK_SCU module build break

Stephen Boyd sboyd at kernel.org
Mon Dec 7 16:55:21 EST 2020


Quoting Dong Aisheng (2020-11-30 00:46:24)
> This issue can be reproduced by CONFIG_IMX_MBOX=m and CONFIG_MXC_CLK_SCU=m.
> It's caused by current Makefile writing can't support build clk-scu.o and
> clk-imx8qxp.o independently. (e.g. MXC_CLK_SCU = y while CLK_IMX8QXP = n)
> 
> "obj-$(CONFIG_MXC_CLK_SCU) += clk-imx-scu.o clk-imx-lpcg-scu.o
> clk-imx-scu-$(CONFIG_CLK_IMX8QXP) += clk-scu.o clk-imx8qxp.o"
> 
> Only enable MXC_CLK_SCU while CLK_IMX8QXP not will cause unable to locate
> the input files during linking.
> 
> Error log:
>   LD [M]  drivers/clk/imx/clk-imx-scu.o
> arm-poky-linux-gnueabi-ld: no input files
> ../scripts/Makefile.build:434: recipe for target 'drivers/clk/imx/clk-imx-scu.o' failed
> make[4]: *** [drivers/clk/imx/clk-imx-scu.o] Error 1
> 
> This patch makes MXC_CLK_SCU to be invisible to users and can only be
> selected by CLK_IMX8QXP option to ensure they're built together.
> 
> And COMPILE_TEST on an option that isn't selectable is meaningless.
> We can remove it from MXC_CLK_SCU because CLK_IMX8QXP selecting MXC_CLK_SCU
> already has COMPILE_TEST.
> 
> Fixes: e0d0d4d86c766 ("clk: imx8qxp: Support building i.MX8QXP clock driver as module")
> Acked-by: Sebastian Andrzej Siewior <bigeasy at linutronix.de>
> Signed-off-by: Dong Aisheng <aisheng.dong at nxp.com>
> ---

Applied to clk-fixes.



More information about the linux-arm-kernel mailing list