[PATCH v5 2/7] platform/x86: topstar-laptop: use consistent naming scheme

* use module prefix naming scheme for functions and programming constructs, * consistent label names Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 82 +-- 1 file changed, 41 insertions(+), 41 deletions(-) diff --git a

[PATCH v5 7/7] platform/x86: topstar-laptop: replace licence text with SPDX tag

Replace verbatim GPLv2 license copy with SPDX identifier. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/platform/x86/topstar-laptop.c b/drivers/platform/x86/topstar-laptop.c index

[PATCH v5 4/7] platform/x86: topstar-laptop: add platform device

* add a platform device to support further addition of a led subsystem, * register the existing input device to this platform device. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 61 +-- 1 file changed, 58 insertions(+), 3

[PATCH v5 3/7] platform/x86: topstar-laptop: split ACPI events and input handling

* get the `acpi_device` from the `topstar_laptop` struct, * split input registering and `sparse_keymap` events from ACPI events handling, * use notify, init and exit functions for ACPI and input handling Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 109

[PATCH v5 5/7] platform/x86: topstar-laptop: add Topstar U931/RVP7 WLAN LED workaround

DMI check. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/Kconfig | 2 + drivers/platform/x86/topstar-laptop.c | 108 ++ 2 files changed, 110 insertions(+) diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig index

[PATCH v5 6/7] platform/x86: topstar-laptop: update copyright and fix some comments

Add myself as an author of the driver. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/topstar-laptop.c b/drivers/platform/x86/topstar-laptop.c index 30a6c5cbc58f

[PATCH v5 1/7] platform/x86: topstar-laptop: revert "convert to module_acpi_driver()"

Revert commit 15165594da65 ("topstar-laptop: convert to module_acpi_driver()") to later add a platform device. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git

[PATCH v5 0/7] Topstar U931/RVP7 ACPI LED Workaround

toggle the WLAN LED state regardless. Based on a DMI check, the LED is registered within the corresponding subsystem, making possible to use a software-based trigger (rfkill for instance, to synchronize the LED with the network interface state). Thank you for your time, -- Guillaume Douézan-Grard

Re: [PATCH v4 5/7] platform/x86: topstar-laptop: add Topstar U931/RVP7 WLAN LED workaround

On Wed, Feb 21, 2018 at 06:25:10PM +0200, Andy Shevchenko wrote: > On Wed, Feb 21, 2018 at 6:10 PM, Guillaume Douézan-Grard > wrote: > > Topstar U931 laptops provide a LED synced with the WLAN adapter > > hard-blocking state. Unfortunately, some models seem to be defective, >

[PATCH v4 6/7] platform/x86: topstar-laptop: update copyright and fix some comments

Add myself as an author of the driver. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/topstar-laptop.c b/drivers/platform/x86/topstar-laptop.c index 30a6c5cbc58f

[PATCH v4 7/7] platform/x86: topstar-laptop: replace licence text with SPDX tag

Replace verbatim GPLv2 license copy with SPDX identifier. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/platform/x86/topstar-laptop.c b/drivers/platform/x86/topstar-laptop.c index

[PATCH v4 5/7] platform/x86: topstar-laptop: add Topstar U931/RVP7 WLAN LED workaround

DMI check. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/Kconfig | 2 + drivers/platform/x86/topstar-laptop.c | 110 +- 2 files changed, 111 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86

[PATCH v4 4/7] platform/x86: topstar-laptop: add platform device

* add a platform device to support further addition of a led subsystem, * register the existing input device to this platform device. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 59 +-- 1 file changed, 57 insertions(+), 2

[PATCH v4 3/7] platform/x86: topstar-laptop: split ACPI events and input handling

* get the `acpi_device` from the `topstar_laptop` struct, * split input registering and `sparse_keymap` events from ACPI events handling, * use notify, init and exit functions for ACPI and input handling Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 109

[PATCH v4 2/7] platform/x86: topstar-laptop: use consistent naming scheme

* use module prefix naming scheme for functions and programming constructs, * consistent label names Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 82 +-- 1 file changed, 41 insertions(+), 41 deletions(-) diff --git a

[PATCH v4 0/7] Topstar U931/RVP7 ACPI LED Workaround

toggle the WLAN LED state regardless. Based on a DMI check, the LED is registered within the corresponding subsystem, making possible to use a software-based trigger (rfkill for instance, to synchronize the LED with the network interface state). Thank you for your time, -- Guillaume Douézan-Grard

[PATCH v4 1/7] platform/x86: topstar-laptop: revert "convert to module_acpi_driver()"

Revert commit 15165594da65 ("topstar-laptop: convert to module_acpi_driver()") to later add a platform device. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git

[PATCH v3 6/6] platform/x86: topstar-laptop: update copyright and fix some comments

Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/platform/x86/topstar-laptop.c b/drivers/platform/x86/topstar-laptop.c index bf9761092c7e..294729c98267 100644 --- a/drivers/platform

[PATCH v3 4/6] platform/x86: topstar-laptop: add platform device

* add a platform device to support further addition of a led subsystem, * register the existing input device to this platform device Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 59 +-- 1 file changed, 57 insertions(+), 2

[PATCH v3 5/6] platform/x86: topstar-laptop: add Topstar U931/RVP7 WLAN LED workaround

DMI check. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/Kconfig | 2 + drivers/platform/x86/topstar-laptop.c | 108 ++ 2 files changed, 110 insertions(+) diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig index

[PATCH v3 3/6] platform/x86: topstar-laptop: split ACPI events and input handling

* get the `acpi_device` from the `topstar_laptop` struct, * split input registering and `sparse_keymap` events from ACPI events handling, * use notify, init and exit functions for ACPI and input handling Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 109

[PATCH v3 2/6] platform/x86: topstar-laptop: use consistent naming scheme

* use module prefix naming scheme for functions and programming constructs, * consistent label names Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 82 +-- 1 file changed, 41 insertions(+), 41 deletions(-) diff --git a

[PATCH v3 1/6] revert "topstar-laptop: convert to module_acpi_driver()"

Revert commit 15165594da65 ("topstar-laptop: convert to module_acpi_driver()") to later add a platform device. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git

[PATCH v3 0/6] Topstar U931/RVP7 ACPI LED Workaround

toggle the WLAN LED state regardless. Based on a DMI check, the LED is registered within the corresponding subsystem, making possible to use a software-based trigger (rfkill for instance, to synchronize the LED with the network interface state). Thank you for your time, -- Guillaume Douézan-Grard

Re: [PATCH v2 4/4] platform/x86: topstar-laptop: add optional WLAN LED workaround

On Sun, Nov 05, 2017 at 02:34:43PM -0800, Darren Hart wrote: > On Sun, Nov 05, 2017 at 03:28:09PM +0200, Andy Shevchenko wrote: > > On Fri, Nov 3, 2017 at 5:07 PM, Guillaume Douézan-Grard > > wrote: > > > On Fri, Nov 03, 2017 at 02:50:52PM +0200, Andy Shevchenko wrote: >

Re: [PATCH v2 4/4] platform/x86: topstar-laptop: add optional WLAN LED workaround

On Fri, Nov 03, 2017 at 02:50:52PM +0200, Andy Shevchenko wrote: > On Sun, Oct 29, 2017 at 1:53 AM, Guillaume Douézan-Grard > wrote: > > Topstar U931 laptops provide an LED synced with the WLAN adapter > > hard-blocking state. Unfortunately, some models seem to be defective, >

[PATCH v2 4/4] platform/x86: topstar-laptop: add optional WLAN LED workaround

`led_workaround` module parameter. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/Kconfig | 2 + drivers/platform/x86/topstar-laptop.c | 83 +++ 2 files changed, 85 insertions(+) diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86

[PATCH v2 3/4] platform/x86: topstar-laptop: add platform device

* add platform device to support further addition of a led subsystem, * add existing input device to the new platform device. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 59 +-- 1 file changed, 57 insertions(+), 2 deletions

[PATCH v2 2/4] platform/x86: topstar-laptop: change to generic module

Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/topstar-laptop.c b/drivers/platform/x86/topstar-laptop.c index 106537fdc4e6..714d9d56c79f 100644 --- a

[PATCH v2 1/4] platform/x86: topstar-laptop: non-functional changes

authors update. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 178 -- 1 file changed, 107 insertions(+), 71 deletions(-) diff --git a/drivers/platform/x86/topstar-laptop.c b/drivers/platform/x86/topstar-laptop.c index

[PATCH v2 0/4] Topstar ACPI LED Workaround

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 Cha

[PATCH 4/4] platform/x86: topstar-laptop: add optional WLAN LED workaround

`led_workaround` module parameter. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/Kconfig | 2 + drivers/platform/x86/topstar-laptop.c | 83 +++ 2 files changed, 85 insertions(+) diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86

[PATCH 1/4] platform/x86: topstar-laptop: non-functional changes

authors update. Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 185 -- 1 file changed, 110 insertions(+), 75 deletions(-) diff --git a/drivers/platform/x86/topstar-laptop.c b/drivers/platform/x86/topstar-laptop.c index

[PATCH 3/4] platform/x86: topstar-laptop: add platform device

Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 57 ++- 1 file changed, 56 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/topstar-laptop.c b/drivers/platform/x86/topstar-laptop.c index ce33754c1f29..e2ee1e5cc734

[PATCH 2/4] platform/x86: topstar-laptop: change to generic module

Signed-off-by: Guillaume Douézan-Grard --- drivers/platform/x86/topstar-laptop.c | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/drivers/platform/x86/topstar-laptop.c b/drivers/platform/x86/topstar-laptop.c index d3197302aa91..ce33754c1f29 100644 --- a

[PATCH 0/4] Topstar ACPI LED Workaround

D 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 Guillaume Douézan-Grard (4): platform/x86: topstar-laptop: non-functional changes pl