thermal: cpu_cooling: fix improper order during initialization

The freq_table array is not populated before calling
thermal_of_cooling_register. The code which populates the freq table was
introduced in commit f6859014.
This should be done before registering new thermal cooling device.
The log shows effects of this wrong decision.
[ 2.172614] cpu cpu1: Failed to get voltage for frequency 1984518656000: -34
[ 2.220863] cpu cpu0: Failed to get voltage for frequency 1984524416000: -34

Cc: <stable@vger.kernel.org> # 3.19+
Fixes: f6859014c7e7 ("thermal: cpu_cooling: Store frequencies in descending order")
Signed-off-by: Lukasz Luba <lukasz.luba@arm.com>
Acked-by: Javi Merino <javi.merino@arm.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>

authored by Lukasz Luba and committed by Zhang Rui f840ab18 852f42a6

Changed files
+8 -8
drivers
thermal
+8 -8
drivers/thermal/cpu_cooling.c
··· 857 857 goto free_power_table; 858 858 } 859 859 860 - snprintf(dev_name, sizeof(dev_name), "thermal-cpufreq-%d", 861 - cpufreq_dev->id); 862 - 863 - cool_dev = thermal_of_cooling_device_register(np, dev_name, cpufreq_dev, 864 - &cpufreq_cooling_ops); 865 - if (IS_ERR(cool_dev)) 866 - goto remove_idr; 867 - 868 860 /* Fill freq-table in descending order of frequencies */ 869 861 for (i = 0, freq = -1; i <= cpufreq_dev->max_level; i++) { 870 862 freq = find_next_max(table, freq); ··· 868 876 else 869 877 pr_debug("%s: freq:%u KHz\n", __func__, freq); 870 878 } 879 + 880 + snprintf(dev_name, sizeof(dev_name), "thermal-cpufreq-%d", 881 + cpufreq_dev->id); 882 + 883 + cool_dev = thermal_of_cooling_device_register(np, dev_name, cpufreq_dev, 884 + &cpufreq_cooling_ops); 885 + if (IS_ERR(cool_dev)) 886 + goto remove_idr; 871 887 872 888 cpufreq_dev->clipped_freq = cpufreq_dev->freq_table[0]; 873 889 cpufreq_dev->cool_dev = cool_dev;