| Message ID | 20191023110811.15086-1-sudeep.holla@arm.com |
|---|---|
| State | Accepted |
| Commit | 4a6e135238798144ca8a2eab65018521c66240da |
| Headers | show |
| Series | [1/2] cpufreq: vexpress-spc: use macros instead of hardcoded values for cluster ids | expand |
On 23-10-19, 12:08, Sudeep Holla wrote: > A15 and A7 cluster identifiers are fixed to 0 and 1 respectively. There are > macros for the same and used in most of the places except this instance. > > Lets use macros instead of hardcoded values for cluster ids even here. > > Cc: Viresh Kumar <viresh.kumar@linaro.org> > Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net> > Signed-off-by: Sudeep Holla <sudeep.holla@arm.com> > --- > drivers/cpufreq/vexpress-spc-cpufreq.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) Applied both. Thanks. -- viresh
diff --git a/drivers/cpufreq/vexpress-spc-cpufreq.c b/drivers/cpufreq/vexpress-spc-cpufreq.c index 3259498d7eaa..093ef8d3a8d4 100644 --- a/drivers/cpufreq/vexpress-spc-cpufreq.c +++ b/drivers/cpufreq/vexpress-spc-cpufreq.c @@ -380,8 +380,9 @@ static int get_cluster_clk_and_freq_table(struct device *cpu_dev, goto put_clusters; /* Assuming 2 cluster, set clk_big_min and clk_little_max */ - clk_big_min = get_table_min(freq_table[0]); - clk_little_max = VIRT_FREQ(1, get_table_max(freq_table[1])); + clk_big_min = get_table_min(freq_table[A15_CLUSTER]); + clk_little_max = VIRT_FREQ(A7_CLUSTER, + get_table_max(freq_table[A7_CLUSTER])); return 0;
A15 and A7 cluster identifiers are fixed to 0 and 1 respectively. There are macros for the same and used in most of the places except this instance. Lets use macros instead of hardcoded values for cluster ids even here. Cc: Viresh Kumar <viresh.kumar@linaro.org> Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com> --- drivers/cpufreq/vexpress-spc-cpufreq.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) -- 2.17.1