Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1590041 > unrolled thread

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

Back to article view | Back to linux.kernel

Contents

  [PATCH v2 0/2] fujitsu-laptop: acpi_fujitsu_bl_notify() cleanup Michał Kępień <kernel@kempniu.pl> - 2017-03-01 07:50 +0100
    Re: [PATCH v2 0/2] fujitsu-laptop: acpi_fujitsu_bl_notify() cleanup Jonathan Woithe <jwoithe@just42.net> - 2017-03-02 00:40 +0100
    Re: [PATCH v2 0/2] fujitsu-laptop: acpi_fujitsu_bl_notify() cleanup Jonathan Woithe <jwoithe@just42.net> - 2017-03-06 00:30 +0100
      Re: [PATCH v2 0/2] fujitsu-laptop: acpi_fujitsu_bl_notify() cleanup Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-03-13 17:10 +0100

#1590041 — [PATCH v2 0/2] fujitsu-laptop: acpi_fujitsu_bl_notify() cleanup

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(-)

-- 
2.12.0

[toc] | [next] | [standalone]

#1590694

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.

These both look innoculous at first glance.  I will review them and test on
hardware within the next 48 hours.

Regards
  jonathan

[toc] | [prev] | [next] | [standalone]

#1592861

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@just42.net>
Reviewed-by: Jonathan Woithe <jwoithe@just42.net>

Regards
  jonathan

[toc] | [prev] | [next] | [standalone]

#1599556

On Mon, Mar 6, 2017 at 12:57 AM, Jonathan Woithe <jwoithe@just42.net> 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@just42.net>
> Reviewed-by: Jonathan Woithe <jwoithe@just42.net>

Pushed to testing, thanks!

-- 
With Best Regards,
Andy Shevchenko

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web