cpufreq: fix compile-test defaults
authorJohan Hovold <johan+linaro@kernel.org>
Thu, 17 Apr 2025 07:28:38 +0000 (09:28 +0200)
committerViresh Kumar <viresh.kumar@linaro.org>
Thu, 17 Apr 2025 08:06:29 +0000 (13:36 +0530)
Commit 3f66425a4fc8 ("cpufreq: Enable COMPILE_TEST on Arm drivers")
enabled compile testing of most Arm CPUFreq drivers but left the
existing default values unchanged so that many drivers are enabled by
default whenever COMPILE_TEST is selected.

This specifically results in the S3C64XX CPUFreq driver being enabled
and initialised during boot of non-S3C64XX platforms with the following
error logged:

cpufreq: Unable to obtain ARMCLK: -2

Commit d4f610a9bafd ("cpufreq: Do not enable by default during compile
testing") recently fixed most of the default values, but two entries
were missed and two could use a more specific default condition.

Fix the default values for drivers that can be compile tested and that
should be enabled by default when not compile testing.

Fixes: 3f66425a4fc8 ("cpufreq: Enable COMPILE_TEST on Arm drivers")
Cc: Rob Herring (Arm) <robh@kernel.org>
Signed-off-by: Johan Hovold <johan+linaro@kernel.org>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
drivers/cpufreq/Kconfig.arm

index d4d625d..0d46402 100644 (file)
@@ -76,7 +76,7 @@ config ARM_VEXPRESS_SPC_CPUFREQ
 config ARM_BRCMSTB_AVS_CPUFREQ
        tristate "Broadcom STB AVS CPUfreq driver"
        depends on (ARCH_BRCMSTB && !ARM_SCMI_CPUFREQ) || COMPILE_TEST
-       default ARCH_BRCMSTB
+       default y if ARCH_BRCMSTB && !ARM_SCMI_CPUFREQ
        help
          Some Broadcom STB SoCs use a co-processor running proprietary firmware
          ("AVS") to handle voltage and frequency scaling. This driver provides
@@ -88,7 +88,7 @@ config ARM_HIGHBANK_CPUFREQ
        tristate "Calxeda Highbank-based"
        depends on ARCH_HIGHBANK || COMPILE_TEST
        depends on CPUFREQ_DT && REGULATOR && PL320_MBOX
-       default m
+       default m if ARCH_HIGHBANK
        help
          This adds the CPUFreq driver for Calxeda Highbank SoC
          based boards.
@@ -133,7 +133,7 @@ config ARM_MEDIATEK_CPUFREQ
 config ARM_MEDIATEK_CPUFREQ_HW
        tristate "MediaTek CPUFreq HW driver"
        depends on ARCH_MEDIATEK || COMPILE_TEST
-       default m
+       default m if ARCH_MEDIATEK
        help
          Support for the CPUFreq HW driver.
          Some MediaTek chipsets have a HW engine to offload the steps
@@ -256,7 +256,7 @@ config ARM_TEGRA194_CPUFREQ
        tristate "Tegra194 CPUFreq support"
        depends on ARCH_TEGRA_194_SOC || ARCH_TEGRA_234_SOC || (64BIT && COMPILE_TEST)
        depends on TEGRA_BPMP
-       default ARCH_TEGRA
+       default ARCH_TEGRA_194_SOC || ARCH_TEGRA_234_SOC
        help
          This adds CPU frequency driver support for Tegra194 SOCs.