Re: [PATCH] cpufreq: governor: Drop gov_cancel_work()

From: Rafael J. Wysocki
Date: Thu Jun 09 2016 - 08:21:40 EST


On Thu, Jun 9, 2016 at 5:21 AM, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> On 09-06-16, 01:45, Rafael J. Wysocki wrote:
>> From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
>>
>> There's no reason for gov_cancel_work() to exist at all, as it only
>> has one caller and the only thing done by that caller is to invoke
>> gov_cancel_work().
>>
>> Accordingly, drop gov_cancel_work() and move its contents to the
>> caller.
>>
>> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
>> ---
>> drivers/cpufreq/cpufreq_governor.c | 19 +++++++------------
>> 1 file changed, 7 insertions(+), 12 deletions(-)
>
> Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>

Thanks!

> I am wondering what happened to your other patches around the governor callbacks
> and why weren't they applied yet :)

They are in linux-next now, aren't they?