b43/legacy: fix beacon change processing

Process beacon change even if the BSSID doesn't
change at the same time. Also fix what I think
is a small locking error in b43legacy, there's
a spin_unlock_irqrestore that looks out of place.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>

1 files changed, 11 insertions, 13 deletions

diff --git a/drivers/net/wireless/b43/main.c b/drivers/net/wireless/b43/main.c
index 2615aaf7df6a..ec8516eadc4f 100644
--- a/drivers/net/wireless/b43/main.c
+++ b/drivers/net/wireless/b43/main.c
@@ -3553,27 +3553,25 @@ static void b43_op_bss_info_changed(struct ieee80211_hw *hw,
 
         B43_WARN_ON(wl->vif != vif);
 
+        spin_lock_irqsave(&wl->irq_lock, flags);
         if (changed & BSS_CHANGED_BSSID) {
-                spin_lock_irqsave(&wl->irq_lock, flags);
                 if (conf->bssid)
                         memcpy(wl->bssid, conf->bssid, ETH_ALEN);
                 else
                         memset(wl->bssid, 0, ETH_ALEN);
+        }
 
-                if (b43_status(dev) >= B43_STAT_INITIALIZED) {
-                        if (b43_is_mode(wl, NL80211_IFTYPE_AP) ||
-                            b43_is_mode(wl, NL80211_IFTYPE_MESH_POINT)) {
-                                B43_WARN_ON(vif->type != wl->if_type);
-                                if (changed & BSS_CHANGED_BEACON)
-                                        b43_update_templates(wl);
-                        } else if (b43_is_mode(wl, NL80211_IFTYPE_ADHOC)) {
-                                if (changed & BSS_CHANGED_BEACON)
-                                        b43_update_templates(wl);
-                        }
+        if (b43_status(dev) >= B43_STAT_INITIALIZED) {
+                if (changed & BSS_CHANGED_BEACON &&
+                    (b43_is_mode(wl, NL80211_IFTYPE_AP) ||
+                     b43_is_mode(wl, NL80211_IFTYPE_MESH_POINT) ||
+                     b43_is_mode(wl, NL80211_IFTYPE_ADHOC)))
+                        b43_update_templates(wl);
+
+                if (changed & BSS_CHANGED_BSSID)
                         b43_write_mac_bssid_templates(dev);
-                }
-                spin_unlock_irqrestore(&wl->irq_lock, flags);
         }
+        spin_unlock_irqrestore(&wl->irq_lock, flags);
 
         b43_mac_suspend(dev);