cpufreq: pmac: don't initialize part of policy set by core
authorViresh Kumar <viresh.kumar@linaro.org>
Thu, 3 Oct 2013 14:58:54 +0000 (20:28 +0530)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 15 Oct 2013 22:50:31 +0000 (00:50 +0200)
Many common initializations of struct policy are moved to core now and hence
this driver doesn't need to do it. This patch removes such code.

Most recent of those changes is to call ->get() in the core after calling
->init().

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/cpufreq/pmac32-cpufreq.c
drivers/cpufreq/pmac64-cpufreq.c

index 8715ab99d80e13a5b6fe1df3049b5b395ff5bb77..672b7f4a7de56b7638c7679a4fd6015487ff3f44 100644 (file)
@@ -396,7 +396,6 @@ static int pmac_cpufreq_cpu_init(struct cpufreq_policy *policy)
                return -ENODEV;
 
        policy->cpuinfo.transition_latency      = transition_latency;
-       policy->cur = cur_freq;
 
        return cpufreq_table_validate_and_show(policy, pmac_cpu_freqs);
 }
index e853b852b34b7dadfb376fb7c84fc62308a0b915..79e40c91378ee35c1654b91d32811b70079cd9a3 100644 (file)
@@ -348,7 +348,6 @@ static unsigned int g5_cpufreq_get_speed(unsigned int cpu)
 static int g5_cpufreq_cpu_init(struct cpufreq_policy *policy)
 {
        policy->cpuinfo.transition_latency = transition_latency;
-       policy->cur = g5_cpu_freqs[g5_query_freq()].frequency;
        /* secondary CPUs are tied to the primary one by the
         * cpufreq core if in the secondary policy we tell it that
         * it actually must be one policy together with all others. */