Re: [PATCH v2 0/2] fujitsu-laptop: acpi_fujitsu_bl_notify() cleanup

From: Jonathan Woithe
Date: Sun Mar 05 2017 - 18:26:02 EST


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>

Regards
jonathan