From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934546AbcAKT2F (ORCPT ); Mon, 11 Jan 2016 14:28:05 -0500 Received: from out4-smtp.messagingengine.com ([66.111.4.28]:54877 "EHLO out4-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932953AbcAKT2B convert rfc822-to-8bit (ORCPT ); Mon, 11 Jan 2016 14:28:01 -0500 Message-Id: <1452540480.2848000.489039714.26068C20@webmail.messagingengine.com> X-Sasl-Enc: KQL8WN3kBY+sYTUMRaswvmohZzKofAcxaM24lNQZwZ4r 1452540480 From: Henrique de Moraes Holschuh To: Darren Hart Cc: Pavel Machek , =?ISO-8859-1?Q?Pali=20Roh=E1r?= , ibm-acpi-devel@lists.sourceforge.net, platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org, "Fabio D'Urso" MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="ISO-8859-1" X-Mailer: MessagingEngine.com Webmail Interface - ajax-6cda141f Subject: Re: [PATCH] thinkpad_acpi: Add support for keyboard backlight Date: Mon, 11 Jan 2016 17:28:00 -0200 In-Reply-To: <20160111190411.GE10643@malice.jf.intel.com> References: <1450982818-11694-1-git-send-email-pali.rohar@gmail.com> <20160104201231.GB5847@amd> <1452361154.673684.487414482.6CCEAE4B@webmail.messagingengine.com> <1452361601.676356.487415834.4652D305@webmail.messagingengine.com> <20160111190411.GE10643@malice.jf.intel.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jan 11, 2016, at 17:04, Darren Hart wrote: > On Sat, Jan 09, 2016 at 03:46:41PM -0200, Henrique de Moraes Holschuh > wrote: > > On Sat, Jan 9, 2016, at 15:39, Henrique de Moraes Holschuh wrote: > > > On Mon, Jan 4, 2016, at 18:12, Pavel Machek wrote: > > > > > This patch adds support for controlling keyboard backlight via standard > > > > > linux led class interface (::kbd_backlight). It uses ACPI HKEY device with > > > > > MLCG and MLCS methods. > > > > > > > > > > Signed-off-by: Pali Rohár > > > > > Tested-by: Fabio D'Urso > > > > > > > > On my thinkpad, keyboard light is controlled by > > > > > > > > /sys/class/leds/tpacpi\:\:thinklight/brightness > > > > > > > > (that's a bad name). > > > > > > That's because the driver was not updated to support your laptop, AND I > > > don't recall if someone mapped the full behavior of the ACPI thinklight > > > interface in your thinkpad :-( > > > > Argh. If by "keyboard light" you mean the LED above the *screen* that > > shines down on the whole keyboard, please disregard my previous reply... > > > > As for the naming, the idea of a LED up there shining in the keyboard > > is: > > 1. patented by IBM > > 2. named "ThinkLight" by IBM, one of the "Think Technologies" in the > > "ThinkPad" (add TM after everything :p) and every > > old-timer thinkpad user knew it by that name. > > 4. called "thinklight" by the driver since before the LED sysfs class > > even existed :p > > > > ibm-acpi, since then renamed thinkpad-acpi *predates* most generic > > interfaces. Heck, it predates sysfs. > > > > So, this is ABI set in stone. If there is a way to add an "alias" of > > kbd_backlight that won't drive userspace crazy, we might do that though. > > But it looks quite risky to me... > > Henrique, so are you taking back your Ack from 10 minutes prior? Hmm? No, the ACK stands. Pavel was talking about another feature altogether, apparently: older thinkpads did not have "keyboard backlight" (as in light from below the keys). They had a "ThinkLight", which is an overhead light that shines down on the keyboard. The two features are not the same (and are handled differently by the firmware, for whatever reason), although they do serve the same purpose. I don't think a thinkpad will ever have both features at the same time, so I have no idea why they changed the firmware interface. The patch adds support to the "keyboard backlight" feature, which was previously NOT supported. > I've dropped this patch. Please let me know if I should pick it back up. Please pick it back up. -- "One disk to rule them all, One disk to find them. One disk to bring them all and in the darkness grind them. In the Land of Redmond where the shadows lie." -- The Silicon Valley Tarot Henrique Holschuh