cpufreq: Disallow ->resolve_freq() for drivers providing ->target_index()
authorViresh Kumar <viresh.kumar@linaro.org>
Thu, 21 Jul 2016 21:39:26 +0000 (14:39 -0700)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 21 Jul 2016 21:45:17 +0000 (23:45 +0200)
The handlers provided by cpufreq core are sufficient for resolving the
frequency for drivers providing ->target_index(), as the core already
has the frequency table and so ->resolve_freq() isn't required for such
platforms.

This patch disallows drivers with ->target_index() callback to use the
->resolve_freq() callback.

Also, it fixes a potential kernel crash for drivers providing ->target()
but no ->resolve_freq().

Fixes: e3c062360870 "cpufreq: add cpufreq_driver_resolve_freq()"
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/cpufreq/cpufreq.c

index b696baeb249d847f83ce24f0ec5fd9db60d65503..3ef9be3965ff16a14f62e1113c8bc0f6dcb45421 100644 (file)
@@ -507,12 +507,20 @@ unsigned int cpufreq_driver_resolve_freq(struct cpufreq_policy *policy,
 {
        target_freq = clamp_val(target_freq, policy->min, policy->max);
        policy->cached_target_freq = target_freq;
+
+       if (cpufreq_driver->target_index) {
+               int idx;
+
+               idx = cpufreq_frequency_table_target(policy, target_freq,
+                                                    CPUFREQ_RELATION_L);
+               policy->cached_resolved_idx = idx;
+               return policy->freq_table[idx].frequency;
+       }
+
        if (cpufreq_driver->resolve_freq)
                return cpufreq_driver->resolve_freq(policy, target_freq);
-       policy->cached_resolved_idx =
-               cpufreq_frequency_table_target(policy, target_freq,
-                                              CPUFREQ_RELATION_L);
-       return policy->freq_table[policy->cached_resolved_idx].frequency;
+
+       return target_freq;
 }
 
 /*********************************************************************