cpufreq: e_powersaver: Don't validate the frequency table twice
authorViresh Kumar <viresh.kumar@linaro.org>
Mon, 26 Feb 2018 05:08:51 +0000 (10:38 +0530)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 20 Mar 2018 11:07:52 +0000 (12:07 +0100)
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 e_powersaver driver.

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

index cdf097b..60bea30 100644 (file)
@@ -184,7 +184,6 @@ static int eps_cpu_init(struct cpufreq_policy *policy)
        struct cpuinfo_x86 *c = &cpu_data(0);
        struct cpufreq_frequency_table *f_table;
        int k, step, voltage;
-       int ret;
        int states;
 #if IS_ENABLED(CONFIG_ACPI_PROCESSOR)
        unsigned int limit;
@@ -359,12 +358,7 @@ static int eps_cpu_init(struct cpufreq_policy *policy)
        }
 
        policy->cpuinfo.transition_latency = 140000; /* 844mV -> 700mV in ns */
-
-       ret = cpufreq_table_validate_and_show(policy, &centaur->freq_table[0]);
-       if (ret) {
-               kfree(centaur);
-               return ret;
-       }
+       policy->freq_table = &centaur->freq_table[0];
 
        return 0;
 }