Revert "mac80211: fix sw scan bracketing"
authorLuis R. Rodriguez <lrodriguez@atheros.com>
Tue, 27 Jul 2010 20:33:08 +0000 (16:33 -0400)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 28 Jul 2010 20:24:02 +0000 (16:24 -0400)
commita0daa0e7592ada797d6835f11529097aabc27ad2
tree4ac6eee98f6ae61833237699070f44e54fa5f48a
parent308883380c615b8b3d2c0eadb8680a921177c0b3
Revert "mac80211: fix sw scan bracketing"

This reverts this commit. While in theory the change is
correct the patch does not address current assumptions made
by some drivers, one which is definitley affected is ath9k.

Prior to this change the scan complete callback would be
called after we returned to the home channel and configured
the hardware RX filters. After this change we call the scan
complete callback prior to both the hw config and the config
filter. At least for ath9k this breaks quite a few assumptions
on the callback, leading to disconnects to the AP after every scan
making the driver pretty useless on STA mode. The goal behind
this commit was to address the now understood spurious warnings
from ath9k and mac80211_hwsim on scanning on two wiphys at the
same time but we have now supressed these and will address this
issue in the next kernel release.

When fixing this for good next we must first review the other
driver's dependence on this logic and perhaps consider removal
of the scan complete callback all together.

Cc: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Luis R. Rodriguez <lrodriguez@atheros.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
net/mac80211/scan.c