Thread (11 messages) 11 messages, 5 authors, 2009-05-30

Re: [ath5k-devel] [PATCH] ath5k: added initial RFKILL support

From: Alan Jenkins <hidden>
Date: 2009-05-30 15:26:25
Also in: lkml

On 5/30/09, Johannes Berg [off-list ref] wrote:
On Sat, 2009-05-30 at 00:33 +0200, Tobias Doerffel wrote:
quoted
quoted
Could you try to work against v11
of my rfkill patch, or better even against the cfg80211 rfkill instead?
I didn't look at both of them yet. However I think we first should try to
get
current rfkill support into mainline as fast as possible. Improvements and

adaptations to reworked rfkill framework can follow.
No other comments from me -- but since the v11 of the rfkill rewrite
will certainly hit the tree before your patch (it's almost in) you
_will_ have to work against it. I'll also try to make the cfg80211
rfkill hit the tree very soon for reasons I'll explain elsewhere -- you
would be better off working against that because then your rfkill code
is very very very simple and doesn't need to do much at all.

johannes
Btw, I believe the new rfkill core behaviour should avoid the concern
I raised about the EeePC.

Reminder: the eeepc-laptop implements platform rfkill on the original
model(s) by hotplugging the entire ath5k device.  If you boot with it
in the blocked state, the old rfkill core would "lock in" that state
as the default.  If you then hotplug the ath5k device _and it comes up
with an rfkill device of it's own_, my concern is that the new rfkill
device would be initialized to a blocked state.  It would be
impossible to enable the wireless.

The new core removes this idea of a separate "default" state, so it
can't have this problem.

Regards
Alan
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help