aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorViresh Kumar <viresh.kumar@linaro.org>2018-02-26 00:08:49 -0500
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2018-03-20 07:07:52 -0400
commit5d8d4f9215ca830f76c6653998e1167198f64be0 (patch)
tree23bec1f706bf7ad87e04116f94ccd5e065ced752
parent3c1f5a462e799dde929411412e9f71d4e09efd16 (diff)
cpufreq: brcmstb: 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 brcmstb driver. Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r--drivers/cpufreq/brcmstb-avs-cpufreq.c6
1 files changed, 1 insertions, 5 deletions
diff --git a/drivers/cpufreq/brcmstb-avs-cpufreq.c b/drivers/cpufreq/brcmstb-avs-cpufreq.c
index 7281a2c19c36..6cdac1aaf23c 100644
--- a/drivers/cpufreq/brcmstb-avs-cpufreq.c
+++ b/drivers/cpufreq/brcmstb-avs-cpufreq.c
@@ -902,11 +902,7 @@ static int brcm_avs_cpufreq_init(struct cpufreq_policy *policy)
902 return ret; 902 return ret;
903 } 903 }
904 904
905 ret = cpufreq_table_validate_and_show(policy, freq_table); 905 policy->freq_table = freq_table;
906 if (ret) {
907 dev_err(dev, "invalid frequency table: %d\n", ret);
908 return ret;
909 }
910 906
911 /* All cores share the same clock and thus the same policy. */ 907 /* All cores share the same clock and thus the same policy. */
912 cpumask_setall(policy->cpus); 908 cpumask_setall(policy->cpus);