cpufreq: arm_scmi: Set fast_switch_possible conditionally
authorNicola Mazzucato <nicola.mazzucato@arm.com>
Wed, 17 Jun 2020 09:43:32 +0000 (10:43 +0100)
committerSudeep Holla <sudeep.holla@arm.com>
Tue, 30 Jun 2020 13:07:43 +0000 (14:07 +0100)
Currently the fast_switch_possible flag is set unconditionally to true.
Based on this, schedutil does not create a thread for frequency
switching and would always use the fast switch path.

However, if the platform does not support SCMI fast channel, we use
polling mode for SCMI message transfer. This may be possible only if
there is dedicated channel for DVFS and all operations are in polling
mode.

Update this by retrieving the fast_switch capability based on the
presence of fast channels in SCMI platform firmware.

Link: https://lore.kernel.org/r/20200617094332.8391-2-nicola.mazzucato@arm.com
Suggested-by: Lukasz Luba <lukasz.luba@arm.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Nicola Mazzucato <nicola.mazzucato@arm.com>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
drivers/cpufreq/scmi-cpufreq.c

index 61623e2ff14955ea4e5d4f966066ebe5b61eeef6..1cf688fcb56b9ebda60284d5bcea1bdcc7d51475 100644 (file)
@@ -198,7 +198,8 @@ static int scmi_cpufreq_init(struct cpufreq_policy *policy)
 
        policy->cpuinfo.transition_latency = latency;
 
-       policy->fast_switch_possible = true;
+       policy->fast_switch_possible =
+               handle->perf_ops->fast_switch_possible(handle, cpu_dev);
 
        em_register_perf_domain(policy->cpus, nr_opp, &em_cb);