phy: allwinner: Make PHY_SUN6I_MIPI_DPHY depend on COMMON_CLK
authorTiezhu Yang <yangtiezhu@loongson.cn>
Tue, 7 Jul 2020 09:22:26 +0000 (17:22 +0800)
committerVinod Koul <vkoul@kernel.org>
Mon, 13 Jul 2020 05:10:53 +0000 (10:40 +0530)
When CONFIG_ARCH_SUNXI is not set but CONFIG_COMPILE_TEST=y,
CONFIG_HAVE_CLK=y, CONFIG_HAVE_LEGACY_CLK=y, there exists
the following build errors with CONFIG_PHY_SUN6I_MIPI_DPHY=y:

drivers/phy/allwinner/phy-sun6i-mipi-dphy.o: In function `sun6i_dphy_init':
phy-sun6i-mipi-dphy.c:(.text+0x320): undefined reference to `clk_set_rate_exclusive'
drivers/phy/allwinner/phy-sun6i-mipi-dphy.o: In function `sun6i_dphy_exit':
phy-sun6i-mipi-dphy.c:(.text+0x2c8): undefined reference to `clk_rate_exclusive_put'

clk_set_rate_exclusive() and clk_rate_exclusive_put() are defined
in drivers/clk/clk.c, this file is built under CONFIG_COMMON_CLK,
so in order to build drivers/phy/allwinner/phy-sun6i-mipi-dphy.c
successful used with various configs, CONFIG_PHY_SUN6I_MIPI_DPHY
should depend on CONFIG_COMMON_CLK.

Fixes: 133552bf03ed ("phy: Remove CONFIG_ARCH_* check for related subdir in Makefile")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
Link: https://lore.kernel.org/r/1594113746-25393-1-git-send-email-yangtiezhu@loongson.cn
Signed-off-by: Vinod Koul <vkoul@kernel.org>
drivers/phy/allwinner/Kconfig

index e760d89..fb58451 100644 (file)
@@ -22,7 +22,7 @@ config PHY_SUN4I_USB
 config PHY_SUN6I_MIPI_DPHY
        tristate "Allwinner A31 MIPI D-PHY Support"
        depends on ARCH_SUNXI || COMPILE_TEST
-       depends on HAS_IOMEM
+       depends on HAS_IOMEM && COMMON_CLK
        depends on RESET_CONTROLLER
        select GENERIC_PHY
        select GENERIC_PHY_MIPI_DPHY