Re: [PATCH v2 0/2] fujitsu-laptop: acpi_fujitsu_bl_notify() cleanup
From: Andy Shevchenko
Date: Mon Mar 13 2017 - 12:01:29 EST
On Mon, Mar 6, 2017 at 12:57 AM, Jonathan Woithe <jwoithe@xxxxxxxxxx> wrote:
> On Wed, Mar 01, 2017 at 07:42:52AM +0100, Micha?? K??pie?? wrote:
>> Here are two minor cleanups for acpi_fujitsu_bl_notify() that I came up
>> with while preparing the sparse keymap migration.
>>
>> Changes from v1:
>>
>> - Rebase on top of reworked Alan Jenkins' cleanup patch series.
>>
>> - Join integer variable declarations into a single line in patch 2/2.
>>
>> drivers/platform/x86/fujitsu-laptop.c | 64 +++++++++++++++--------------------
>> 1 file changed, 28 insertions(+), 36 deletions(-)
>
> These two clean ups, as their descriptions indicate, improve the clarity of
> the driver code and permit some minor optimisations. No regressions are
> evident when tested on S7020 hardware. Please apply.
>
> Tested-by: Jonathan Woithe <jwoithe@xxxxxxxxxx>
> Reviewed-by: Jonathan Woithe <jwoithe@xxxxxxxxxx>
Pushed to testing, thanks!
--
With Best Regards,
Andy Shevchenko