* [PATCH v3 1/2] platform/x86: asus-wmi: Call led hw_changed API on kbd brightness change @ 2018-06-20 14:46 Chris Chiu 2018-06-20 14:46 ` [PATCH v3 2/2] platform/x86: asus-wmi: Add keyboard backlight toggle support Chris Chiu 2018-06-27 10:27 ` [PATCH v3 1/2] platform/x86: asus-wmi: Call led hw_changed API on kbd brightness change Chris Chiu 0 siblings, 2 replies; 7+ messages in thread From: Chris Chiu @ 2018-06-20 14:46 UTC (permalink / raw) To: corentin.chary, dvhart, andy.shevchenko Cc: linux-kernel, platform-driver-x86, acpi4asus-user, hdegoede, linux, Chris Chiu Make asus-wmi notify on hotkey kbd brightness changes, listen for brightness events and update the brightness directly in the driver. Create new do_kbd_led_set function for in-driver update, and leave kbd_led_set for original led_classdev call path. Update the brightness by led_classdev_notify_brightness_hw_changed. This will allow userspace to monitor (poll) for brightness changes on the LED without reporting via input keymapping. Signed-off-by: Chris Chiu <chiu@endlessm.com> --- drivers/platform/x86/asus-wmi.c | 26 ++++++++++++++++++++++---- 1 file changed, 22 insertions(+), 4 deletions(-) diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-wmi.c index 1f6e68f0b646..f2b9037f6ded 100644 --- a/drivers/platform/x86/asus-wmi.c +++ b/drivers/platform/x86/asus-wmi.c @@ -460,6 +460,7 @@ static void kbd_led_update(struct work_struct *work) ctrl_param = 0x80 | (asus->kbd_led_wk & 0x7F); asus_wmi_set_devstate(ASUS_WMI_DEVID_KBD_BACKLIGHT, ctrl_param, NULL); + led_classdev_notify_brightness_hw_changed(&asus->kbd_led, asus->kbd_led_wk); } static int kbd_led_read(struct asus_wmi *asus, int *level, int *env) @@ -490,15 +491,16 @@ static int kbd_led_read(struct asus_wmi *asus, int *level, int *env) return retval; } -static void kbd_led_set(struct led_classdev *led_cdev, - enum led_brightness value) +static void do_kbd_led_set(struct led_classdev *led_cdev, int value) { struct asus_wmi *asus; + int max_level; asus = container_of(led_cdev, struct asus_wmi, kbd_led); + max_level = asus->kbd_led.max_brightness; - if (value > asus->kbd_led.max_brightness) - value = asus->kbd_led.max_brightness; + if (value > max_level) + value = max_level; else if (value < 0) value = 0; @@ -506,6 +508,12 @@ static void kbd_led_set(struct led_classdev *led_cdev, queue_work(asus->led_workqueue, &asus->kbd_led_work); } +static void kbd_led_set(struct led_classdev *led_cdev, + enum led_brightness value) +{ + do_kbd_led_set(led_cdev, value); +} + static enum led_brightness kbd_led_get(struct led_classdev *led_cdev) { struct asus_wmi *asus; @@ -656,6 +664,7 @@ static int asus_wmi_led_init(struct asus_wmi *asus) asus->kbd_led_wk = led_val; asus->kbd_led.name = "asus::kbd_backlight"; + asus->kbd_led.flags = LED_BRIGHT_HW_CHANGED; asus->kbd_led.brightness_set = kbd_led_set; asus->kbd_led.brightness_get = kbd_led_get; asus->kbd_led.max_brightness = 3; @@ -1745,6 +1754,15 @@ static void asus_wmi_notify(u32 value, void *context) } } + if (code == NOTIFY_KBD_BRTUP) { + do_kbd_led_set(&asus->kbd_led, asus->kbd_led_wk + 1); + goto exit; + } + if (code == NOTIFY_KBD_BRTDWN) { + do_kbd_led_set(&asus->kbd_led, asus->kbd_led_wk - 1); + goto exit; + } + if (is_display_toggle(code) && asus->driver->quirks->no_display_toggle) goto exit; -- 2.11.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v3 2/2] platform/x86: asus-wmi: Add keyboard backlight toggle support 2018-06-20 14:46 [PATCH v3 1/2] platform/x86: asus-wmi: Call led hw_changed API on kbd brightness change Chris Chiu @ 2018-06-20 14:46 ` Chris Chiu 2018-06-27 10:27 ` [PATCH v3 1/2] platform/x86: asus-wmi: Call led hw_changed API on kbd brightness change Chris Chiu 1 sibling, 0 replies; 7+ messages in thread From: Chris Chiu @ 2018-06-20 14:46 UTC (permalink / raw) To: corentin.chary, dvhart, andy.shevchenko Cc: linux-kernel, platform-driver-x86, acpi4asus-user, hdegoede, linux, Chris Chiu Some ASUS laptops like UX550GE has hotkey (Fn+F7) for keyboard backlight toggle which would emit the scan code 0xc7 each keypress. On the UX550GE, the max keyboard brightness level is 3 so the toggle would not be simply on/off the led but need to be cyclic. Per ASUS spec, it should increment the brightness for each keypress, then toggle(off) the LED when it already reached the max level. Signed-off-by: Chris Chiu <chiu@endlessm.com> --- drivers/platform/x86/asus-wmi.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-wmi.c index f2b9037f6ded..51dc019fa91d 100644 --- a/drivers/platform/x86/asus-wmi.c +++ b/drivers/platform/x86/asus-wmi.c @@ -67,6 +67,7 @@ MODULE_LICENSE("GPL"); #define NOTIFY_BRNDOWN_MAX 0x2e #define NOTIFY_KBD_BRTUP 0xc4 #define NOTIFY_KBD_BRTDWN 0xc5 +#define NOTIFY_KBD_BRTTOGGLE 0xc7 /* WMI Methods */ #define ASUS_WMI_METHODID_SPEC 0x43455053 /* BIOS SPECification */ @@ -1762,6 +1763,13 @@ static void asus_wmi_notify(u32 value, void *context) do_kbd_led_set(&asus->kbd_led, asus->kbd_led_wk - 1); goto exit; } + if (code == NOTIFY_KBD_BRTTOGGLE) { + if (asus->kbd_led_wk == asus->kbd_led.max_brightness) + do_kbd_led_set(&asus->kbd_led, 0); + else + do_kbd_led_set(&asus->kbd_led, asus->kbd_led_wk + 1); + goto exit; + } if (is_display_toggle(code) && asus->driver->quirks->no_display_toggle) -- 2.11.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v3 1/2] platform/x86: asus-wmi: Call led hw_changed API on kbd brightness change 2018-06-20 14:46 [PATCH v3 1/2] platform/x86: asus-wmi: Call led hw_changed API on kbd brightness change Chris Chiu 2018-06-20 14:46 ` [PATCH v3 2/2] platform/x86: asus-wmi: Add keyboard backlight toggle support Chris Chiu @ 2018-06-27 10:27 ` Chris Chiu 2018-07-02 12:10 ` Andy Shevchenko 1 sibling, 1 reply; 7+ messages in thread From: Chris Chiu @ 2018-06-27 10:27 UTC (permalink / raw) To: Corentin Chary, Darren Hart, Andy Shevchenko Cc: Linux Kernel, Platform Driver, acpi4asus-user, Hans de Goede, Linux Upstreaming Team, Chris Chiu On Wed, Jun 20, 2018 at 10:46 PM, Chris Chiu <chiu@endlessm.com> wrote: > Make asus-wmi notify on hotkey kbd brightness changes, listen for > brightness events and update the brightness directly in the driver. > Create new do_kbd_led_set function for in-driver update, and leave > kbd_led_set for original led_classdev call path. > > Update the brightness by led_classdev_notify_brightness_hw_changed. > This will allow userspace to monitor (poll) for brightness changes > on the LED without reporting via input keymapping. > > Signed-off-by: Chris Chiu <chiu@endlessm.com> > --- > drivers/platform/x86/asus-wmi.c | 26 ++++++++++++++++++++++---- > 1 file changed, 22 insertions(+), 4 deletions(-) > > diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-wmi.c > index 1f6e68f0b646..f2b9037f6ded 100644 > --- a/drivers/platform/x86/asus-wmi.c > +++ b/drivers/platform/x86/asus-wmi.c > @@ -460,6 +460,7 @@ static void kbd_led_update(struct work_struct *work) > ctrl_param = 0x80 | (asus->kbd_led_wk & 0x7F); > > asus_wmi_set_devstate(ASUS_WMI_DEVID_KBD_BACKLIGHT, ctrl_param, NULL); > + led_classdev_notify_brightness_hw_changed(&asus->kbd_led, asus->kbd_led_wk); > } > > static int kbd_led_read(struct asus_wmi *asus, int *level, int *env) > @@ -490,15 +491,16 @@ static int kbd_led_read(struct asus_wmi *asus, int *level, int *env) > return retval; > } > > -static void kbd_led_set(struct led_classdev *led_cdev, > - enum led_brightness value) > +static void do_kbd_led_set(struct led_classdev *led_cdev, int value) > { > struct asus_wmi *asus; > + int max_level; > > asus = container_of(led_cdev, struct asus_wmi, kbd_led); > + max_level = asus->kbd_led.max_brightness; > > - if (value > asus->kbd_led.max_brightness) > - value = asus->kbd_led.max_brightness; > + if (value > max_level) > + value = max_level; > else if (value < 0) > value = 0; > > @@ -506,6 +508,12 @@ static void kbd_led_set(struct led_classdev *led_cdev, > queue_work(asus->led_workqueue, &asus->kbd_led_work); > } > > +static void kbd_led_set(struct led_classdev *led_cdev, > + enum led_brightness value) > +{ > + do_kbd_led_set(led_cdev, value); > +} > + > static enum led_brightness kbd_led_get(struct led_classdev *led_cdev) > { > struct asus_wmi *asus; > @@ -656,6 +664,7 @@ static int asus_wmi_led_init(struct asus_wmi *asus) > > asus->kbd_led_wk = led_val; > asus->kbd_led.name = "asus::kbd_backlight"; > + asus->kbd_led.flags = LED_BRIGHT_HW_CHANGED; > asus->kbd_led.brightness_set = kbd_led_set; > asus->kbd_led.brightness_get = kbd_led_get; > asus->kbd_led.max_brightness = 3; > @@ -1745,6 +1754,15 @@ static void asus_wmi_notify(u32 value, void *context) > } > } > > + if (code == NOTIFY_KBD_BRTUP) { > + do_kbd_led_set(&asus->kbd_led, asus->kbd_led_wk + 1); > + goto exit; > + } > + if (code == NOTIFY_KBD_BRTDWN) { > + do_kbd_led_set(&asus->kbd_led, asus->kbd_led_wk - 1); > + goto exit; > + } > + > if (is_display_toggle(code) && > asus->driver->quirks->no_display_toggle) > goto exit; > -- > 2.11.0 > Gentle ping. Any comment for the v3 patch? Chris ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v3 1/2] platform/x86: asus-wmi: Call led hw_changed API on kbd brightness change 2018-06-27 10:27 ` [PATCH v3 1/2] platform/x86: asus-wmi: Call led hw_changed API on kbd brightness change Chris Chiu @ 2018-07-02 12:10 ` Andy Shevchenko 2018-08-15 9:34 ` Chris Chiu 0 siblings, 1 reply; 7+ messages in thread From: Andy Shevchenko @ 2018-07-02 12:10 UTC (permalink / raw) To: Chris Chiu Cc: Corentin Chary, Darren Hart, Linux Kernel, Platform Driver, acpi4asus-user, Hans de Goede, Linux Upstreaming Team On Wed, Jun 27, 2018 at 1:27 PM, Chris Chiu <chiu@endlessm.com> wrote: > On Wed, Jun 20, 2018 at 10:46 PM, Chris Chiu <chiu@endlessm.com> wrote: >> Make asus-wmi notify on hotkey kbd brightness changes, listen for >> brightness events and update the brightness directly in the driver. >> Create new do_kbd_led_set function for in-driver update, and leave >> kbd_led_set for original led_classdev call path. >> >> Update the brightness by led_classdev_notify_brightness_hw_changed. >> This will allow userspace to monitor (poll) for brightness changes >> on the LED without reporting via input keymapping. LGTM, I'm about to push it to my review and testing queue, so, we will have 1 day or so time to gather objections if any. Thanks! >> >> Signed-off-by: Chris Chiu <chiu@endlessm.com> >> --- >> drivers/platform/x86/asus-wmi.c | 26 ++++++++++++++++++++++---- >> 1 file changed, 22 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-wmi.c >> index 1f6e68f0b646..f2b9037f6ded 100644 >> --- a/drivers/platform/x86/asus-wmi.c >> +++ b/drivers/platform/x86/asus-wmi.c >> @@ -460,6 +460,7 @@ static void kbd_led_update(struct work_struct *work) >> ctrl_param = 0x80 | (asus->kbd_led_wk & 0x7F); >> >> asus_wmi_set_devstate(ASUS_WMI_DEVID_KBD_BACKLIGHT, ctrl_param, NULL); >> + led_classdev_notify_brightness_hw_changed(&asus->kbd_led, asus->kbd_led_wk); >> } >> >> static int kbd_led_read(struct asus_wmi *asus, int *level, int *env) >> @@ -490,15 +491,16 @@ static int kbd_led_read(struct asus_wmi *asus, int *level, int *env) >> return retval; >> } >> >> -static void kbd_led_set(struct led_classdev *led_cdev, >> - enum led_brightness value) >> +static void do_kbd_led_set(struct led_classdev *led_cdev, int value) >> { >> struct asus_wmi *asus; >> + int max_level; >> >> asus = container_of(led_cdev, struct asus_wmi, kbd_led); >> + max_level = asus->kbd_led.max_brightness; >> >> - if (value > asus->kbd_led.max_brightness) >> - value = asus->kbd_led.max_brightness; >> + if (value > max_level) >> + value = max_level; >> else if (value < 0) >> value = 0; >> >> @@ -506,6 +508,12 @@ static void kbd_led_set(struct led_classdev *led_cdev, >> queue_work(asus->led_workqueue, &asus->kbd_led_work); >> } >> >> +static void kbd_led_set(struct led_classdev *led_cdev, >> + enum led_brightness value) >> +{ >> + do_kbd_led_set(led_cdev, value); >> +} >> + >> static enum led_brightness kbd_led_get(struct led_classdev *led_cdev) >> { >> struct asus_wmi *asus; >> @@ -656,6 +664,7 @@ static int asus_wmi_led_init(struct asus_wmi *asus) >> >> asus->kbd_led_wk = led_val; >> asus->kbd_led.name = "asus::kbd_backlight"; >> + asus->kbd_led.flags = LED_BRIGHT_HW_CHANGED; >> asus->kbd_led.brightness_set = kbd_led_set; >> asus->kbd_led.brightness_get = kbd_led_get; >> asus->kbd_led.max_brightness = 3; >> @@ -1745,6 +1754,15 @@ static void asus_wmi_notify(u32 value, void *context) >> } >> } >> >> + if (code == NOTIFY_KBD_BRTUP) { >> + do_kbd_led_set(&asus->kbd_led, asus->kbd_led_wk + 1); >> + goto exit; >> + } >> + if (code == NOTIFY_KBD_BRTDWN) { >> + do_kbd_led_set(&asus->kbd_led, asus->kbd_led_wk - 1); >> + goto exit; >> + } >> + >> if (is_display_toggle(code) && >> asus->driver->quirks->no_display_toggle) >> goto exit; >> -- >> 2.11.0 >> > > Gentle ping. Any comment for the v3 patch? > > Chris -- With Best Regards, Andy Shevchenko ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v3 1/2] platform/x86: asus-wmi: Call led hw_changed API on kbd brightness change 2018-07-02 12:10 ` Andy Shevchenko @ 2018-08-15 9:34 ` Chris Chiu 2018-08-15 9:45 ` Andy Shevchenko 0 siblings, 1 reply; 7+ messages in thread From: Chris Chiu @ 2018-08-15 9:34 UTC (permalink / raw) To: Andy Shevchenko Cc: Corentin Chary, Darren Hart, Linux Kernel, Platform Driver, acpi4asus-user, Hans de Goede, Linux Upstreaming Team On Mon, Jul 2, 2018 at 8:10 PM, Andy Shevchenko <andy.shevchenko@gmail.com> wrote: > On Wed, Jun 27, 2018 at 1:27 PM, Chris Chiu <chiu@endlessm.com> wrote: >> On Wed, Jun 20, 2018 at 10:46 PM, Chris Chiu <chiu@endlessm.com> wrote: >>> Make asus-wmi notify on hotkey kbd brightness changes, listen for >>> brightness events and update the brightness directly in the driver. >>> Create new do_kbd_led_set function for in-driver update, and leave >>> kbd_led_set for original led_classdev call path. >>> >>> Update the brightness by led_classdev_notify_brightness_hw_changed. >>> This will allow userspace to monitor (poll) for brightness changes >>> on the LED without reporting via input keymapping. > > LGTM, I'm about to push it to my review and testing queue, so, we will > have 1 day or so time to gather objections if any. > Thanks! > Gentle ping. How's the review so far? Maybe need more time for more info? Chris >>> >>> Signed-off-by: Chris Chiu <chiu@endlessm.com> >>> --- >>> drivers/platform/x86/asus-wmi.c | 26 ++++++++++++++++++++++---- >>> 1 file changed, 22 insertions(+), 4 deletions(-) >>> >>> diff --git a/drivers/platform/x86/asus-wmi.c b/drivers/platform/x86/asus-wmi.c >>> index 1f6e68f0b646..f2b9037f6ded 100644 >>> --- a/drivers/platform/x86/asus-wmi.c >>> +++ b/drivers/platform/x86/asus-wmi.c >>> @@ -460,6 +460,7 @@ static void kbd_led_update(struct work_struct *work) >>> ctrl_param = 0x80 | (asus->kbd_led_wk & 0x7F); >>> >>> asus_wmi_set_devstate(ASUS_WMI_DEVID_KBD_BACKLIGHT, ctrl_param, NULL); >>> + led_classdev_notify_brightness_hw_changed(&asus->kbd_led, asus->kbd_led_wk); >>> } >>> >>> static int kbd_led_read(struct asus_wmi *asus, int *level, int *env) >>> @@ -490,15 +491,16 @@ static int kbd_led_read(struct asus_wmi *asus, int *level, int *env) >>> return retval; >>> } >>> >>> -static void kbd_led_set(struct led_classdev *led_cdev, >>> - enum led_brightness value) >>> +static void do_kbd_led_set(struct led_classdev *led_cdev, int value) >>> { >>> struct asus_wmi *asus; >>> + int max_level; >>> >>> asus = container_of(led_cdev, struct asus_wmi, kbd_led); >>> + max_level = asus->kbd_led.max_brightness; >>> >>> - if (value > asus->kbd_led.max_brightness) >>> - value = asus->kbd_led.max_brightness; >>> + if (value > max_level) >>> + value = max_level; >>> else if (value < 0) >>> value = 0; >>> >>> @@ -506,6 +508,12 @@ static void kbd_led_set(struct led_classdev *led_cdev, >>> queue_work(asus->led_workqueue, &asus->kbd_led_work); >>> } >>> >>> +static void kbd_led_set(struct led_classdev *led_cdev, >>> + enum led_brightness value) >>> +{ >>> + do_kbd_led_set(led_cdev, value); >>> +} >>> + >>> static enum led_brightness kbd_led_get(struct led_classdev *led_cdev) >>> { >>> struct asus_wmi *asus; >>> @@ -656,6 +664,7 @@ static int asus_wmi_led_init(struct asus_wmi *asus) >>> >>> asus->kbd_led_wk = led_val; >>> asus->kbd_led.name = "asus::kbd_backlight"; >>> + asus->kbd_led.flags = LED_BRIGHT_HW_CHANGED; >>> asus->kbd_led.brightness_set = kbd_led_set; >>> asus->kbd_led.brightness_get = kbd_led_get; >>> asus->kbd_led.max_brightness = 3; >>> @@ -1745,6 +1754,15 @@ static void asus_wmi_notify(u32 value, void *context) >>> } >>> } >>> >>> + if (code == NOTIFY_KBD_BRTUP) { >>> + do_kbd_led_set(&asus->kbd_led, asus->kbd_led_wk + 1); >>> + goto exit; >>> + } >>> + if (code == NOTIFY_KBD_BRTDWN) { >>> + do_kbd_led_set(&asus->kbd_led, asus->kbd_led_wk - 1); >>> + goto exit; >>> + } >>> + >>> if (is_display_toggle(code) && >>> asus->driver->quirks->no_display_toggle) >>> goto exit; >>> -- >>> 2.11.0 >>> >> >> Gentle ping. Any comment for the v3 patch? >> >> Chris > > > > -- > With Best Regards, > Andy Shevchenko ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v3 1/2] platform/x86: asus-wmi: Call led hw_changed API on kbd brightness change 2018-08-15 9:34 ` Chris Chiu @ 2018-08-15 9:45 ` Andy Shevchenko 2018-08-15 9:49 ` Chris Chiu 0 siblings, 1 reply; 7+ messages in thread From: Andy Shevchenko @ 2018-08-15 9:45 UTC (permalink / raw) To: Chris Chiu Cc: Corentin Chary, Darren Hart, Linux Kernel, Platform Driver, acpi4asus-user, Hans de Goede, Linux Upstreaming Team On Wed, Aug 15, 2018 at 12:34 PM, Chris Chiu <chiu@endlessm.com> wrote: > On Mon, Jul 2, 2018 at 8:10 PM, Andy Shevchenko > <andy.shevchenko@gmail.com> wrote: >> On Wed, Jun 27, 2018 at 1:27 PM, Chris Chiu <chiu@endlessm.com> wrote: >>> On Wed, Jun 20, 2018 at 10:46 PM, Chris Chiu <chiu@endlessm.com> wrote: >>>> Make asus-wmi notify on hotkey kbd brightness changes, listen for >>>> brightness events and update the brightness directly in the driver. >>>> Create new do_kbd_led_set function for in-driver update, and leave >>>> kbd_led_set for original led_classdev call path. >>>> >>>> Update the brightness by led_classdev_notify_brightness_hw_changed. >>>> This will allow userspace to monitor (poll) for brightness changes >>>> on the LED without reporting via input keymapping. >> >> LGTM, I'm about to push it to my review and testing queue, so, we will >> have 1 day or so time to gather objections if any. >> Thanks! >> > > Gentle ping. How's the review so far? Maybe need more time for more info? Sorry, I thought we have no objections and I pushed it to our for-next quite time ago [1]. Should I do something differently? [1]: http://git.infradead.org/linux-platform-drivers-x86.git/commitdiff/dbb3d78f61badeadc7448f9d88a070a76612860f -- With Best Regards, Andy Shevchenko ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v3 1/2] platform/x86: asus-wmi: Call led hw_changed API on kbd brightness change 2018-08-15 9:45 ` Andy Shevchenko @ 2018-08-15 9:49 ` Chris Chiu 0 siblings, 0 replies; 7+ messages in thread From: Chris Chiu @ 2018-08-15 9:49 UTC (permalink / raw) To: Andy Shevchenko Cc: Corentin Chary, Darren Hart, Linux Kernel, Platform Driver, acpi4asus-user, Hans de Goede, Linux Upstreaming Team On Wed, Aug 15, 2018 at 5:45 PM, Andy Shevchenko <andy.shevchenko@gmail.com> wrote: > On Wed, Aug 15, 2018 at 12:34 PM, Chris Chiu <chiu@endlessm.com> wrote: >> On Mon, Jul 2, 2018 at 8:10 PM, Andy Shevchenko >> <andy.shevchenko@gmail.com> wrote: >>> On Wed, Jun 27, 2018 at 1:27 PM, Chris Chiu <chiu@endlessm.com> wrote: >>>> On Wed, Jun 20, 2018 at 10:46 PM, Chris Chiu <chiu@endlessm.com> wrote: >>>>> Make asus-wmi notify on hotkey kbd brightness changes, listen for >>>>> brightness events and update the brightness directly in the driver. >>>>> Create new do_kbd_led_set function for in-driver update, and leave >>>>> kbd_led_set for original led_classdev call path. >>>>> >>>>> Update the brightness by led_classdev_notify_brightness_hw_changed. >>>>> This will allow userspace to monitor (poll) for brightness changes >>>>> on the LED without reporting via input keymapping. >>> >>> LGTM, I'm about to push it to my review and testing queue, so, we will >>> have 1 day or so time to gather objections if any. >>> Thanks! >>> >> >> Gentle ping. How's the review so far? Maybe need more time for more info? > > Sorry, I thought we have no objections and I pushed it to our for-next > quite time ago [1]. > Should I do something differently? > > [1]: http://git.infradead.org/linux-platform-drivers-x86.git/commitdiff/dbb3d78f61badeadc7448f9d88a070a76612860f > > -- > With Best Regards, > Andy Shevchenko It's good to know. Thank you. Chris ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-08-15 9:49 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-06-20 14:46 [PATCH v3 1/2] platform/x86: asus-wmi: Call led hw_changed API on kbd brightness change Chris Chiu 2018-06-20 14:46 ` [PATCH v3 2/2] platform/x86: asus-wmi: Add keyboard backlight toggle support Chris Chiu 2018-06-27 10:27 ` [PATCH v3 1/2] platform/x86: asus-wmi: Call led hw_changed API on kbd brightness change Chris Chiu 2018-07-02 12:10 ` Andy Shevchenko 2018-08-15 9:34 ` Chris Chiu 2018-08-15 9:45 ` Andy Shevchenko 2018-08-15 9:49 ` Chris Chiu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).