Message ID | 3cc5b83b-f81c-4bd7-b7ff-4d02db4e25d8@arm.com |
---|---|
State | New |
Headers | show |
Series | cpufreq: Fix initialization with disabled boost | expand |
diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index d7426e1d8bdd..e85139bd0436 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -1630,7 +1630,7 @@ static int cpufreq_online(unsigned int cpu) */ if (cpufreq_driver->set_boost && policy->boost_supported && (new_policy || !cpufreq_boost_enabled())) { - ret = policy_set_boost(policy, cpufreq_boost_enabled()); + ret = cpufreq_driver->set_boost(policy, cpufreq_boost_enabled()); if (ret) { /* If the set_boost fails, the online operation is not affected */ pr_info("%s: CPU%d: Cannot %s BOOST\n", __func__, policy->cpu,
The boost_enabled early return in policy_set_boost() caused the boost disabled at initialization to not actually set the initial policy->max, therefore effectively enabling boost while it should have been enabled. Fixes: 27241c8b63bd ("cpufreq: Introduce policy_set_boost()") Reported-by: Robin Murphy <robin.murphy@arm.com> Signed-off-by: Christian Loehle <christian.loehle@arm.com> --- drivers/cpufreq/cpufreq.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)