On Tue, Aug 18, 2009 at 10:26:33AM +0200, Johannes Berg wrote: > > The logic is just: > > > > if (idle) { > > issue "radio disable" command > > } else { > > issue "radio enable" command > > set channel > > set tx power > > set antenna config > > } > > > > So if a channel change hapens while we are idle, that event won't get > > lost -- it'll just happen later. > > Ok, yeah, that looks fine. OK, thanks. Since your patch is already in, I've replaced this patch by: >From 1af73ff8d1309b924f6f4accd61245b57374dd78 Mon Sep 17 00:00:00 2001 From: Lennert Buytenhek <buytenh@xxxxxxxxxxxxxx> Date: Mon, 17 Aug 2009 23:59:40 +0200 Subject: [PATCH 15/29] mwl8k: implement rfkill By checking IEEE80211_CONF_CHANGE_IDLE in mwl8k_config(). Signed-off-by: Lennert Buytenhek <buytenh@xxxxxxxxxxx> --- drivers/net/wireless/mwl8k.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/drivers/net/wireless/mwl8k.c b/drivers/net/wireless/mwl8k.c index 177d92f..65eefe8 100644 --- a/drivers/net/wireless/mwl8k.c +++ b/drivers/net/wireless/mwl8k.c @@ -2854,6 +2854,12 @@ static int mwl8k_config_wt(struct work_struct *wt) struct mwl8k_priv *priv = hw->priv; int rc = 0; + if (conf->flags & IEEE80211_CONF_IDLE) { + mwl8k_cmd_802_11_radio_disable(hw); + priv->current_channel = NULL; + goto mwl8k_config_exit; + } + if (mwl8k_cmd_802_11_radio_enable(hw)) { rc = -EINVAL; goto mwl8k_config_exit; -- 1.5.6.4 -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html