rtl818x_pci: Fix BSSID register written incorrectly
authorAndrea Merello <andrea.merello@gmail.com>
Mon, 30 Jun 2014 16:17:48 +0000 (18:17 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 1 Jul 2014 18:26:26 +0000 (14:26 -0400)
BSSID register was written with six byte-writes.
It seems that, similarly to what happens with MAC registers, they needs to be
written with one 16-bit and one 32-bit writes, otherwise the write does not work.

The byte write didn't work only on my rtl8185, while it worked on rtl8180 and
rtl8187se, BTW since there are probably a number of different ASIC revisions out
of there, I let the change to affect all cards.
It shouldn't hurt anyway.

Signed-off-by: Andrea Merello <andrea.merello@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/rtl818x/rtl8180/dev.c

index 1e2592918fc628f00f5d1dd4d68decdc141342a5..e2dcedee5e4143560208b86f42a32a1e1054b3cb 100644 (file)
@@ -1461,9 +1461,10 @@ static void rtl8180_bss_info_changed(struct ieee80211_hw *dev,
        vif_priv = (struct rtl8180_vif *)&vif->drv_priv;
 
        if (changed & BSS_CHANGED_BSSID) {
-               for (i = 0; i < ETH_ALEN; i++)
-                       rtl818x_iowrite8(priv, &priv->map->BSSID[i],
-                                        info->bssid[i]);
+               rtl818x_iowrite16(priv, (__le16 __iomem *)&priv->map->BSSID[0],
+                                 le16_to_cpu(*(__le16 *)info->bssid));
+               rtl818x_iowrite32(priv, (__le32 __iomem *)&priv->map->BSSID[2],
+                                 le32_to_cpu(*(__le32 *)(info->bssid + 2)));
 
                if (is_valid_ether_addr(info->bssid)) {
                        if (vif->type == NL80211_IFTYPE_ADHOC)