From: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
To: Ivo van Doorn <ivdoorn@gmail.com>
Cc: linux-kernel@vger.kernel.org, "Thomas Renninger" <trenn@suse.de>,
"Inaky Perez-Gonzalez" <inaky@linux.intel.com>,
"Iñaky Pérez-González" <inaky.perez-gonzalez@intel.com>,
"John W. Linville" <linville@tuxdriver.com>,
"David S. Miller" <davem@davemloft.net>
Subject: Re: [PATCH 09/15] rfkill: add the WWAN radio type
Date: Tue, 20 May 2008 22:12:57 -0300 [thread overview]
Message-ID: <20080521011257.GA26427@khazad-dum.debian.net> (raw)
In-Reply-To: <200805201208.53740.IvDoorn@gmail.com>
On Tue, 20 May 2008, Ivo van Doorn wrote:
> On Sunday 18 May 2008, Henrique de Moraes Holschuh wrote:
> > Unfortunately, instead of adding a generic Wireless WAN type, a technology-
> > specific type (WiMAX) was added. That's useless for other WWAN devices,
> > such as EDGE, UMTS, X-RTT and other such radios.
> >
> > Add a WWAN rfkill type for generic wireless WAN devices. No keys are added
> > as most devices really want to use KEY_WLAN for WWAN control (in a cycle of
> > none, WLAN, WWAN, WLAN+WWAN) and need no specific keycode added.
> >
> > Signed-off-by: Inaky Perez-Gonzalez <inaky@linux.intel.com>
>
> If WiMAX is a subset of the WWAN technology, shouldn't we replace WiMAX completely
> in rfkill? Otherwise people might get ideas and add the other technologies seperately as well. ;)
>
> Other then that, the addition of WWAN is fine with me. :)
Actually, I have no reason to believe there *isn't* a device with
KEY_WIMAX... but we need the WWAN type before we bother with a WiMax
type :-)
IMO, we should fix the issue by adding a "supertype", i.e., a class.
Use that for generic grouping, and let type be more specific when there
is a reason for it. Repeat the class as the type to get a "generic"
type, when there is no reason to bother with more specific types.
Stuff like "WiMax", "802.1a", "EDGE", "GRPS", "802.1bg" would be the
type, and class would be "WLAN", "WWAN", etc.
We would only deal with classes on rfkill-input. Anything else more
specific should be done in userspace instead, disabling rfkill-input.
--
"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
next prev parent reply other threads:[~2008-05-21 1:13 UTC|newest]
Thread overview: 58+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-18 18:47 [RFC] rfkill class rework Henrique de Moraes Holschuh
2008-05-18 18:47 ` [PATCH 01/15] ACPI: thinkpad-acpi: fix initialization error paths Henrique de Moraes Holschuh
2008-05-18 18:47 ` [PATCH 02/15] ACPI: thinkpad-acpi: fix LED handling on older ThinkPads Henrique de Moraes Holschuh
2008-05-18 18:47 ` [PATCH 03/15] Input: rename SW_RADIO to SW_RFKILL_ALL (v2) Henrique de Moraes Holschuh
2008-05-18 18:47 ` [PATCH 04/15] rfkill: clarify meaning of rfkill states Henrique de Moraes Holschuh
2008-05-20 10:08 ` Ivo van Doorn
2008-05-18 18:47 ` [PATCH 05/15] rfkill: fix minor typo in kernel doc Henrique de Moraes Holschuh
2008-05-20 10:08 ` Ivo van Doorn
2008-05-18 18:47 ` [PATCH 06/15] rfkill: handle SW_RFKILL_ALL events Henrique de Moraes Holschuh
2008-05-20 10:08 ` Ivo van Doorn
2008-05-18 18:47 ` [PATCH 07/15] rfkill: add parameter to disable radios by default Henrique de Moraes Holschuh
2008-05-18 18:47 ` [PATCH 08/15] rfkill: add read-write rfkill switch support Henrique de Moraes Holschuh
2008-05-20 10:08 ` Ivo van Doorn
2008-05-18 18:47 ` [PATCH 09/15] rfkill: add the WWAN radio type Henrique de Moraes Holschuh
2008-05-20 10:08 ` Ivo van Doorn
2008-05-21 1:12 ` Henrique de Moraes Holschuh [this message]
2008-05-21 3:35 ` Inaky Perez-Gonzalez
2008-05-21 3:42 ` Henrique de Moraes Holschuh
2008-05-21 6:48 ` Inaky Perez-Gonzalez
2008-05-21 14:07 ` Henrique de Moraes Holschuh
2008-05-18 18:48 ` [PATCH 10/15] rfkill: rework suspend and resume handlers Henrique de Moraes Holschuh
2008-05-20 10:08 ` Ivo van Doorn
2008-05-18 18:48 ` [PATCH 11/15] rfkill: add notifier chains support Henrique de Moraes Holschuh
2008-05-19 8:44 ` Thomas Renninger
2008-05-19 13:10 ` Henrique de Moraes Holschuh
2008-05-20 10:09 ` Ivo van Doorn
2008-05-18 18:48 ` [PATCH 12/15] rfkill: add type string helper Henrique de Moraes Holschuh
2008-05-20 10:09 ` Ivo van Doorn
2008-05-18 18:48 ` [PATCH 13/15] rfkill: add uevent notifications Henrique de Moraes Holschuh
2008-05-20 10:09 ` Ivo van Doorn
2008-05-18 18:48 ` [PATCH 14/15] rfkill: do not allow userspace to override ALL RADIOS OFF Henrique de Moraes Holschuh
2008-05-20 10:09 ` Ivo van Doorn
2008-05-22 20:51 ` Henrique de Moraes Holschuh
2008-05-23 14:15 ` Ivo van Doorn
2008-05-27 14:08 ` Henrique de Moraes Holschuh
2008-05-27 14:38 ` Ivo van Doorn
2008-05-27 17:41 ` Henrique de Moraes Holschuh
2008-05-27 18:13 ` Ivo van Doorn
2008-05-18 18:48 ` [PATCH 15/15] rfkill: document rw rfkill switches and clarify input subsystem interactions Henrique de Moraes Holschuh
2008-05-19 17:51 ` Randy Dunlap
2008-05-19 22:04 ` Henrique de Moraes Holschuh
2008-05-19 22:52 ` Elias Oltmanns
2008-05-19 22:56 ` Randy Dunlap
2008-05-20 10:09 ` Ivo van Doorn
2008-05-20 15:54 ` Henrique de Moraes Holschuh
2008-05-20 17:18 ` Ivo van Doorn
2008-05-21 1:44 ` Henrique de Moraes Holschuh
2008-05-29 0:45 ` [PATCH 15/15] rfkill: document rw rfkill switches and clarify input subsystem interactions (v2) Henrique de Moraes Holschuh
2008-05-29 13:02 ` Ivo van Doorn
2008-05-29 16:26 ` Henrique de Moraes Holschuh
2008-05-29 17:19 ` Ivo van Doorn
2008-05-29 17:22 ` Henrique de Moraes Holschuh
2008-05-29 17:40 ` Ivo van Doorn
2008-05-29 17:46 ` Henrique de Moraes Holschuh
2008-05-29 18:58 ` Dmitry Torokhov
2008-05-29 21:16 ` Henrique de Moraes Holschuh
2008-05-29 21:25 ` [PATCH] Input: rename SW_RADIO to SW_RFKILL_ALL (v2) Henrique de Moraes Holschuh
2008-06-04 3:11 ` [PATCH 15/15] rfkill: document rw rfkill switches and clarify input subsystem interactions (v2) Henrique de Moraes Holschuh
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=20080521011257.GA26427@khazad-dum.debian.net \
--to=hmh@hmh.eng.br \
--cc=davem@davemloft.net \
--cc=inaky.perez-gonzalez@intel.com \
--cc=inaky@linux.intel.com \
--cc=ivdoorn@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=trenn@suse.de \
/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
all inboxes | Powered by JetHome®