cpufreq: mediatek: Don't validate the frequency table twice
authorViresh Kumar <viresh.kumar@linaro.org>
Mon, 26 Feb 2018 05:08:55 +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 mediatek driver.

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

index 84d658d5702997d43964e944b19970a6cdbf388b..eb8920d3981815f930d55465b5318c4356cca37b 100644 (file)
@@ -460,21 +460,12 @@ static int mtk_cpufreq_init(struct cpufreq_policy *policy)
                return ret;
        }
 
-       ret = cpufreq_table_validate_and_show(policy, freq_table);
-       if (ret) {
-               pr_err("%s: invalid frequency table: %d\n", __func__, ret);
-               goto out_free_cpufreq_table;
-       }
-
        cpumask_copy(policy->cpus, &info->cpus);
+       policy->freq_table = freq_table;
        policy->driver_data = info;
        policy->clk = info->cpu_clk;
 
        return 0;
-
-out_free_cpufreq_table:
-       dev_pm_opp_free_cpufreq_table(info->cpu_dev, &freq_table);
-       return ret;
 }
 
 static int mtk_cpufreq_exit(struct cpufreq_policy *policy)