lkml.org 
[lkml]   [2017]   [Nov]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v2 0/4] Topstar ACPI LED Workaround
On Sun, Oct 29, 2017 at 1:49 AM, Guillaume Douézan-Grard
<gdouezangrard@gmail.com> wrote:
> Hi Andy,
>
> Thank you for your review. Changes since the previous submission are
> highlighted below.

Thanks for an update.

Unfortunately patch series needs more work (and we have time, since
it's -rc7 already, to improve).

I'm going to comment per patch slightly later.

>
> On Topstar U931 Notebooks, an issue prevents the WLAN toggle key from
> being properly managed by the Embedded Controller and successfully
> disconnect the adapter. A specific ACPI method allows to toggle the WLAN
> LED state regardless.
>
> These are barebone laptops, sold under quite a lot of brands and
> configurations, with different firmwares and so on. I can only say for sure
> that this issue is present for all the models sold under a specific brand,
> that's why I'm reluctant to enable this by default with a DMI check.
>
> Thus, the new `led_workaround` option registers this LED with the
> corresponding subsystem, making possible to use a software-based trigger
> (rfkill for instance to synchronize the LED with the softkill state).
>
> Thank you for your time,
>
> --
> Guillaume Douézan-Grard
>
>
> Changes since v1:
>
> * leave devices names unchanged for ABI compatibility,
> * fix label names,
> * separate module authors definition,
> * add commit description to Patch 3.
>
>
> Guillaume Douézan-Grard (4):
> platform/x86: topstar-laptop: non-functional changes
> platform/x86: topstar-laptop: change to generic module
> platform/x86: topstar-laptop: add platform device
> platform/x86: topstar-laptop: add optional WLAN LED workaround
>
> drivers/platform/x86/Kconfig | 2 +
> drivers/platform/x86/topstar-laptop.c | 334 ++++++++++++++++++++++++++--------
> 2 files changed, 265 insertions(+), 71 deletions(-)
>
> --
> 2.14.3
>



--
With Best Regards,
Andy Shevchenko

\
 
 \ /
  Last update: 2017-11-03 12:57    [W:0.094 / U:0.572 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site