Re: [PATCH 2/2] cpufreq: governor: CPUFREQ_GOV_STOP never fails

2016-05-12 Thread Viresh Kumar
On 12-05-16, 15:14, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki 
> 
> None of the cpufreq governors currently in the tree will ever fail
> an invocation of the ->governor() callback with the event argument
> equal to CPUFREQ_GOV_STOP (unless invoked with incorrect arguments
> which doesn't matter anyway) and it is rather difficult to imagine
> a valid reason for such a failure.
> 
> Accordingly, rearrange the code in the core to make it clear that
> this call never fails.
> 
> Signed-off-by: Rafael J. Wysocki 
> ---
>  drivers/cpufreq/cpufreq.c |   40 +++-
>  1 file changed, 11 insertions(+), 29 deletions(-)

Acked-by: Viresh Kumar 

-- 
viresh


Re: [PATCH 2/2] cpufreq: governor: CPUFREQ_GOV_STOP never fails

2016-05-12 Thread Viresh Kumar
On 12-05-16, 15:14, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki 
> 
> None of the cpufreq governors currently in the tree will ever fail
> an invocation of the ->governor() callback with the event argument
> equal to CPUFREQ_GOV_STOP (unless invoked with incorrect arguments
> which doesn't matter anyway) and it is rather difficult to imagine
> a valid reason for such a failure.
> 
> Accordingly, rearrange the code in the core to make it clear that
> this call never fails.
> 
> Signed-off-by: Rafael J. Wysocki 
> ---
>  drivers/cpufreq/cpufreq.c |   40 +++-
>  1 file changed, 11 insertions(+), 29 deletions(-)

Acked-by: Viresh Kumar 

-- 
viresh