On Mon, Apr 03, 2017 at 11:38:56AM +0200, Michał Kępień wrote: > This series contains a few cleanups for the call_fext_func() function. > It does not depend on the previous series I submitted for > fujitsu-laptop, which is why I am sending it in parallel with the > backlight cleanup series. > > While patches 1 and 2 are tangible improvements, I will understand if > you deem patch 3 to be unnecessary churn. I decided to submit it anyway > as I think it might bring some benefit to a casual reader of the > module's code, but it is basically just a suggestion and as it is the > last patch, it can simply be omitted when the series is applied. > > drivers/platform/x86/fujitsu-laptop.c | 39 +++++++++++------------------------ > 1 file changed, 12 insertions(+), 27 deletions(-) These all look like worthwhile changes to me. Jonathan, are you aware of any usage of the arguments in 3/3 that would argue against changing their names to the more specific ones? Pushed to the temporary fujitsu branch for testing, awaiting ack from Jonathan. -- Darren Hart VMware Open Source Technology Center
