diff options
author | Luciano Coelho <luciano.coelho@nokia.com> | 2009-12-11 08:40:47 -0500 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2009-12-28 16:31:30 -0500 |
commit | cd264769f7a571fc2a61765757ed85c56f02e9d3 (patch) | |
tree | c932d29e28673807c5544af2335a050944a64c04 /drivers | |
parent | a2d0e3f13037fa7dc4094c47e0378800a064ee26 (diff) |
wl1271: check result code from the join command
We were not checking the return value from the call to wl1271_cmd_join().
Added a check to make things more reliable.
Signed-off-by: Luciano Coelho <luciano.coelho@nokia.com>
Reviewed-by: Juuso Oikarinen <juuso.oikarinen@nokia.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/net/wireless/wl12xx/wl1271_main.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/drivers/net/wireless/wl12xx/wl1271_main.c b/drivers/net/wireless/wl12xx/wl1271_main.c index 7f164e532211..bf1d0fd93bcc 100644 --- a/drivers/net/wireless/wl12xx/wl1271_main.c +++ b/drivers/net/wireless/wl12xx/wl1271_main.c | |||
@@ -1545,7 +1545,11 @@ static void wl1271_op_bss_info_changed(struct ieee80211_hw *hw, | |||
1545 | memcmp(wl->bssid, bss_conf->bssid, ETH_ALEN)) { | 1545 | memcmp(wl->bssid, bss_conf->bssid, ETH_ALEN)) { |
1546 | wl->rx_config |= CFG_BSSID_FILTER_EN; | 1546 | wl->rx_config |= CFG_BSSID_FILTER_EN; |
1547 | memcpy(wl->bssid, bss_conf->bssid, ETH_ALEN); | 1547 | memcpy(wl->bssid, bss_conf->bssid, ETH_ALEN); |
1548 | wl1271_cmd_join(wl); | 1548 | ret = wl1271_cmd_join(wl); |
1549 | if (ret < 0) { | ||
1550 | wl1271_warning("cmd join failed %d", ret); | ||
1551 | goto out_sleep; | ||
1552 | } | ||
1549 | wl->joined = true; | 1553 | wl->joined = true; |
1550 | } | 1554 | } |
1551 | 1555 | ||