mwl8k: remove duplicate local per-vif copy of ieee80211_bss_conf
authorLennert Buytenhek <buytenh@wantstofly.org>
Mon, 30 Nov 2009 17:33:09 +0000 (18:33 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 28 Dec 2009 21:31:28 +0000 (16:31 -0500)
Signed-off-by: Lennert Buytenhek <buytenh@marvell.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/mwl8k.c

index 6fa2bde5b618ab8733904eddb25cd4a07412d98b..97a95decf9a8217c20bb9ff1f9c96d00312133e4 100644 (file)
@@ -204,12 +204,11 @@ struct mwl8k_priv {
 
 /* Per interface specific private data */
 struct mwl8k_vif {
-       /* BSS config of AP or IBSS from mac80211*/
-       struct ieee80211_bss_conf bss_info;
+       /* Local MAC address.  */
+       u8 mac_addr[ETH_ALEN];
 
-       /* BSSID of AP or IBSS */
-       u8      bssid[ETH_ALEN];
-       u8      mac_addr[ETH_ALEN];
+       /* BSSID of AP.  */
+       u8 bssid[ETH_ALEN];
 
        /* Index into station database. Returned by UPDATE_STADB.  */
        u8      peer_id;
@@ -2123,7 +2122,6 @@ static int
 mwl8k_cmd_set_aid(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
 {
        struct mwl8k_vif *mv_vif = MWL8K_VIF(vif);
-       struct ieee80211_bss_conf *info = &mv_vif->bss_info;
        struct mwl8k_cmd_update_set_aid *cmd;
        u16 prot_mode;
        int rc;
@@ -2134,14 +2132,14 @@ mwl8k_cmd_set_aid(struct ieee80211_hw *hw, struct ieee80211_vif *vif)
 
        cmd->header.code = cpu_to_le16(MWL8K_CMD_SET_AID);
        cmd->header.length = cpu_to_le16(sizeof(*cmd));
-       cmd->aid = cpu_to_le16(info->aid);
+       cmd->aid = cpu_to_le16(vif->bss_conf.aid);
 
        memcpy(cmd->bssid, mv_vif->bssid, ETH_ALEN);
 
-       if (info->use_cts_prot) {
+       if (vif->bss_conf.use_cts_prot) {
                prot_mode = MWL8K_FRAME_PROT_11G;
        } else {
-               switch (info->ht_operation_mode &
+               switch (vif->bss_conf.ht_operation_mode &
                        IEEE80211_HT_OP_MODE_PROTECTION) {
                case IEEE80211_HT_OP_MODE_PROTECTION_20MHZ:
                        prot_mode = MWL8K_FRAME_PROT_11N_HT_40MHZ_ONLY;
@@ -2651,7 +2649,6 @@ static int mwl8k_cmd_update_stadb(struct ieee80211_hw *hw,
                struct ieee80211_vif *vif, __u32 action)
 {
        struct mwl8k_vif *mv_vif = MWL8K_VIF(vif);
-       struct ieee80211_bss_conf *info = &mv_vif->bss_info;
        struct mwl8k_cmd_update_stadb *cmd;
        struct peer_capability_info *peer_info;
        int rc;
@@ -2672,7 +2669,8 @@ static int mwl8k_cmd_update_stadb(struct ieee80211_hw *hw,
        case MWL8K_STA_DB_MODIFY_ENTRY:
                /* Build peer_info block */
                peer_info->peer_type = MWL8K_PEER_TYPE_ACCESSPOINT;
-               peer_info->basic_caps = cpu_to_le16(info->assoc_capability);
+               peer_info->basic_caps =
+                       cpu_to_le16(vif->bss_conf.assoc_capability);
                memcpy(peer_info->legacy_rates, mwl8k_rateids,
                       sizeof(mwl8k_rateids));
                peer_info->interop = 1;
@@ -2960,11 +2958,8 @@ static void mwl8k_bss_info_changed(struct ieee80211_hw *hw,
        if (rc)
                return;
 
-       if (info->assoc) {
-               memcpy(&mwl8k_vif->bss_info, info,
-                       sizeof(struct ieee80211_bss_conf));
-
-               memcpy(mwl8k_vif->bssid, info->bssid, ETH_ALEN);
+       if (vif->bss_conf.assoc) {
+               memcpy(mwl8k_vif->bssid, vif->bss_conf.bssid, ETH_ALEN);
 
                /* Install rates */
                rc = mwl8k_cmd_set_rate(hw, vif);
@@ -2978,12 +2973,13 @@ static void mwl8k_bss_info_changed(struct ieee80211_hw *hw,
                        goto out;
 
                /* Set radio preamble */
-               rc = mwl8k_set_radio_preamble(hw, info->use_short_preamble);
+               rc = mwl8k_set_radio_preamble(hw,
+                               vif->bss_conf.use_short_preamble);
                if (rc)
                        goto out;
 
                /* Set slot time */
-               rc = mwl8k_cmd_set_slot(hw, info->use_short_slot);
+               rc = mwl8k_cmd_set_slot(hw, vif->bss_conf.use_short_slot);
                if (rc)
                        goto out;
 
@@ -3006,8 +3002,6 @@ static void mwl8k_bss_info_changed(struct ieee80211_hw *hw,
                priv->capture_beacon = true;
        } else {
                rc = mwl8k_cmd_update_stadb(hw, vif, MWL8K_STA_DB_DEL_ENTRY);
-               memset(&mwl8k_vif->bss_info, 0,
-                       sizeof(struct ieee80211_bss_conf));
                memset(mwl8k_vif->bssid, 0, ETH_ALEN);
        }
 
@@ -3239,9 +3233,9 @@ static void mwl8k_finalize_join_worker(struct work_struct *work)
        struct mwl8k_priv *priv =
                container_of(work, struct mwl8k_priv, finalize_join_worker);
        struct sk_buff *skb = priv->beacon_skb;
-       u8 dtim = MWL8K_VIF(priv->vif)->bss_info.dtim_period;
 
-       mwl8k_cmd_finalize_join(priv->hw, skb->data, skb->len, dtim);
+       mwl8k_cmd_finalize_join(priv->hw, skb->data, skb->len,
+                               priv->vif->bss_conf.dtim_period);
        dev_kfree_skb(skb);
 
        priv->beacon_skb = NULL;