diff options
author | Nick Kossifidis <mick@madwifi-project.org> | 2009-02-08 23:17:45 -0500 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2009-02-13 13:44:52 -0500 |
commit | fa8419d08e58c82add921b4e4eb01eb589b8586b (patch) | |
tree | 190e751535776197d5149345870014c3f89e2e7f | |
parent | 1e3e6e8fe2f422f75619b1110f78bb638ae905b9 (diff) |
ath5k: Don't call reset on config_interface
* We call reset too often and this can result various PHY problems.
On config_interface we don't need to reset the whole device.
TODO: Create a function hw_fast_reset that only resets the PCU
(tx/rx stuff) and not the whole device so that we can use this for
stuck beacons etc.
Signed-off-by: Nick Kossifidis <mickflemm@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r-- | drivers/net/wireless/ath5k/base.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/net/wireless/ath5k/base.c b/drivers/net/wireless/ath5k/base.c index 3d36e84b1e05..6837ca9f3831 100644 --- a/drivers/net/wireless/ath5k/base.c +++ b/drivers/net/wireless/ath5k/base.c | |||
@@ -2884,7 +2884,7 @@ ath5k_config_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif, | |||
2884 | { | 2884 | { |
2885 | struct ath5k_softc *sc = hw->priv; | 2885 | struct ath5k_softc *sc = hw->priv; |
2886 | struct ath5k_hw *ah = sc->ah; | 2886 | struct ath5k_hw *ah = sc->ah; |
2887 | int ret; | 2887 | int ret = 0; |
2888 | 2888 | ||
2889 | mutex_lock(&sc->lock); | 2889 | mutex_lock(&sc->lock); |
2890 | if (sc->vif != vif) { | 2890 | if (sc->vif != vif) { |
@@ -2910,9 +2910,7 @@ ath5k_config_interface(struct ieee80211_hw *hw, struct ieee80211_vif *vif, | |||
2910 | } | 2910 | } |
2911 | ath5k_beacon_update(sc, beacon); | 2911 | ath5k_beacon_update(sc, beacon); |
2912 | } | 2912 | } |
2913 | mutex_unlock(&sc->lock); | ||
2914 | 2913 | ||
2915 | return ath5k_reset_wake(sc); | ||
2916 | unlock: | 2914 | unlock: |
2917 | mutex_unlock(&sc->lock); | 2915 | mutex_unlock(&sc->lock); |
2918 | return ret; | 2916 | return ret; |