Messages in this thread | | | From | "Rafael J. Wysocki" <> | Subject | Re: [PATCH V2 1/5] cpufreq: Remove the redundant !setpolicy check | Date | Thu, 27 Jun 2019 23:52:31 +0200 |
| |
On Thursday, June 20, 2019 5:05:46 AM CEST Viresh Kumar wrote: > cpufreq_start_governor() is only called for !setpolicy case, checking it > again is not required. > > Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> > --- > drivers/cpufreq/cpufreq.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c > index 85ff958e01f1..54befd775bd6 100644 > --- a/drivers/cpufreq/cpufreq.c > +++ b/drivers/cpufreq/cpufreq.c > @@ -2153,7 +2153,7 @@ static int cpufreq_start_governor(struct cpufreq_policy *policy) > > pr_debug("%s: for CPU %u\n", __func__, policy->cpu); > > - if (cpufreq_driver->get && !cpufreq_driver->setpolicy) > + if (cpufreq_driver->get) > cpufreq_update_current_freq(policy); > > if (policy->governor->start) { >
Applied, thanks!
| |