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

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

index 7281a2c19c362177a57a962f8d587ef40764b92e..6cdac1aaf23c5de81869f15d5591bb23f2beacd2 100644 (file)
@@ -902,11 +902,7 @@ static int brcm_avs_cpufreq_init(struct cpufreq_policy *policy)
                return ret;
        }
 
-       ret = cpufreq_table_validate_and_show(policy, freq_table);
-       if (ret) {
-               dev_err(dev, "invalid frequency table: %d\n", ret);
-               return ret;
-       }
+       policy->freq_table = freq_table;
 
        /* All cores share the same clock and thus the same policy. */
        cpumask_setall(policy->cpus);