diff options
author | Arik Nemtsov <arik@wizery.com> | 2013-03-12 17:19:45 +0200 |
---|---|---|
committer | Luciano Coelho <coelho@ti.com> | 2013-03-25 12:33:13 +0200 |
commit | 75592be5e35b31eb28dacf578cfe82f0e9bc0ac0 (patch) | |
tree | ae3e76a49dc4e6907ab2c71c60011774c51ea59b | |
parent | 0e752df6fda25993acc30e1162808fbb2543be03 (diff) | |
download | linux-3.10-75592be5e35b31eb28dacf578cfe82f0e9bc0ac0.tar.gz linux-3.10-75592be5e35b31eb28dacf578cfe82f0e9bc0ac0.tar.bz2 linux-3.10-75592be5e35b31eb28dacf578cfe82f0e9bc0ac0.zip |
wlcore: correctly check state before regdomain conf
The wlcore state was checked without the mutex being taken. This leads
to WARN_ONs sometimes if a notification arrives when the driver
is on, but the mutex is only taken after it is off. This usually
happens if stopping the driver while connected to a network.
Signed-off-by: Arik Nemtsov <arik@wizery.com>
Signed-off-by: Luciano Coelho <coelho@ti.com>
-rw-r--r-- | drivers/net/wireless/ti/wlcore/main.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/net/wireless/ti/wlcore/main.c b/drivers/net/wireless/ti/wlcore/main.c index 43865d1348e..248daa9bb5f 100644 --- a/drivers/net/wireless/ti/wlcore/main.c +++ b/drivers/net/wireless/ti/wlcore/main.c @@ -108,8 +108,7 @@ static void wl1271_reg_notify(struct wiphy *wiphy, } - if (likely(wl->state == WLCORE_STATE_ON)) - wlcore_regdomain_config(wl); + wlcore_regdomain_config(wl); } static int wl1271_set_rx_streaming(struct wl1271 *wl, struct wl12xx_vif *wlvif, @@ -3364,6 +3363,10 @@ void wlcore_regdomain_config(struct wl1271 *wl) return; mutex_lock(&wl->mutex); + + if (unlikely(wl->state != WLCORE_STATE_ON)) + goto out; + ret = wl1271_ps_elp_wakeup(wl); if (ret < 0) goto out; |