hamradio: use dev_addr_set() for setting device address
authorJakub Kicinski <kuba@kernel.org>
Tue, 12 Oct 2021 16:06:33 +0000 (09:06 -0700)
committerJakub Kicinski <kuba@kernel.org>
Wed, 13 Oct 2021 16:41:37 +0000 (09:41 -0700)
Use dev_addr_set() instead of writing to netdev->dev_addr
directly in hamradio drivers.

Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/hamradio/6pack.c
drivers/net/hamradio/baycom_epp.c
drivers/net/hamradio/bpqether.c
drivers/net/hamradio/dmascc.c
drivers/net/hamradio/hdlcdrv.c
drivers/net/hamradio/mkiss.c
drivers/net/hamradio/scc.c
drivers/net/hamradio/yam.c

index 6192244b304ab976e0b17f7fd3eb568585f16a54..f4e8793e995d7a5bb06b66dae9877074ea205f0b 100644 (file)
@@ -288,7 +288,7 @@ static int sp_set_mac_address(struct net_device *dev, void *addr)
 
        netif_tx_lock_bh(dev);
        netif_addr_lock(dev);
-       memcpy(dev->dev_addr, &sa->sax25_call, AX25_ADDR_LEN);
+       __dev_addr_set(dev, &sa->sax25_call, AX25_ADDR_LEN);
        netif_addr_unlock(dev);
        netif_tx_unlock_bh(dev);
 
@@ -317,7 +317,7 @@ static void sp_setup(struct net_device *dev)
 
        /* Only activated in AX.25 mode */
        memcpy(dev->broadcast, &ax25_bcast, AX25_ADDR_LEN);
-       memcpy(dev->dev_addr, &ax25_defaddr, AX25_ADDR_LEN);
+       dev_addr_set(dev, (u8 *)&ax25_defaddr);
 
        dev->flags              = 0;
 }
@@ -726,7 +726,7 @@ static int sixpack_ioctl(struct tty_struct *tty, struct file *file,
                        }
 
                        netif_tx_lock_bh(dev);
-                       memcpy(dev->dev_addr, &addr, AX25_ADDR_LEN);
+                       __dev_addr_set(dev, &addr, AX25_ADDR_LEN);
                        netif_tx_unlock_bh(dev);
                        err = 0;
                        break;
index f6428ff780abcd0d8d19f3eb705ca0719ffb19f5..62da837cc707639048b139b89299e7e6fd2921c6 100644 (file)
@@ -1159,7 +1159,7 @@ static void baycom_probe(struct net_device *dev)
        dev->mtu = AX25_DEF_PACLEN;        /* eth_mtu is the default */
        dev->addr_len = AX25_ADDR_LEN;     /* sizeof an ax.25 address */
        memcpy(dev->broadcast, &ax25_bcast, AX25_ADDR_LEN);
-       memcpy(dev->dev_addr, &null_ax25_address, AX25_ADDR_LEN);
+       dev_addr_set(dev, (u8 *)&null_ax25_address);
        dev->tx_queue_len = 16;
 
        /* New style flags */
index 1ac816164a9bb91d807f3db1b6c50eb86a34e546..30af0081e2bef5745d9a21320c2b95bcd218081a 100644 (file)
@@ -457,9 +457,6 @@ static void bpq_setup(struct net_device *dev)
        dev->netdev_ops      = &bpq_netdev_ops;
        dev->needs_free_netdev = true;
 
-       memcpy(dev->broadcast, &ax25_bcast, AX25_ADDR_LEN);
-       memcpy(dev->dev_addr,  &ax25_defaddr, AX25_ADDR_LEN);
-
        dev->flags      = 0;
        dev->features   = NETIF_F_LLTX; /* Allow recursion */
 
@@ -472,6 +469,8 @@ static void bpq_setup(struct net_device *dev)
        dev->mtu             = AX25_DEF_PACLEN;
        dev->addr_len        = AX25_ADDR_LEN;
 
+       memcpy(dev->broadcast, &ax25_bcast, AX25_ADDR_LEN);
+       dev_addr_set(dev, (u8 *)&ax25_defaddr);
 }
 
 /*
index a4ba1b79d2d04f4c73f5a2ce055602eedc79baef..7e527499d3ad6a6844ee77a80710014b37d075e4 100644 (file)
@@ -426,7 +426,7 @@ static void __init dev_setup(struct net_device *dev)
        dev->addr_len = AX25_ADDR_LEN;
        dev->tx_queue_len = 64;
        memcpy(dev->broadcast, &ax25_bcast, AX25_ADDR_LEN);
-       memcpy(dev->dev_addr, &ax25_defaddr, AX25_ADDR_LEN);
+       dev_addr_set(dev, (u8 *)&ax25_defaddr);
 }
 
 static const struct net_device_ops scc_netdev_ops = {
index ab5327acec220fe78e627cfab527076ec880b370..b0edb91bb10aa77cde35abec50863251d171c516 100644 (file)
@@ -675,7 +675,7 @@ static void hdlcdrv_setup(struct net_device *dev)
        dev->mtu = AX25_DEF_PACLEN;        /* eth_mtu is the default */
        dev->addr_len = AX25_ADDR_LEN;     /* sizeof an ax.25 address */
        memcpy(dev->broadcast, &ax25_bcast, AX25_ADDR_LEN);
-       memcpy(dev->dev_addr, &ax25_defaddr, AX25_ADDR_LEN);
+       dev_addr_set(dev, (u8 *)&ax25_defaddr);
        dev->tx_queue_len = 16;
 }
 
index 8666110bec555c4eff5d37b0589c3d7e172cb035..867252a0247b1019a2cec072830f85c689ea06c4 100644 (file)
@@ -344,7 +344,7 @@ static int ax_set_mac_address(struct net_device *dev, void *addr)
 
        netif_tx_lock_bh(dev);
        netif_addr_lock(dev);
-       memcpy(dev->dev_addr, &sa->sax25_call, AX25_ADDR_LEN);
+       __dev_addr_set(dev, &sa->sax25_call, AX25_ADDR_LEN);
        netif_addr_unlock(dev);
        netif_tx_unlock_bh(dev);
 
@@ -647,7 +647,7 @@ static void ax_setup(struct net_device *dev)
 
 
        memcpy(dev->broadcast, &ax25_bcast, AX25_ADDR_LEN);
-       memcpy(dev->dev_addr,  &ax25_defaddr,  AX25_ADDR_LEN);
+       dev_addr_set(dev, (u8 *)&ax25_defaddr);
 
        dev->flags      = IFF_BROADCAST | IFF_MULTICAST;
 }
@@ -850,7 +850,7 @@ static int mkiss_ioctl(struct tty_struct *tty, struct file *file,
                }
 
                netif_tx_lock_bh(dev);
-               memcpy(dev->dev_addr, addr, AX25_ADDR_LEN);
+               __dev_addr_set(dev, addr, AX25_ADDR_LEN);
                netif_tx_unlock_bh(dev);
 
                err = 0;
index 9b745d09d327a4da330030520a4b9d289fa2cb7c..3d59dac063ac430d27af9919b649b3f4501f55f4 100644 (file)
@@ -1563,9 +1563,6 @@ static void scc_net_setup(struct net_device *dev)
        dev->netdev_ops      = &scc_netdev_ops;
        dev->header_ops      = &ax25_header_ops;
 
-       memcpy(dev->broadcast, &ax25_bcast,  AX25_ADDR_LEN);
-       memcpy(dev->dev_addr,  &ax25_defaddr, AX25_ADDR_LEN);
        dev->flags      = 0;
 
        dev->type = ARPHRD_AX25;
@@ -1573,6 +1570,8 @@ static void scc_net_setup(struct net_device *dev)
        dev->mtu = AX25_DEF_PACLEN;
        dev->addr_len = AX25_ADDR_LEN;
 
+       memcpy(dev->broadcast, &ax25_bcast,  AX25_ADDR_LEN);
+       dev_addr_set(dev, (u8 *)&ax25_defaddr);
 }
 
 /* ----> open network device <---- */
index e557ccf98bcfd1b41a9309ef88741e9ba63be4be..6376b8485976cf454ebf9e8a4939c708f3116bd1 100644 (file)
@@ -1107,7 +1107,7 @@ static void yam_setup(struct net_device *dev)
        dev->mtu = AX25_MTU;
        dev->addr_len = AX25_ADDR_LEN;
        memcpy(dev->broadcast, &ax25_bcast, AX25_ADDR_LEN);
-       memcpy(dev->dev_addr, &ax25_defaddr, AX25_ADDR_LEN);
+       dev_addr_set(dev, (u8 *)&ax25_defaddr);
 }
 
 static int __init yam_init_driver(void)