Re: [PATCH 1/2] cpufreq: arm_big_little: make vexpress driver dependent on bL core driver

On Friday, March 14, 2014 12:10:54 PM Viresh Kumar wrote:

Currently vexpress big LITTLE driver selects ARM_BIG_LITTLE_CPUFREQ and so if CONFIG_BIG_LITTLE isn't enabled and CONFIG_ARM_VEXPRESS_SPC_CPUFREQ is enabled we get below warnings while compiling:

warning: (ARM_VEXPRESS_SPC_CPUFREQ) selects ARM_BIG_LITTLE_CPUFREQ which has unmet direct dependencies (ARCH_HAS_CPUFREQ && CPU_FREQ && (ARM || ARM64) && ARM && BIG_LITTLE && ARM_CPU_TOPOLOGY && HAVE_CLK)

To fix this make ARM_VEXPRESS_SPC_CPUFREQ dependent on ARM_BIG_LITTLE_CPUFREQ instead of selecting it.

This also moves entry for ARM_VEXPRESS_SPC_CPUFREQ along with other big LITTLE config entries.

Signed-off-by: Viresh Kumar viresh.kumar@linaro.org

Hi Rafael,

Hi,

Both of these are fixes, please see if they can make it to 3.14 only.

Quite frankly, I'm afraid they won't make it.

I'm sending a pull request for -rc7 in a while and I won't add anything to it without running it through linux-next at least for a day.

And since I'm expecting no -rc8 in this cycle, I have no plans for any pull requests next week. At least not for 3.14.

Also I'm going to travel during the last week of March, so probably I'll send my first pull request for the 3.15 merge window before departure.

Thanks, Rafael

drivers/cpufreq/Kconfig.arm | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/drivers/cpufreq/Kconfig.arm b/drivers/cpufreq/Kconfig.arm index 3129749..9fb6270 100644 --- a/drivers/cpufreq/Kconfig.arm +++ b/drivers/cpufreq/Kconfig.arm @@ -2,6 +2,7 @@ # ARM CPU Frequency scaling drivers # +# big LITTLE core layer and glue drivers config ARM_BIG_LITTLE_CPUFREQ tristate "Generic ARM big LITTLE CPUfreq driver" depends on ARM && BIG_LITTLE && ARM_CPU_TOPOLOGY && HAVE_CLK @@ -16,6 +17,14 @@ config ARM_DT_BL_CPUFREQ This enables probing via DT for Generic CPUfreq driver for ARM big.LITTLE platform. This gets frequency tables from DT. +config ARM_VEXPRESS_SPC_CPUFREQ

•    tristate "Versatile Express SPC based CPUfreq driver"
  

• depends on ARM_BIG_LITTLE_CPUFREQ && ARCH_VEXPRESS_SPC

•    help
  

•      This add the CPUfreq driver support for Versatile Express
  

•  big.LITTLE platforms using SPC for power management.
  

config ARM_EXYNOS_CPUFREQ bool @@ -241,11 +250,3 @@ config ARM_TEGRA_CPUFREQ default y help This adds the CPUFreq driver support for TEGRA SOCs.

-config ARM_VEXPRESS_SPC_CPUFREQ

•    tristate "Versatile Express SPC based CPUfreq driver"
  

•    select ARM_BIG_LITTLE_CPUFREQ
  

•    depends on ARCH_VEXPRESS_SPC
  

•    help
  

•      This add the CPUfreq driver support for Versatile Express
  

•  big.LITTLE platforms using SPC for power management.