From: "Pali Rohár" <pali.rohar@gmail.com>
To: Hans de Goede <hdegoede@redhat.com>
Cc: Andy Shevchenko <andy.shevchenko@gmail.com>,
Darren Hart <dvhart@infradead.org>,
Andy Shevchenko <andy@infradead.org>,
Henrique de Moraes Holschuh <ibm-acpi@hmh.eng.br>,
Jacek Anaszewski <jacek.anaszewski@gmail.com>,
Pavel Machek <pavel@ucw.cz>,
Platform Driver <platform-driver-x86@vger.kernel.org>,
Linux LED Subsystem <linux-leds@vger.kernel.org>
Subject: Re: [PATCH v8 0/7] platform/x86: Notify userspace about hotkeys changing kbd-backlight brightness
Date: Thu, 2 Mar 2017 15:22:53 +0100 [thread overview]
Message-ID: <20170302142253.GC29684@pali> (raw)
In-Reply-To: <98a12674-9763-670b-5978-c4150162cd79@redhat.com>
On Thursday 02 March 2017 15:12:52 Hans de Goede wrote:
> Hi,
>
> On 02-03-17 00:10, Andy Shevchenko wrote:
> >On Thu, Feb 9, 2017 at 5:44 PM, Hans de Goede <hdegoede@redhat.com> wrote:
> >
> >>Here is v8 of the platform drivers changes matching / using the new
> >>v8 LED subsys led_classdev_notify_brightness_hw_changed() functionality
> >>queued in -next.
> >>
> >>There have been some changes (and preparation patches added) compared
> >>to the previous versions since the new LED api requires the driver to
> >>only call led_classdev_notify_brightness_hw_changed() on hotkey triggered
> >>changes and the ACPI events indicating brightness changes also get
> >>triggered when setting the brightness to led_set_brightness (or sysfs).
> >>
> >>This series depends on the patch adding
> >>led_classdev_notify_brightness_hw_changed() to the LED subsystem,
> >>Jacek can you create a stable branch with just that patch which the
> >>platform/x86 platform maintainers can merge, so that they can apply
> >>this series ?
> >
> >I briefly read the discussion and looks like I'll wait for v9 after
> >merge window is closed.
>
> Ok.
>
> Note the 3 acpi-thinkpad patches have been in
> linux-platform-drivers-x86.git/testing for a while now and there has been
> no discussion surrounding those. Do you want me to send a v9 of those too,
> or just the Dell driver patches ?
If we all agree on change for Dell driver to just drop one event, maybe
we can discuss if similar approach should be used also for Thinkpad
driver...
> Regards,
>
> Hans
--
Pali Rohár
pali.rohar@gmail.com
next prev parent reply other threads:[~2017-03-02 14:22 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-09 15:44 [PATCH v8 0/7] platform/x86: Notify userspace about hotkeys changing kbd-backlight brightness Hans de Goede
2017-02-09 15:44 ` [PATCH v8 1/7] platform/x86/thinkpad_acpi: Stop setting led_classdev brightness directly Hans de Goede
2017-02-09 18:09 ` Henrique de Moraes Holschuh
2017-03-01 11:27 ` Pali Rohár
2017-03-01 11:57 ` Hans de Goede
2017-03-01 12:00 ` Pali Rohár
2017-03-01 12:04 ` Hans de Goede
2017-03-01 14:24 ` Marco Trevisan (Treviño)
2017-02-09 15:44 ` [PATCH v8 2/7] platform/x86/thinkpad_acpi: Use brightness_set_blocking callback for LEDs Hans de Goede
2017-02-09 18:00 ` Henrique de Moraes Holschuh
2017-02-09 15:44 ` [PATCH v8 3/7] platform/x86/thinkpad: Call led_classdev_notify_brightness_hw_changed on kbd brightness change Hans de Goede
2017-02-09 18:08 ` Henrique de Moraes Holschuh
2017-02-13 22:52 ` Andy Shevchenko
2017-02-14 9:25 ` Hans de Goede
2017-02-14 9:33 ` Andy Shevchenko
2017-02-17 3:45 ` Darren Hart
2017-02-14 9:36 ` Pali Rohár
2017-02-09 15:44 ` [PATCH v8 4/7] platform/x86/dell-*: Add a generic dell-laptop notifier chain Hans de Goede
2017-02-21 14:18 ` Pali Rohár
2017-02-09 15:44 ` [PATCH v8 5/7] platform/x86/dell-laptop: Refactor kbd_led_triggers_store() Hans de Goede
2017-02-21 14:02 ` Pali Rohár
2017-02-09 15:44 ` [PATCH v8 6/7] platform/x86/dell-laptop: Protect kbd_state against races Hans de Goede
2017-02-21 14:06 ` Pali Rohár
2017-02-21 14:18 ` Hans de Goede
2017-02-21 14:25 ` Pali Rohár
2017-02-21 14:42 ` Hans de Goede
2017-02-21 14:53 ` Pali Rohár
2017-02-09 15:44 ` [PATCH v8 7/7] platform/x86/dell-*: Call led_classdev_notify_brightness_hw_changed on kbd brightness change Hans de Goede
2017-02-21 14:11 ` Pali Rohár
2017-02-21 14:40 ` Hans de Goede
2017-02-21 14:50 ` Pali Rohár
2017-02-21 14:56 ` Hans de Goede
2017-02-21 15:13 ` Pali Rohár
2017-02-21 16:14 ` Hans de Goede
2017-02-21 17:08 ` Pali Rohár
2017-02-22 8:36 ` Hans de Goede
2017-02-22 8:49 ` Pali Rohár
2017-02-22 10:24 ` Hans de Goede
2017-02-22 12:01 ` Pali Rohár
2017-02-22 12:20 ` Hans de Goede
2017-03-01 11:15 ` Pali Rohár
2017-03-01 12:02 ` Hans de Goede
2017-03-01 12:55 ` Pali Rohár
2017-03-01 13:58 ` Hans de Goede
2017-03-03 12:00 ` Pali Rohár
2017-03-06 13:39 ` Hans de Goede
2017-03-16 10:11 ` Hans de Goede
2017-02-21 20:47 ` Jacek Anaszewski
2017-02-09 20:21 ` [PATCH v8 0/7] platform/x86: Notify userspace about hotkeys changing kbd-backlight brightness Jacek Anaszewski
2017-02-11 20:08 ` Pavel Machek
2017-03-01 23:10 ` Andy Shevchenko
2017-03-02 14:12 ` Hans de Goede
2017-03-02 14:22 ` Pali Rohár [this message]
2017-03-02 14:30 ` Hans de Goede
2017-03-02 14:34 ` Pali Rohár
2017-03-02 15:27 ` Andy Shevchenko
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170302142253.GC29684@pali \
--to=pali.rohar@gmail.com \
--cc=andy.shevchenko@gmail.com \
--cc=andy@infradead.org \
--cc=dvhart@infradead.org \
--cc=hdegoede@redhat.com \
--cc=ibm-acpi@hmh.eng.br \
--cc=jacek.anaszewski@gmail.com \
--cc=linux-leds@vger.kernel.org \
--cc=pavel@ucw.cz \
--cc=platform-driver-x86@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).