On 07/25/2011 08:29 AM, Eliad Peller wrote:
The hw is currently not configured when going
back on-channel.
I am less sure about this patch. With the existing code,
I think it should catch going from on channel to off
and do the hw config properly.
With your change it will also reconfig the hardware, but it will
reconfig even if we were already on-channel (if, for instance,
local->tmp_channel is oper-channel), right?
Can you please explain in more detail how this code is
broken?
Thanks,
Ben
Signed-off-by: Eliad Peller<eliad@xxxxxxxxxx>
---
net/mac80211/work.c | 3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/net/mac80211/work.c b/net/mac80211/work.c
index 3291958..e6abb15 100644
--- a/net/mac80211/work.c
+++ b/net/mac80211/work.c
@@ -1075,7 +1075,6 @@ static void ieee80211_work_work(struct work_struct *work)
}
if (!remain_off_channel&& local->tmp_channel) {
- bool on_oper_chan = ieee80211_cfg_on_oper_channel(local);
local->tmp_channel = NULL;
/* If tmp_channel wasn't operating channel, then
* we need to go back on-channel.
@@ -1085,7 +1084,7 @@ static void ieee80211_work_work(struct work_struct *work)
* we still need to do a hardware config. Currently,
* we cannot be here while scanning, however.
*/
- if (ieee80211_cfg_on_oper_channel(local)&& !on_oper_chan)
+ if (!ieee80211_cfg_on_oper_channel(local))
ieee80211_hw_config(local, 0);
/* At the least, we need to disable offchannel_ps,
--
Ben Greear <greearb@xxxxxxxxxxxxxxx>
Candela Technologies Inc http://www.candelatech.com
--
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
