Re: [PATCH][RFC] cpufreq: Avoid warning during resume by return EAGAIN if cpufreq is unavailable

From: Viresh Kumar
Date: Mon Jun 27 2016 - 03:20:35 EST


On 27-06-16, 03:12, Rafael J. Wysocki wrote:
> ---
> drivers/cpufreq/cpufreq.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> Index: linux-pm/drivers/cpufreq/cpufreq.c
> ===================================================================
> --- linux-pm.orig/drivers/cpufreq/cpufreq.c
> +++ linux-pm/drivers/cpufreq/cpufreq.c
> @@ -1544,9 +1544,6 @@ static unsigned int cpufreq_update_curre
> {
> unsigned int new_freq;
>
> - if (cpufreq_suspended)
> - return 0;
> -
> new_freq = cpufreq_driver->get(policy->cpu);
> if (!new_freq)
> return 0;
> @@ -2280,6 +2277,10 @@ int cpufreq_update_policy(unsigned int c
> * -> ask driver for current freq and notify governors about a change
> */
> if (cpufreq_driver->get && !cpufreq_driver->setpolicy) {
> + if (cpufreq_suspended) {
> + ret = -EAGAIN;
> + goto unlock;
> + }
> new_policy.cur = cpufreq_update_current_freq(policy);
> if (WARN_ON(!new_policy.cur)) {
> ret = -EIO;

Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>
--
viresh