cpufreq: powernv: Don't validate the frequency table twice

The cpufreq core is already validating the CPU frequency table after
calling the ->init() callback of the cpufreq drivers and the drivers
don't need to do the same anymore. Though they need to set the
policy->freq_table field directly from the ->init() callback now.

Stop validating the frequency table from powernv driver.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

diff --git a/drivers/cpufreq/powernv-cpufreq.c b/drivers/cpufreq/powernv-cpufreq.c
index 29cdec198657b714174a4fa173ee1e5085e1cf10..0591874856d3c8ba037537bc863c4bbbfe21e0a5 100644 (file)
--- a/drivers/cpufreq/powernv-cpufreq.c
+++ b/drivers/cpufreq/powernv-cpufreq.c
@@ -812,7 +812,7 @@ gpstates_done:
 
 static int powernv_cpufreq_cpu_init(struct cpufreq_policy *policy)
 {
-       int base, i, ret;
+       int base, i;
        struct kernfs_node *kn;
        struct global_pstate_info *gpstates;
 
@@ -848,15 +848,10 @@ static int powernv_cpufreq_cpu_init(struct cpufreq_policy *policy)
        gpstates->timer.expires = jiffies +
                                msecs_to_jiffies(GPSTATE_TIMER_INTERVAL);
        spin_lock_init(&gpstates->gpstate_lock);
-       ret = cpufreq_table_validate_and_show(policy, powernv_freqs);
-
-       if (ret < 0) {
-               kfree(policy->driver_data);
-               return ret;
-       }
 
+       policy->freq_table = powernv_freqs;
        policy->fast_switch_possible = true;
-       return ret;
+       return 0;
 }
 
 static int powernv_cpufreq_cpu_exit(struct cpufreq_policy *policy)