Messages in this thread | | | Date | Tue, 8 Sep 2015 07:30:44 +0530 | From | Viresh Kumar <> | Subject | Re: [PATCH V2 4/9] cpufreq: governor: Drop __gov_queue_work() |
| |
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 <viresh.kumar@linaro.org> > > Quite frankly I don't see the point.
But isn't that just an unnecessary wrapper ?
> I'd even remove the inline from its definition and let the compiler decide > what to do with it.
What if the compiler decides to link it? Why add a function call for (almost) no use?
-- viresh
| |