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

Groups > linux.kernel > #1592378

Re: [PATCH v2 0/4] fujitsu_init() cleanup

Show all headers | View raw

Hi Michael

On Wed, Mar 01, 2017 at 09:10:40AM +0100, Micha?? K??pie?? wrote:
> These patches should make fujitsu_init() a bit more palatable.  No
> changes are made to platform device code yet, for clarity these will be
> posted in a separate series after this one gets applied.

I have a preliminary report.  The backlight functionality remains functional
on an S7020 across all four of the patches in this series and with the
additional 2-patch cleanup series applied.

With regard to patch 2/4 you wrote:
> Jonathan, this *really* needs testing on relevant hardware.  After
> applying this patch, you should be able to turn LCD backlight on and off
> using /sys/class/backlight/fujitsu-laptop/bl_power.  Also, the value
> returned by that attribute upon read should be in sync with actual
> backlight state even right after loading the module (i.e. before writing
> anything to bl_power).  Please let me know if any of the above is not
> true and the module works correctly without this patch applied.

With patch 2/4 applied:

 * It is possible to read bl_power

 * It is possible to write a value to bl_power and read that value back

 * Writing values to bl_power does not appear to affect the LCD panel in 
   any way.  That is, the backlight remains unchanged regardless of the 
   value written.

 * Behaviour is the same both under X and from the terminal.

Backing out patch 2/4 but with all others still in place, resulted in no
change in behaviour.  So while bl_power had no effect with patch 2/4 in
place, it seems that patch 2/4 is *not* the cause of this.

I shall run some more bl_power tests and complete a review of the code later
this weekend.

Regards
  jonathan

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 0/4] fujitsu_init() cleanup Michał Kępień <kernel@kempniu.pl> - 2017-03-01 09:20 +0100
  [PATCH v2 1/4] platform/x86: fujitsu-laptop: register backlight device in a separate function Michał Kępień <kernel@kempniu.pl> - 2017-03-01 09:20 +0100
  [PATCH v2 2/4] platform/x86: fujitsu-laptop: do not use call_fext_func() for LCD blanking Michał Kępień <kernel@kempniu.pl> - 2017-03-01 09:20 +0100
  [PATCH v2 3/4] platform/x86: fujitsu-laptop: only register backlight device if FUJ02B1 is present Michał Kępień <kernel@kempniu.pl> - 2017-03-01 09:20 +0100
  Re: [PATCH v2 0/4] fujitsu_init() cleanup [resend due to vger error] Jonathan Woithe <jwoithe@just42.net> - 2017-03-02 04:50 +0100
  Re: [PATCH v2 0/4] fujitsu_init() cleanup Michał Kępień <kernel@kempniu.pl> - 2017-03-02 09:30 +0100
    Re: [PATCH v2 0/4] fujitsu_init() cleanup Jonathan Woithe <jwoithe@just42.net> - 2017-03-03 13:50 +0100
  Re: [PATCH v2 0/4] fujitsu_init() cleanup Jonathan Woithe <jwoithe@just42.net> - 2017-03-04 03:10 +0100
    Re: [PATCH v2 0/4] fujitsu_init() cleanup Jonathan Woithe <jwoithe@just42.net> - 2017-03-06 01:20 +0100
      Re: [PATCH v2 0/4] fujitsu_init() cleanup Michał Kępień <kernel@kempniu.pl> - 2017-03-06 06:00 +0100
        Re: [PATCH v2 0/4] fujitsu_init() cleanup Jonathan Woithe <jwoithe@just42.net> - 2017-03-06 06:10 +0100
          Re: [PATCH v2 0/4] fujitsu_init() cleanup Jonathan Woithe <jwoithe@just42.net> - 2017-03-06 10:00 +0100
            Re: [PATCH v2 0/4] fujitsu_init() cleanup Michał Kępień <kernel@kempniu.pl> - 2017-03-06 10:40 +0100
              Re: [PATCH v2 0/4] fujitsu_init() cleanup Michał Kępień <kernel@kempniu.pl> - 2017-03-06 19:50 +0100
                Re: [PATCH v2 0/4] fujitsu_init() cleanup Jonathan Woithe <jwoithe@just42.net> - 2017-03-07 05:00 +0100
                Re: [PATCH v2 0/4] fujitsu_init() cleanup Michał Kępień <kernel@kempniu.pl> - 2017-03-07 09:20 +0100

csiph-web