Re: [PATCH V2 4/9] cpufreq: governor: Drop __gov_queue_work()

2015-09-08 Thread Rafael J. Wysocki
On Tuesday, September 08, 2015 07:30:44 AM Viresh Kumar wrote: > On 08-09-15, 03:15, Rafael J. Wysocki wrote: > > On Monday, July 27, 2015 05:58:09 PM Viresh Kumar wrote: > > > __gov_queue_work() isn't required anymore and can be merged with > > > gov_queue_work(). Do it. > > > > > >

Re: [PATCH V2 4/9] cpufreq: governor: Drop __gov_queue_work()

2015-09-08 Thread Rafael J. Wysocki
On Tuesday, September 08, 2015 07:30:44 AM Viresh Kumar wrote: > On 08-09-15, 03:15, Rafael J. Wysocki wrote: > > On Monday, July 27, 2015 05:58:09 PM Viresh Kumar wrote: > > > __gov_queue_work() isn't required anymore and can be merged with > > > gov_queue_work(). Do it. > > > > > >

Re: [PATCH V2 4/9] cpufreq: governor: Drop __gov_queue_work()

2015-09-07 Thread Viresh Kumar
On 08-09-15, 03:15, Rafael J. Wysocki wrote: > On Monday, July 27, 2015 05:58:09 PM Viresh Kumar wrote: > > __gov_queue_work() isn't required anymore and can be merged with > > gov_queue_work(). Do it. > > > > Signed-off-by: Viresh Kumar > > Quite frankly I don't see the point. But isn't that

Re: [PATCH V2 4/9] cpufreq: governor: Drop __gov_queue_work()

2015-09-07 Thread Rafael J. Wysocki
On Monday, July 27, 2015 05:58:09 PM Viresh Kumar wrote: > __gov_queue_work() isn't required anymore and can be merged with > gov_queue_work(). Do it. > > Signed-off-by: Viresh Kumar Quite frankly I don't see the point. I'd even remove the inline from its definition and let the compiler decide

Re: [PATCH V2 4/9] cpufreq: governor: Drop __gov_queue_work()

2015-09-07 Thread Viresh Kumar
On 08-09-15, 03:15, Rafael J. Wysocki wrote: > On Monday, July 27, 2015 05:58:09 PM Viresh Kumar wrote: > > __gov_queue_work() isn't required anymore and can be merged with > > gov_queue_work(). Do it. > > > > Signed-off-by: Viresh Kumar > > Quite frankly I don't see

Re: [PATCH V2 4/9] cpufreq: governor: Drop __gov_queue_work()

2015-09-07 Thread Rafael J. Wysocki
On Monday, July 27, 2015 05:58:09 PM Viresh Kumar wrote: > __gov_queue_work() isn't required anymore and can be merged with > gov_queue_work(). Do it. > > Signed-off-by: Viresh Kumar Quite frankly I don't see the point. I'd even remove the inline from its definition

[PATCH V2 4/9] cpufreq: governor: Drop __gov_queue_work()

2015-07-27 Thread Viresh Kumar
__gov_queue_work() isn't required anymore and can be merged with gov_queue_work(). Do it. Signed-off-by: Viresh Kumar --- drivers/cpufreq/cpufreq_governor.c | 17 ++--- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/drivers/cpufreq/cpufreq_governor.c

[PATCH V2 4/9] cpufreq: governor: Drop __gov_queue_work()

2015-07-27 Thread Viresh Kumar
__gov_queue_work() isn't required anymore and can be merged with gov_queue_work(). Do it. Signed-off-by: Viresh Kumar viresh.ku...@linaro.org --- drivers/cpufreq/cpufreq_governor.c | 17 ++--- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git