wl1251: use beacon interval and dtim period provided by mac80211
authorKalle Valo <kalle.valo@nokia.com>
Fri, 7 Aug 2009 10:34:12 +0000 (13:34 +0300)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 14 Aug 2009 13:13:39 +0000 (09:13 -0400)
wl1251 was using hardcoded beacon intervals and dtim periods, use the ones
provided by mac80211 instead.

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.h
drivers/net/wireless/wl12xx/wl1251_cmd.c
drivers/net/wireless/wl12xx/wl1251_main.c

index 44c5001beedaef8457a84ceb0c14835d990d455c..86d2abea4e1efa16c735f9be3093760d8c2867a4 100644 (file)
@@ -360,6 +360,9 @@ struct wl1251 {
        /* PSM mode requested */
        bool psm_requested;
 
+       u16 beacon_int;
+       u8 dtim_period;
+
        /* in dBm */
        int power_level;
 
@@ -392,6 +395,9 @@ void wl1251_disable_interrupts(struct wl1251 *wl);
 
 #define WL1251_TX_QUEUE_MAX_LENGTH 20
 
+#define WL1251_DEFAULT_BEACON_INT 100
+#define WL1251_DEFAULT_DTIM_PERIOD 1
+
 #define CHIP_ID_1251_PG10                 (0x7010101)
 #define CHIP_ID_1251_PG11                 (0x7020101)
 #define CHIP_ID_1251_PG12                 (0x7030101)
index 50daa9bda4aea3a762395c118abaf073bd5a606f..9cd46ce88e51c6b5cdcd6cb2bd9b8bb353d45237 100644 (file)
@@ -273,7 +273,10 @@ int wl1251_cmd_join(struct wl1251 *wl, u8 bss_type, u16 beacon_interval,
        if (ret < 0)
                goto out;
 
-       wl1251_debug(DEBUG_CMD, "cmd join");
+       wl1251_debug(DEBUG_CMD, "cmd join%s %d %d%s",
+                    bss_type == BSS_TYPE_IBSS ? " ibss" : "",
+                    beacon_interval, dtim_interval,
+                    wait ? " wait" : "");
 
        /* Reverse order BSSID */
        bssid = (u8 *) &join->bssid_lsb;
index 47b82faac6975eb9b52fab7e24fb8e41b684892f..7014233454147021ea69179fc5c4ff38cc19ca09 100644 (file)
@@ -324,8 +324,8 @@ static void wl1251_filter_work(struct work_struct *work)
        if (ret < 0)
                goto out;
 
-       /* FIXME: replace the magic numbers with proper definitions */
-       ret = wl1251_cmd_join(wl, wl->bss_type, 100, 1, false);
+       ret = wl1251_cmd_join(wl, wl->bss_type, wl->beacon_int,
+                                  wl->dtim_period, false);
        if (ret < 0)
                goto out_sleep;
 
@@ -564,8 +564,8 @@ static int wl1251_op_config(struct ieee80211_hw *hw, u32 changed)
                goto out;
 
        if (channel != wl->channel) {
-               /* FIXME: use beacon interval provided by mac80211 */
-               ret = wl1251_cmd_join(wl, wl->bss_type, 100, 1, false);
+               ret = wl1251_cmd_join(wl, wl->bss_type, wl->beacon_int,
+                                          wl->dtim_period, false);
                if (ret < 0)
                        goto out_sleep;
 
@@ -1057,6 +1057,11 @@ static void wl1251_op_bss_info_changed(struct ieee80211_hw *hw,
 
        if (changed & BSS_CHANGED_ASSOC) {
                if (bss_conf->assoc) {
+                       wl->beacon_int = bss_conf->beacon_int;
+                       wl->dtim_period = bss_conf->dtim_period;
+
+                       /* FIXME: call join */
+
                        wl->aid = bss_conf->aid;
 
                        ret = wl1251_build_ps_poll(wl, wl->aid);
@@ -1074,6 +1079,10 @@ static void wl1251_op_bss_info_changed(struct ieee80211_hw *hw,
                                if (ret < 0)
                                        goto out_sleep;
                        }
+               } else {
+                       /* use defaults when not associated */
+                       wl->beacon_int = WL1251_DEFAULT_BEACON_INT;
+                       wl->dtim_period = WL1251_DEFAULT_DTIM_PERIOD;
                }
        }
        if (changed & BSS_CHANGED_ERP_SLOT) {
@@ -1113,7 +1122,9 @@ static void wl1251_op_bss_info_changed(struct ieee80211_hw *hw,
                        goto out;
 
                if (wl->bss_type != BSS_TYPE_IBSS) {
-                       ret = wl1251_cmd_join(wl, wl->bss_type, 100, 5, true);
+                       ret = wl1251_cmd_join(wl, wl->bss_type,
+                                             wl->beacon_int,
+                                             wl->dtim_period, true);
                        if (ret < 0)
                                goto out_sleep;
                        wl1251_warning("Set ctsprotect failed %d", ret);
@@ -1139,7 +1150,8 @@ static void wl1251_op_bss_info_changed(struct ieee80211_hw *hw,
                if (ret < 0)
                        goto out;
 
-               ret = wl1251_cmd_join(wl, wl->bss_type, 100, 1, false);
+               ret = wl1251_cmd_join(wl, wl->bss_type, wl->beacon_int,
+                                     wl->dtim_period, false);
 
                if (ret < 0)
                        goto out;
@@ -1329,6 +1341,8 @@ struct ieee80211_hw *wl1251_alloc_hw(void)
        wl->psm_requested = false;
        wl->tx_queue_stopped = false;
        wl->power_level = WL1251_DEFAULT_POWER_LEVEL;
+       wl->beacon_int = WL1251_DEFAULT_BEACON_INT;
+       wl->dtim_period = WL1251_DEFAULT_DTIM_PERIOD;
 
        for (i = 0; i < FW_TX_CMPLT_BLOCK_SIZE; i++)
                wl->tx_frames[i] = NULL;