Re: [PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Darren Hart]

On Fri, Apr 07, 2017 at 09:41:35PM +0930, Jonathan Woithe wrote:
> On Wed, Apr 05, 2017 at 08:07:54PM -0700, Darren Hart wrote:
> > On Thu, Apr 06, 2017 at 10:15:14AM +0930, Jonathan Woithe wrote:
> > > On Wed, Apr 05, 2017 at 09:55:34PM +0200, Micha?? K??pie?? wrote:
> > > > > On Wed, Apr 05, 2017 at 08:48:59AM +0200, Micha?? K??pie?? wrote:
> > > > > > This series introduces further changes to the way LCD backlight is
> > > > > > handled by fujitsu-laptop.  These changes include fixing a bug in 
> > > > > > code
> > > > > > responsible for generating brightness-related input events, 
> > > > > > cleaning up
> > > > > > handling of module parameters, reducing code duplication, removing
> > > > > > superfluous debug messages and other fixes.
> > > > > > 
> > > > > > This series was tested on a Lifebook S7020 and a Lifebook E744.
> > > > > > 
> > > > > > This series is based on the testing branch as it requires earlier 
> > > > > > patch
> > > > > > series I submitted in order to apply cleanly.
> > > > > > 
> > > > > > Changes from v1:
> > > > > > 
> > > > > >   - Update debug message logged by set_lcd_level() to reflect code 
> > > > > > flow
> > > > > > changes introduced by patch 04/11.
> > > > > 
> > > > > Queued to testing, thanks! 
> > > > 
> > > > Jonathan, I would still love to hear your opinion on this series.  Take
> > > > your time, though, I do not see any reason to rush things.  I will only
> > > > send the next series once you ack this one.
> > > 
> > > Sure, no problem.  As mentioned earlier, this week has been busy.  I am
> > > hoping I might find the time to complete my review this evening.  If not, 
> > > it
> > > will be some time over the weekend.
> > 
> > Eeek, I jumped the gun on that. I've moved this from testing to fujitsu, and
> > it'll wait there until Jonathan gets a chance to review. Apologies Jonathan.
> 
> That's fine, these things happen.
> 
> I've completed a review of this series.  It represents another worthwhile
> evolutionary clean up of the fujitsu-laptop driver, this time concentrating
> on the backlight functions.  Importantly, it makes adjustments for changes
> in related kernel infrastructure as it has evolved over the last few
> years (the handling of module parameters, method return values and so on).
> 
> I am happy for this series to be merged.  Thanks Michael for your continued
> clean up efforts.
> 
> Reviewed-by: Jonathan Woithe 

Queued to testing, thanks!

-- 
Darren Hart
VMware Open Source Technology Center

Re: [PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Darren Hart]

On Fri, Apr 07, 2017 at 09:41:35PM +0930, Jonathan Woithe wrote:
> On Wed, Apr 05, 2017 at 08:07:54PM -0700, Darren Hart wrote:
> > On Thu, Apr 06, 2017 at 10:15:14AM +0930, Jonathan Woithe wrote:
> > > On Wed, Apr 05, 2017 at 09:55:34PM +0200, Micha?? K??pie?? wrote:
> > > > > On Wed, Apr 05, 2017 at 08:48:59AM +0200, Micha?? K??pie?? wrote:
> > > > > > This series introduces further changes to the way LCD backlight is
> > > > > > handled by fujitsu-laptop.  These changes include fixing a bug in 
> > > > > > code
> > > > > > responsible for generating brightness-related input events, 
> > > > > > cleaning up
> > > > > > handling of module parameters, reducing code duplication, removing
> > > > > > superfluous debug messages and other fixes.
> > > > > > 
> > > > > > This series was tested on a Lifebook S7020 and a Lifebook E744.
> > > > > > 
> > > > > > This series is based on the testing branch as it requires earlier 
> > > > > > patch
> > > > > > series I submitted in order to apply cleanly.
> > > > > > 
> > > > > > Changes from v1:
> > > > > > 
> > > > > >   - Update debug message logged by set_lcd_level() to reflect code 
> > > > > > flow
> > > > > > changes introduced by patch 04/11.
> > > > > 
> > > > > Queued to testing, thanks! 
> > > > 
> > > > Jonathan, I would still love to hear your opinion on this series.  Take
> > > > your time, though, I do not see any reason to rush things.  I will only
> > > > send the next series once you ack this one.
> > > 
> > > Sure, no problem.  As mentioned earlier, this week has been busy.  I am
> > > hoping I might find the time to complete my review this evening.  If not, 
> > > it
> > > will be some time over the weekend.
> > 
> > Eeek, I jumped the gun on that. I've moved this from testing to fujitsu, and
> > it'll wait there until Jonathan gets a chance to review. Apologies Jonathan.
> 
> That's fine, these things happen.
> 
> I've completed a review of this series.  It represents another worthwhile
> evolutionary clean up of the fujitsu-laptop driver, this time concentrating
> on the backlight functions.  Importantly, it makes adjustments for changes
> in related kernel infrastructure as it has evolved over the last few
> years (the handling of module parameters, method return values and so on).
> 
> I am happy for this series to be merged.  Thanks Michael for your continued
> clean up efforts.
> 
> Reviewed-by: Jonathan Woithe 

Queued to testing, thanks!

-- 
Darren Hart
VMware Open Source Technology Center

Re: [PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Jonathan Woithe]

On Wed, Apr 05, 2017 at 08:07:54PM -0700, Darren Hart wrote:
> On Thu, Apr 06, 2017 at 10:15:14AM +0930, Jonathan Woithe wrote:
> > On Wed, Apr 05, 2017 at 09:55:34PM +0200, Micha?? K??pie?? wrote:
> > > > On Wed, Apr 05, 2017 at 08:48:59AM +0200, Micha?? K??pie?? wrote:
> > > > > This series introduces further changes to the way LCD backlight is
> > > > > handled by fujitsu-laptop.  These changes include fixing a bug in code
> > > > > responsible for generating brightness-related input events, cleaning 
> > > > > up
> > > > > handling of module parameters, reducing code duplication, removing
> > > > > superfluous debug messages and other fixes.
> > > > > 
> > > > > This series was tested on a Lifebook S7020 and a Lifebook E744.
> > > > > 
> > > > > This series is based on the testing branch as it requires earlier 
> > > > > patch
> > > > > series I submitted in order to apply cleanly.
> > > > > 
> > > > > Changes from v1:
> > > > > 
> > > > >   - Update debug message logged by set_lcd_level() to reflect code 
> > > > > flow
> > > > > changes introduced by patch 04/11.
> > > > 
> > > > Queued to testing, thanks! 
> > > 
> > > Jonathan, I would still love to hear your opinion on this series.  Take
> > > your time, though, I do not see any reason to rush things.  I will only
> > > send the next series once you ack this one.
> > 
> > Sure, no problem.  As mentioned earlier, this week has been busy.  I am
> > hoping I might find the time to complete my review this evening.  If not, it
> > will be some time over the weekend.
> 
> Eeek, I jumped the gun on that. I've moved this from testing to fujitsu, and
> it'll wait there until Jonathan gets a chance to review. Apologies Jonathan.

That's fine, these things happen.

I've completed a review of this series.  It represents another worthwhile
evolutionary clean up of the fujitsu-laptop driver, this time concentrating
on the backlight functions.  Importantly, it makes adjustments for changes
in related kernel infrastructure as it has evolved over the last few
years (the handling of module parameters, method return values and so on).

I am happy for this series to be merged.  Thanks Michael for your continued
clean up efforts.

Reviewed-by: Jonathan Woithe 

Regards
  jonathan

Re: [PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Jonathan Woithe]

On Wed, Apr 05, 2017 at 08:07:54PM -0700, Darren Hart wrote:
> On Thu, Apr 06, 2017 at 10:15:14AM +0930, Jonathan Woithe wrote:
> > On Wed, Apr 05, 2017 at 09:55:34PM +0200, Micha?? K??pie?? wrote:
> > > > On Wed, Apr 05, 2017 at 08:48:59AM +0200, Micha?? K??pie?? wrote:
> > > > > This series introduces further changes to the way LCD backlight is
> > > > > handled by fujitsu-laptop.  These changes include fixing a bug in code
> > > > > responsible for generating brightness-related input events, cleaning 
> > > > > up
> > > > > handling of module parameters, reducing code duplication, removing
> > > > > superfluous debug messages and other fixes.
> > > > > 
> > > > > This series was tested on a Lifebook S7020 and a Lifebook E744.
> > > > > 
> > > > > This series is based on the testing branch as it requires earlier 
> > > > > patch
> > > > > series I submitted in order to apply cleanly.
> > > > > 
> > > > > Changes from v1:
> > > > > 
> > > > >   - Update debug message logged by set_lcd_level() to reflect code 
> > > > > flow
> > > > > changes introduced by patch 04/11.
> > > > 
> > > > Queued to testing, thanks! 
> > > 
> > > Jonathan, I would still love to hear your opinion on this series.  Take
> > > your time, though, I do not see any reason to rush things.  I will only
> > > send the next series once you ack this one.
> > 
> > Sure, no problem.  As mentioned earlier, this week has been busy.  I am
> > hoping I might find the time to complete my review this evening.  If not, it
> > will be some time over the weekend.
> 
> Eeek, I jumped the gun on that. I've moved this from testing to fujitsu, and
> it'll wait there until Jonathan gets a chance to review. Apologies Jonathan.

That's fine, these things happen.

I've completed a review of this series.  It represents another worthwhile
evolutionary clean up of the fujitsu-laptop driver, this time concentrating
on the backlight functions.  Importantly, it makes adjustments for changes
in related kernel infrastructure as it has evolved over the last few
years (the handling of module parameters, method return values and so on).

I am happy for this series to be merged.  Thanks Michael for your continued
clean up efforts.

Reviewed-by: Jonathan Woithe 

Regards
  jonathan

Re: [PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Darren Hart]

On Thu, Apr 06, 2017 at 10:15:14AM +0930, Jonathan Woithe wrote:
> Hi Michael
> 
> On Wed, Apr 05, 2017 at 09:55:34PM +0200, Micha?? K??pie?? wrote:
> > > On Wed, Apr 05, 2017 at 08:48:59AM +0200, Micha?? K??pie?? wrote:
> > > > This series introduces further changes to the way LCD backlight is
> > > > handled by fujitsu-laptop.  These changes include fixing a bug in code
> > > > responsible for generating brightness-related input events, cleaning up
> > > > handling of module parameters, reducing code duplication, removing
> > > > superfluous debug messages and other fixes.
> > > > 
> > > > This series was tested on a Lifebook S7020 and a Lifebook E744.
> > > > 
> > > > This series is based on the testing branch as it requires earlier patch
> > > > series I submitted in order to apply cleanly.
> > > > 
> > > > Changes from v1:
> > > > 
> > > >   - Update debug message logged by set_lcd_level() to reflect code flow
> > > > changes introduced by patch 04/11.
> > > 
> > > Queued to testing, thanks! 
> > 
> > Jonathan, I would still love to hear your opinion on this series.  Take
> > your time, though, I do not see any reason to rush things.  I will only
> > send the next series once you ack this one.
> 
> Sure, no problem.  As mentioned earlier, this week has been busy.  I am
> hoping I might find the time to complete my review this evening.  If not, it
> will be some time over the weekend.

Eeek, I jumped the gun on that. I've moved this from testing to fujitsu, and
it'll wait there until Jonathan gets a chance to review. Apologies Jonathan.

-- 
Darren Hart
VMware Open Source Technology Center

Re: [PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Darren Hart]

On Thu, Apr 06, 2017 at 10:15:14AM +0930, Jonathan Woithe wrote:
> Hi Michael
> 
> On Wed, Apr 05, 2017 at 09:55:34PM +0200, Micha?? K??pie?? wrote:
> > > On Wed, Apr 05, 2017 at 08:48:59AM +0200, Micha?? K??pie?? wrote:
> > > > This series introduces further changes to the way LCD backlight is
> > > > handled by fujitsu-laptop.  These changes include fixing a bug in code
> > > > responsible for generating brightness-related input events, cleaning up
> > > > handling of module parameters, reducing code duplication, removing
> > > > superfluous debug messages and other fixes.
> > > > 
> > > > This series was tested on a Lifebook S7020 and a Lifebook E744.
> > > > 
> > > > This series is based on the testing branch as it requires earlier patch
> > > > series I submitted in order to apply cleanly.
> > > > 
> > > > Changes from v1:
> > > > 
> > > >   - Update debug message logged by set_lcd_level() to reflect code flow
> > > > changes introduced by patch 04/11.
> > > 
> > > Queued to testing, thanks! 
> > 
> > Jonathan, I would still love to hear your opinion on this series.  Take
> > your time, though, I do not see any reason to rush things.  I will only
> > send the next series once you ack this one.
> 
> Sure, no problem.  As mentioned earlier, this week has been busy.  I am
> hoping I might find the time to complete my review this evening.  If not, it
> will be some time over the weekend.

Eeek, I jumped the gun on that. I've moved this from testing to fujitsu, and
it'll wait there until Jonathan gets a chance to review. Apologies Jonathan.

-- 
Darren Hart
VMware Open Source Technology Center

Re: [PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Jonathan Woithe]

Hi Michael

On Wed, Apr 05, 2017 at 09:55:34PM +0200, Micha?? K??pie?? wrote:
> > On Wed, Apr 05, 2017 at 08:48:59AM +0200, Micha?? K??pie?? wrote:
> > > This series introduces further changes to the way LCD backlight is
> > > handled by fujitsu-laptop.  These changes include fixing a bug in code
> > > responsible for generating brightness-related input events, cleaning up
> > > handling of module parameters, reducing code duplication, removing
> > > superfluous debug messages and other fixes.
> > > 
> > > This series was tested on a Lifebook S7020 and a Lifebook E744.
> > > 
> > > This series is based on the testing branch as it requires earlier patch
> > > series I submitted in order to apply cleanly.
> > > 
> > > Changes from v1:
> > > 
> > >   - Update debug message logged by set_lcd_level() to reflect code flow
> > > changes introduced by patch 04/11.
> > 
> > Queued to testing, thanks! 
> 
> Jonathan, I would still love to hear your opinion on this series.  Take
> your time, though, I do not see any reason to rush things.  I will only
> send the next series once you ack this one.

Sure, no problem.  As mentioned earlier, this week has been busy.  I am
hoping I might find the time to complete my review this evening.  If not, it
will be some time over the weekend.

Regards
  jonathan

Re: [PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Jonathan Woithe]

Hi Michael

On Wed, Apr 05, 2017 at 09:55:34PM +0200, Micha?? K??pie?? wrote:
> > On Wed, Apr 05, 2017 at 08:48:59AM +0200, Micha?? K??pie?? wrote:
> > > This series introduces further changes to the way LCD backlight is
> > > handled by fujitsu-laptop.  These changes include fixing a bug in code
> > > responsible for generating brightness-related input events, cleaning up
> > > handling of module parameters, reducing code duplication, removing
> > > superfluous debug messages and other fixes.
> > > 
> > > This series was tested on a Lifebook S7020 and a Lifebook E744.
> > > 
> > > This series is based on the testing branch as it requires earlier patch
> > > series I submitted in order to apply cleanly.
> > > 
> > > Changes from v1:
> > > 
> > >   - Update debug message logged by set_lcd_level() to reflect code flow
> > > changes introduced by patch 04/11.
> > 
> > Queued to testing, thanks! 
> 
> Jonathan, I would still love to hear your opinion on this series.  Take
> your time, though, I do not see any reason to rush things.  I will only
> send the next series once you ack this one.

Sure, no problem.  As mentioned earlier, this week has been busy.  I am
hoping I might find the time to complete my review this evening.  If not, it
will be some time over the weekend.

Regards
  jonathan

Re: [PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Michał Kępień]

> On Wed, Apr 05, 2017 at 08:48:59AM +0200, Michał Kępień wrote:
> > This series introduces further changes to the way LCD backlight is
> > handled by fujitsu-laptop.  These changes include fixing a bug in code
> > responsible for generating brightness-related input events, cleaning up
> > handling of module parameters, reducing code duplication, removing
> > superfluous debug messages and other fixes.
> > 
> > This series was tested on a Lifebook S7020 and a Lifebook E744.
> > 
> > This series is based on the testing branch as it requires earlier patch
> > series I submitted in order to apply cleanly.
> > 
> > Changes from v1:
> > 
> >   - Update debug message logged by set_lcd_level() to reflect code flow
> > changes introduced by patch 04/11.
> 
> Queued to testing, thanks! 

Jonathan, I would still love to hear your opinion on this series.  Take
your time, though, I do not see any reason to rush things.  I will only
send the next series once you ack this one.

-- 
Best regards,
Michał Kępień

Re: [PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Michał Kępień]

> On Wed, Apr 05, 2017 at 08:48:59AM +0200, Michał Kępień wrote:
> > This series introduces further changes to the way LCD backlight is
> > handled by fujitsu-laptop.  These changes include fixing a bug in code
> > responsible for generating brightness-related input events, cleaning up
> > handling of module parameters, reducing code duplication, removing
> > superfluous debug messages and other fixes.
> > 
> > This series was tested on a Lifebook S7020 and a Lifebook E744.
> > 
> > This series is based on the testing branch as it requires earlier patch
> > series I submitted in order to apply cleanly.
> > 
> > Changes from v1:
> > 
> >   - Update debug message logged by set_lcd_level() to reflect code flow
> > changes introduced by patch 04/11.
> 
> Queued to testing, thanks! 

Jonathan, I would still love to hear your opinion on this series.  Take
your time, though, I do not see any reason to rush things.  I will only
send the next series once you ack this one.

-- 
Best regards,
Michał Kępień

Re: [PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Darren Hart]

On Wed, Apr 05, 2017 at 08:48:59AM +0200, Michał Kępień wrote:
> This series introduces further changes to the way LCD backlight is
> handled by fujitsu-laptop.  These changes include fixing a bug in code
> responsible for generating brightness-related input events, cleaning up
> handling of module parameters, reducing code duplication, removing
> superfluous debug messages and other fixes.
> 
> This series was tested on a Lifebook S7020 and a Lifebook E744.
> 
> This series is based on the testing branch as it requires earlier patch
> series I submitted in order to apply cleanly.
> 
> Changes from v1:
> 
>   - Update debug message logged by set_lcd_level() to reflect code flow
> changes introduced by patch 04/11.

Queued to testing, thanks! 

> 
>  drivers/platform/x86/fujitsu-laptop.c | 157 
> ++
>  1 file changed, 46 insertions(+), 111 deletions(-)
> 
> -- 
> 2.12.2
> 
> 

-- 
Darren Hart
VMware Open Source Technology Center

Re: [PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Darren Hart]

On Wed, Apr 05, 2017 at 08:48:59AM +0200, Michał Kępień wrote:
> This series introduces further changes to the way LCD backlight is
> handled by fujitsu-laptop.  These changes include fixing a bug in code
> responsible for generating brightness-related input events, cleaning up
> handling of module parameters, reducing code duplication, removing
> superfluous debug messages and other fixes.
> 
> This series was tested on a Lifebook S7020 and a Lifebook E744.
> 
> This series is based on the testing branch as it requires earlier patch
> series I submitted in order to apply cleanly.
> 
> Changes from v1:
> 
>   - Update debug message logged by set_lcd_level() to reflect code flow
> changes introduced by patch 04/11.

Queued to testing, thanks! 

> 
>  drivers/platform/x86/fujitsu-laptop.c | 157 
> ++
>  1 file changed, 46 insertions(+), 111 deletions(-)
> 
> -- 
> 2.12.2
> 
> 

-- 
Darren Hart
VMware Open Source Technology Center

[PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Michał Kępień]

This series introduces further changes to the way LCD backlight is
handled by fujitsu-laptop.  These changes include fixing a bug in code
responsible for generating brightness-related input events, cleaning up
handling of module parameters, reducing code duplication, removing
superfluous debug messages and other fixes.

This series was tested on a Lifebook S7020 and a Lifebook E744.

This series is based on the testing branch as it requires earlier patch
series I submitted in order to apply cleanly.

Changes from v1:

  - Update debug message logged by set_lcd_level() to reflect code flow
changes introduced by patch 04/11.

 drivers/platform/x86/fujitsu-laptop.c | 157 ++
 1 file changed, 46 insertions(+), 111 deletions(-)

-- 
2.12.2

[PATCH v2 00/11] fujitsu-laptop: backlight cleanup

[Michał Kępień]

This series introduces further changes to the way LCD backlight is
handled by fujitsu-laptop.  These changes include fixing a bug in code
responsible for generating brightness-related input events, cleaning up
handling of module parameters, reducing code duplication, removing
superfluous debug messages and other fixes.

This series was tested on a Lifebook S7020 and a Lifebook E744.

This series is based on the testing branch as it requires earlier patch
series I submitted in order to apply cleanly.

Changes from v1:

  - Update debug message logged by set_lcd_level() to reflect code flow
changes introduced by patch 04/11.

 drivers/platform/x86/fujitsu-laptop.c | 157 ++
 1 file changed, 46 insertions(+), 111 deletions(-)

-- 
2.12.2