b43: Fix suspend/resume
authorMichael Buesch <mb@bu3sch.de>
Wed, 23 Jan 2008 20:10:56 +0000 (21:10 +0100)
committerDavid S. Miller <davem@davemloft.net>
Fri, 1 Feb 2008 03:26:45 +0000 (19:26 -0800)
This fixes suspend/resume.

We must not overwrite the MAC addresses on resume. Otherwise
the card won't ACK any packets anymore.

Signed-off-by: Michael Buesch <mb@bu3sch.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/wireless/b43/main.c
drivers/net/wireless/b43/xmit.c

index f21355da84bd3f796f587cabd8650d1550063e5b..64c154d080d865fdde6a792e7ea5008af459a25b 100644 (file)
@@ -3532,8 +3532,6 @@ static int b43_wireless_core_init(struct b43_wldev *dev)
        b43_bluetooth_coext_enable(dev);
 
        ssb_bus_powerup(bus, 1);        /* Enable dynamic PCTL */
-       memset(wl->bssid, 0, ETH_ALEN);
-       memset(wl->mac_addr, 0, ETH_ALEN);
        b43_upload_card_macaddress(dev);
        b43_security_init(dev);
        b43_rng_init(wl);
@@ -3632,6 +3630,14 @@ static int b43_op_start(struct ieee80211_hw *hw)
        int err = 0;
        bool do_rfkill_exit = 0;
 
+       /* Kill all old instance specific information to make sure
+        * the card won't use it in the short timeframe between start
+        * and mac80211 reconfiguring it. */
+       memset(wl->bssid, 0, ETH_ALEN);
+       memset(wl->mac_addr, 0, ETH_ALEN);
+       wl->filter_flags = 0;
+       wl->radiotap_enabled = 0;
+
        /* First register RFkill.
         * LEDs that are registered later depend on it. */
        b43_rfkill_init(dev);
index 3fc53e8b4416092c26a2f2bbb7ead93b4ecda98b..7de2814d527e90f21fe783c28dc958be34cacbc8 100644 (file)
@@ -237,20 +237,23 @@ void b43_generate_txhdr(struct b43_wldev *dev,
 
                B43_WARN_ON(key_idx >= dev->max_nr_keys);
                key = &(dev->key[key_idx]);
-               B43_WARN_ON(!key->keyconf);
-
-               /* Hardware appends ICV. */
-               plcp_fragment_len += txctl->icv_len;
-
-               key_idx = b43_kidx_to_fw(dev, key_idx);
-               mac_ctl |= (key_idx << B43_TXH_MAC_KEYIDX_SHIFT) &
-                          B43_TXH_MAC_KEYIDX;
-               mac_ctl |= (key->algorithm << B43_TXH_MAC_KEYALG_SHIFT) &
-                          B43_TXH_MAC_KEYALG;
-               wlhdr_len = ieee80211_get_hdrlen(fctl);
-               iv_len = min((size_t) txctl->iv_len,
-                            ARRAY_SIZE(txhdr->iv));
-               memcpy(txhdr->iv, ((u8 *) wlhdr) + wlhdr_len, iv_len);
+
+               if (likely(key->keyconf)) {
+                       /* This key is valid. Use it for encryption. */
+
+                       /* Hardware appends ICV. */
+                       plcp_fragment_len += txctl->icv_len;
+
+                       key_idx = b43_kidx_to_fw(dev, key_idx);
+                       mac_ctl |= (key_idx << B43_TXH_MAC_KEYIDX_SHIFT) &
+                                  B43_TXH_MAC_KEYIDX;
+                       mac_ctl |= (key->algorithm << B43_TXH_MAC_KEYALG_SHIFT) &
+                                  B43_TXH_MAC_KEYALG;
+                       wlhdr_len = ieee80211_get_hdrlen(fctl);
+                       iv_len = min((size_t) txctl->iv_len,
+                                    ARRAY_SIZE(txhdr->iv));
+                       memcpy(txhdr->iv, ((u8 *) wlhdr) + wlhdr_len, iv_len);
+               }
        }
        if (b43_is_old_txhdr_format(dev)) {
                b43_generate_plcp_hdr((struct b43_plcp_hdr4 *)(&txhdr->old_format.plcp),