From mboxrd@z Thu Jan 1 00:00:00 1970 From: Darren Hart Subject: Re: [PATCH v9 4/4] platform/x86/dell-*: Call led_classdev_notify_brightness_hw_changed on kbd brightness change Date: Fri, 17 Mar 2017 15:33:05 -0700 Message-ID: <20170317223305.GA18063@fury> References: <20170316105535.8885-1-hdegoede@redhat.com> <20170316105535.8885-4-hdegoede@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from bombadil.infradead.org ([65.50.211.133]:56689 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751061AbdCQWeP (ORCPT ); Fri, 17 Mar 2017 18:34:15 -0400 Content-Disposition: inline In-Reply-To: <20170316105535.8885-4-hdegoede@redhat.com> Sender: platform-driver-x86-owner@vger.kernel.org List-ID: To: Hans de Goede Cc: Andy Shevchenko , Pali =?iso-8859-1?Q?Roh=E1r?= , platform-driver-x86@vger.kernel.org On Thu, Mar 16, 2017 at 11:55:35AM +0100, Hans de Goede wrote: > Make dell-wmi notify on hotkey kbd brightness changes, listen for this > in dell-laptop and call led_classdev_notify_brightness_hw_changed. > > This will allow userspace to monitor (poll) for brightness changes on > these LEDs caused by the hotkey. > > Signed-off-by: Hans de Goede Thanks Hans, this appears to be consistent with the conclusion on v8 between you and Pali. While I don't care for the cross-driver-dependency, that's pre-existing and not something I have a solution for. So this looks good to me, pending Pali's final review. Pali, I know you have had some reservations reading through the v8 discussion. I believe Hans has addressed each of those sufficiently for the purposes of this patch set. As a follow-on effort, I'd like to discuss the future of libsmbios with the Dell folks and see if we can't phase it out. Hans, a couple of nits on this patch. To keep the subject under 80, I used: platform/x86: dell-*: Call new led hw_changed API on kbd brightness change since you used the full led function name in the commit message anyway. I presume the changelog was intended to go after the --- and you didn't want it going into the commit itself, so I've removed it. I'll await Pali's final Reviewed-by before pushing to testing, but you can see my minor tweaks listed above in the dell branch: git://git.infradead.org/linux-platform-drivers-x86.git dell http://git.infradead.org/linux-platform-drivers-x86.git/log/refs/heads/dell Thanks, -- Darren Hart VMware Open Source Technology Center