wl1251: fix channel setting in wl1251_op_config()
authorKalle Valo <kalle.valo@nokia.com>
Fri, 7 Aug 2009 10:34:49 +0000 (13:34 +0300)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 14 Aug 2009 13:13:41 +0000 (09:13 -0400)
There is a bug in wl1251_op_config(). It was calling join with previous
channel. Fix it by setting assigning wl->channel before calling join command.

Signed-off-by: Kalle Valo <kalle.valo@nokia.com>
Reviewed-by: Vidhya Govindan <vidhya.govindan@nokia.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/wl12xx/wl1251_main.c

index e575b7869fbca69591ede1b39e7b29489f2b3ed2..568b640aaa5293dc927ca3790bc2ad8c549bd5d7 100644 (file)
@@ -593,12 +593,12 @@ static int wl1251_op_config(struct ieee80211_hw *hw, u32 changed)
                goto out;
 
        if (channel != wl->channel) {
+               wl->channel = channel;
+
                ret = wl1251_join(wl, wl->bss_type, wl->channel,
                                  wl->beacon_int, wl->dtim_period);
                if (ret < 0)
                        goto out_sleep;
-
-               wl->channel = channel;
        }
 
        ret = wl1251_build_null_data(wl);