wireless: use eth_hw_addr_set() for dev->addr_len cases
authorJakub Kicinski <kuba@kernel.org>
Mon, 18 Oct 2021 23:50:09 +0000 (16:50 -0700)
committerKalle Valo <kvalo@codeaurora.org>
Wed, 20 Oct 2021 09:39:43 +0000 (12:39 +0300)
Convert all WiFi drivers from memcpy(... dev->addr_len)
to eth_hw_addr_set():

  @@
  expression dev, np;
  @@
  - memcpy(dev->dev_addr, np, dev->addr_len)
  + eth_hw_addr_set(dev, np)

Manually checked the netdevs are allocated with alloc_etherdev(),
so dev->addr_len must be equal to ETH_ALEN.

Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20211018235021.1279697-4-kuba@kernel.org
drivers/net/wireless/atmel/atmel.c
drivers/net/wireless/zydas/zd1201.c

index 3d782fabbfe6d504135b40f15c8b5fc314759606..8290cf881a1bd831ef21b6427692386c6daa2df0 100644 (file)
@@ -1296,7 +1296,7 @@ static int atmel_set_mac_address(struct net_device *dev, void *p)
 {
        struct sockaddr *addr = p;
 
-       memcpy (dev->dev_addr, addr->sa_data, dev->addr_len);
+       eth_hw_addr_set(dev, addr->sa_data);
        return atmel_open(dev);
 }
 
index 097805b55c59fe1026d7d43d82654963d62f5124..6b735fee7b187b73d873a5c2a9697652ad57353b 100644 (file)
@@ -857,7 +857,7 @@ static int zd1201_set_mac_address(struct net_device *dev, void *p)
            addr->sa_data, dev->addr_len, 1);
        if (err)
                return err;
-       memcpy(dev->dev_addr, addr->sa_data, dev->addr_len);
+       eth_hw_addr_set(dev, addr->sa_data);
 
        return zd1201_mac_reset(zd);
 }