Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756083AbbGFWnj (ORCPT ); Mon, 6 Jul 2015 18:43:39 -0400 Received: from bombadil.infradead.org ([198.137.202.9]:45195 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755941AbbGFWnc (ORCPT ); Mon, 6 Jul 2015 18:43:32 -0400 Date: Mon, 6 Jul 2015 15:43:28 -0700 From: Darren Hart To: Alex Hung Cc: Pali =?iso-8859-1?Q?Roh=E1r?= , Corentin Chary , "platform-driver-x86@vger.kernel.org" , acpi4asus-user@lists.sourceforge.net, linux-kernel@vger.kernel.org Subject: Re: [PATCH][v2] asus-rbtn: new driver for asus radio button for Windows 8 Message-ID: <20150706224328.GB111639@vmdeb7> References: <20150626145621.GU2890@pali> <20150629122953.GF16230@pali> <20150630085814.GE27720@pali> <20150701114849.GE17577@pali> <20150703072523.GB1841@pali> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2673 Lines: 72 On Mon, Jul 06, 2015 at 09:35:40AM +0800, Alex Hung wrote: > ATK4001 is an ACPI device for wireless hotkey, similar to how Dell and > HP are doing it. It is just ASUS who decides LED should be controlled > by software unlike HP whose LED is driven by hardware pins on mini > card. Alex, please refrain from top posting on Linux kernel mailing lists, it breaks the established practice the readers are setup for. Regarding the ATK4001 device, it did sound like it did more than control the radios and the associcated LED. If that is all it does, then asus-rbtn is fine. If it does something beyond that, we need understand what that is, as a more platform-centric name would be more appropriate. Thanks, > > On Fri, Jul 3, 2015 at 3:25 PM, Pali Roh?r wrote: > > On Thursday 02 July 2015 15:10:39 Alex Hung wrote: > >> Thanks for the support. I will create v3 based with LED triggers. > >> > >> Just for information. ASUS's wording is as below: > >> > >> Fn+F2 can be used to turn on or off all radio capabilities in the > >> device (as known as airplane mode switch). I don't have any > >> preferences on the name. We may use the term airplane-mode do if > >> asus-rbtn is not preferred. Let me know if there are any suggestions. > >> > > > > This is OK, but I though that ACPI ATK4001 device is some > > multifunctional device, which is doing more then LED control... (At > > least I understand it from your comments). So in this case I would not > > call driver led/radio related. Right? > > > >> > >> On Wed, Jul 1, 2015 at 7:48 PM, Pali Roh?r wrote: > >> > On Wednesday 01 July 2015 00:09:41 Alex Hung wrote: > >> >> ATK4001 is an independent ACPI device, and Method(HSWC) is its method > >> >> to control LED (actually it has other functions but only LED is needed > >> >> so far). > >> > > >> > If this driver is for supporting ACPI ATK4001 device (which has couple > >> > of methods, not only LED) it is really good idea to name it asus-rbtn? > >> > > >> > Darren, you as maintainer of platform drivers, what do you think? I > >> > believe in future this driver will be extended to support more functions > >> > and so name asus-rbtn will be confusing. > >> > > >> > -- > >> > Pali Roh?r > >> > pali.rohar@gmail.com > >> > > > > -- > > Pali Roh?r > > pali.rohar@gmail.com > > > > -- > Cheers, > Alex Hung > -- Darren Hart Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/