On Apr 2, 2014, Viresh Kumar viresh.kumar@linaro.org wrote:
did some changes to driver and by mistake made cpuclk as a 'static' local variable, which wasn't actually required. Fix it.
FWIW, the same mistake is present in at32.
I've just finished bisecting the cpufreq regression on loongson2. Indeed, this s/static// patch does not fix it, but it was 652ed95d5fa, the patch that moved cpuclk into loongson2_cpufreq_cpu_init, that caused the regression.
Reverting all the changes to loongson2_cpufreq.c in 652ed95d5fa makes cpufreq work again, as opposed to printing nan% in all cpufreq stats in cpufreq-info's output, and freezing shortly thereafter.
- static struct clk *cpuclk;
- struct clk *cpuclk;