Commit 4f1f0bc8 authored by Ulf Hansson's avatar Ulf Hansson Committed by Sudeep Holla

cpufreq: scmi: Avoid one OF parsing in scmi_get_sharing_cpus()

The domain-id for the cpu_dev has already been parsed at the point when
scmi_get_sharing_cpus() is getting called. Let's pass it as an in-parameter
to avoid the unnecessary OF parsing.
Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
Acked-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Link: https://lore.kernel.org/r/20230825112633.236607-7-ulf.hansson@linaro.orgSigned-off-by: default avatarSudeep Holla <sudeep.holla@arm.com>
parent 9b578d83
...@@ -82,15 +82,12 @@ static int scmi_cpu_domain_id(struct device *cpu_dev) ...@@ -82,15 +82,12 @@ static int scmi_cpu_domain_id(struct device *cpu_dev)
} }
static int static int
scmi_get_sharing_cpus(struct device *cpu_dev, struct cpumask *cpumask) scmi_get_sharing_cpus(struct device *cpu_dev, int domain,
struct cpumask *cpumask)
{ {
int cpu, domain, tdomain; int cpu, tdomain;
struct device *tcpu_dev; struct device *tcpu_dev;
domain = scmi_cpu_domain_id(cpu_dev);
if (domain < 0)
return domain;
for_each_possible_cpu(cpu) { for_each_possible_cpu(cpu) {
if (cpu == cpu_dev->id) if (cpu == cpu_dev->id)
continue; continue;
...@@ -163,7 +160,7 @@ static int scmi_cpufreq_init(struct cpufreq_policy *policy) ...@@ -163,7 +160,7 @@ static int scmi_cpufreq_init(struct cpufreq_policy *policy)
} }
/* Obtain CPUs that share SCMI performance controls */ /* Obtain CPUs that share SCMI performance controls */
ret = scmi_get_sharing_cpus(cpu_dev, policy->cpus); ret = scmi_get_sharing_cpus(cpu_dev, domain, policy->cpus);
if (ret) { if (ret) {
dev_warn(cpu_dev, "failed to get sharing cpumask\n"); dev_warn(cpu_dev, "failed to get sharing cpumask\n");
goto out_free_cpumask; goto out_free_cpumask;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment