Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1181163 > unrolled thread
| Started by | Alex Hung <alex.hung@canonical.com> |
|---|---|
| First post | 2015-07-10 04:00 +0200 |
| Last post | 2015-07-10 04:00 +0200 |
| Articles | 1 — 1 participant |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH][v2] asus-rbtn: new driver for asus radio button for Windows 8 Alex Hung <alex.hung@canonical.com> - 2015-07-10 04:00 +0200
| From | Alex Hung <alex.hung@canonical.com> |
|---|---|
| Date | 2015-07-10 04:00 +0200 |
| Subject | Re: [PATCH][v2] asus-rbtn: new driver for asus radio button for Windows 8 |
| Message-ID | <pKvLA-7UM-1@gated-at.bofh.it> |
On Fri, Jul 10, 2015 at 4:52 AM, Darren Hart <dvhart@infradead.org> wrote: > On Tue, Jul 07, 2015 at 04:25:18PM +0200, Pali Rohár wrote: >> On Monday 06 July 2015 15:43:28 Darren Hart wrote: >> > 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, >> >> Yes, I understand too that ATK4001 device has more functions as one for >> LED control. And in this case it is not good to use name from one >> specific functionality. >> >> And about name: >> >> I chose name rbtn for dell driver because ACPI device in DSDT table is >> named RBTN and acpi id is DELRBTN. >> >> I think that acpi-rbtn.c (radio button) is not ideal name for driver >> which at SW level controls LED device associated with wireless devices. >> >> I would rather follow acpi device name, but in this case ATK4001 is even >> worse name which does not say anything... >> >> Maybe better name could be asus-wireless? Still I do not have any good >> name, so choose something... > > I suggested rbtn to try and start showing some kind of consistency in the > directory - however, it's no better than many of the other existing options, > including -rfkill -wireless -laptop, etc. > > Speaking of -laptop, the asus-laptop driver supports ATK0100 and ATK0101. This > ATK4001 device appears to perform the same type of features. The question is > whether or not it can be reasonably incorporated into the existing asus-laptop > driver or not. > > (I intended to do that review myself, but I haven't found the time in 48 hours, > so I'm just going to send this out and ask that Alex have a look and provide his > thoughts on asus-laptop) > > -- > Darren Hart > Intel Open Source Technology Center Certainly, I should have time to look into asus-laptop next week. -- Cheers, Alex Hung -- 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/
Back to top | Article view | linux.kernel
csiph-web