cpufreq: qcom-hw: add CONFIG_COMMON_CLK dependency
authorArnd Bergmann <arnd@arndb.de>
Thu, 15 Feb 2024 08:33:14 +0000 (09:33 +0100)
committerViresh Kumar <viresh.kumar@linaro.org>
Thu, 15 Feb 2024 08:38:23 +0000 (14:08 +0530)
It is still possible to compile-test a kernel without CONFIG_COMMON_CLK
for some ancient ARM boards or other architectures, but this causes a
link failure in the qcom-cpufreq-hw driver:

ERROR: modpost: "devm_clk_hw_register" [drivers/cpufreq/qcom-cpufreq-hw.ko] undefined!
ERROR: modpost: "devm_of_clk_add_hw_provider" [drivers/cpufreq/qcom-cpufreq-hw.ko] undefined!
ERROR: modpost: "of_clk_hw_onecell_get" [drivers/cpufreq/qcom-cpufreq-hw.ko] undefined!

Add a Kconfig dependency here to make sure this always work. Apparently
this bug has been in the kernel for a while without me running into it
on randconfig builds as COMMON_CLK is almost always enabled.

I have cross-checked by building an allmodconfig kernel with COMMON_CLK
disabled, which showed no other driver having this problem.

Fixes: 4370232c727b ("cpufreq: qcom-hw: Add CPU clock provider support")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
drivers/cpufreq/Kconfig.arm

index f911606897b8d1dcdb23e7dfdafc012f0879211e..a0ebad77666e305ad0c47a7473832d1a60d082fb 100644 (file)
@@ -173,6 +173,7 @@ config ARM_QCOM_CPUFREQ_NVMEM
 config ARM_QCOM_CPUFREQ_HW
        tristate "QCOM CPUFreq HW driver"
        depends on ARCH_QCOM || COMPILE_TEST
+       depends on COMMON_CLK
        help
          Support for the CPUFreq HW driver.
          Some QCOM chipsets have a HW engine to offload the steps