Messages in this thread | | | Date | Sun, 13 Apr 2008 15:16:21 -0300 | From | Henrique de Moraes Holschuh <> | Subject | Re: [PATCH 6/8] rfkill: add the WWAN radio type |
| |
On Sun, 13 Apr 2008, Ivo van Doorn wrote: > On Sunday 13 April 2008, Henrique de Moraes Holschuh wrote: > > On Sat, 12 Apr 2008, Inaky Perez-Gonzalez wrote: > > > Most rfkill hw keys are attached to the device. So I can plug wifi, wimax, > > > 3G, UWB and BT dongles (or cards) into a system and they each provide a HW > > > switch (or should provide). > > > > > > If each overrides another one, we have a problem. > > > > Yeah, and we have to avoid such problems. > > > > But let me ask you one thing. Are we talking of different rfkill > > switches, OR are we talking about different KEYs or BUTTONS in the > > keyboard/keypad/console/remote control? > > > > Because one thing has very little to do with the other. No network > > device driver shall generate an input event after I am done with the > > next patch set... > > Not sure if that will be the desired course of action. > The original rfkill function didn't work with input devices either and just > handled everything in rfkill. > But the input layer people requested the change to use input devices and let > rfkill hook into that, because a rfkill switch/key can be considered an input device. > > I've CC'ed Dmitry into this discussion since he was the one who suggested the > input device interaction initialy (and created the rfkill-input module).
In order to avoid a lot of misunderstandings, we better name the "enable/disable radio apparatus" (the circuit/config register that causes a radio to disable its RF path) and the "input hardware rf-switch device" (the thing the user presses/moves) in different ways, at least when we are talking about them.
Right now both are usually called rfkill switch, and much of the mess comes from that...
"input hardware rf-switch devices" ARE input devices, belong to the input layer, and issue input events.
"enable/disable radio apparatus" don't, and the input layer should not be abused as a "status reporting" feature for those. We certainly will benefit from those issuing kernel notifications through a notification chain for them, though.
And many devices are both at the same time (e.g. a b43 in certain hardware configurations), and many drivers have both types (e.g. thinkpad-acpi's hotkey subsystem has "input hardare rf-switch devices" functions, and thinkpad-acpi's bluetooth firmware handling have "enable/disable radio apparatus" functions).
-- "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
| |