ethernet: Convert mac address uses of 6 to ETH_ALEN
authorJoe Perches <joe@perches.com>
Thu, 1 Aug 2013 23:17:49 +0000 (16:17 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 2 Aug 2013 19:33:54 +0000 (12:33 -0700)
Use the normal #define to help grep find mac addresses
and ensure that addresses are aligned.

pasemi.h has an unaligned access to mac_addr, unchanged
for now.

Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Olof Johansson <olof@lixom.net> # pasemi_mac pieces
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/8390/ax88796.c
drivers/net/ethernet/amd/pcnet32.c
drivers/net/ethernet/broadcom/cnic_if.h
drivers/net/ethernet/dec/tulip/tulip_core.c
drivers/net/ethernet/i825xx/sun3_82586.h
drivers/net/ethernet/myricom/myri10ge/myri10ge.c
drivers/net/ethernet/nuvoton/w90p910_ether.c
drivers/net/ethernet/pasemi/pasemi_mac.c
drivers/net/ethernet/pasemi/pasemi_mac.h
drivers/net/ethernet/qlogic/netxen/netxen_nic_hw.c
drivers/net/ethernet/qlogic/qlge/qlge.h

index e1d26433d61921b3e0460421212006bcb5e0e9ee..b7232a9b7756cb48f8fd143b4f5a31f45757151a 100644 (file)
@@ -707,7 +707,7 @@ static int ax_init_dev(struct net_device *dev)
 
 #ifdef CONFIG_AX88796_93CX6
        if (ax->plat->flags & AXFLG_HAS_93CX6) {
-               unsigned char mac_addr[6];
+               unsigned char mac_addr[ETH_ALEN];
                struct eeprom_93cx6 eeprom;
 
                eeprom.data = ei_local;
@@ -719,7 +719,7 @@ static int ax_init_dev(struct net_device *dev)
                                       (__le16 __force *)mac_addr,
                                       sizeof(mac_addr) >> 1);
 
-               memcpy(dev->dev_addr, mac_addr, 6);
+               memcpy(dev->dev_addr, mac_addr, ETH_ALEN);
        }
 #endif
        if (ax->plat->wordlength == 2) {
index ed21307276430895fadc736b16f8539afe1713cf..2d8e28819779eed0388109791fe0126f5af763f8 100644 (file)
@@ -1521,7 +1521,7 @@ pcnet32_probe1(unsigned long ioaddr, int shared, struct pci_dev *pdev)
        char *chipname;
        struct net_device *dev;
        const struct pcnet32_access *a = NULL;
-       u8 promaddr[6];
+       u8 promaddr[ETH_ALEN];
        int ret = -ENODEV;
 
        /* reset the chip */
@@ -1665,10 +1665,10 @@ pcnet32_probe1(unsigned long ioaddr, int shared, struct pci_dev *pdev)
        }
 
        /* read PROM address and compare with CSR address */
-       for (i = 0; i < 6; i++)
+       for (i = 0; i < ETH_ALEN; i++)
                promaddr[i] = inb(ioaddr + i);
 
-       if (memcmp(promaddr, dev->dev_addr, 6) ||
+       if (memcmp(promaddr, dev->dev_addr, ETH_ALEN) ||
            !is_valid_ether_addr(dev->dev_addr)) {
                if (is_valid_ether_addr(promaddr)) {
                        if (pcnet32_debug & NETIF_MSG_PROBE) {
index 82a03120fcb1a7de37b063270438473dd9753923..95aff7642b853c58c281e8fd78c3e9e86a8aa254 100644 (file)
@@ -238,8 +238,8 @@ struct cnic_sock {
        u16     src_port;
        u16     dst_port;
        u16     vlan_id;
-       unsigned char old_ha[6];
-       unsigned char ha[6];
+       unsigned char old_ha[ETH_ALEN];
+       unsigned char ha[ETH_ALEN];
        u32     mtu;
        u32     cid;
        u32     l5_cid;
@@ -308,7 +308,7 @@ struct cnic_dev {
 #define CNIC_F_BNX2_CLASS      3
 #define CNIC_F_BNX2X_CLASS     4
        atomic_t        ref_count;
-       u8              mac_addr[6];
+       u8              mac_addr[ETH_ALEN];
 
        int             max_iscsi_conn;
        int             max_fcoe_conn;
index c94152f1c6be5396b511902a28478bae1c00e442..4e8cfa2ac803abb13c9c692ef971618cc26d664b 100644 (file)
@@ -1304,7 +1304,9 @@ static int tulip_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
 {
        struct tulip_private *tp;
        /* See note below on the multiport cards. */
-       static unsigned char last_phys_addr[6] = {0x00, 'L', 'i', 'n', 'u', 'x'};
+       static unsigned char last_phys_addr[ETH_ALEN] = {
+               0x00, 'L', 'i', 'n', 'u', 'x'
+       };
        static int last_irq;
        static int multiport_cnt;       /* For four-port boards w/one EEPROM */
        int i, irq;
@@ -1627,8 +1629,8 @@ static int tulip_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
                dev->dev_addr[i] = last_phys_addr[i] + 1;
 #if defined(CONFIG_SPARC)
                addr = of_get_property(dp, "local-mac-address", &len);
-               if (addr && len == 6)
-                       memcpy(dev->dev_addr, addr, 6);
+               if (addr && len == ETH_ALEN)
+                       memcpy(dev->dev_addr, addr, ETH_ALEN);
 #endif
 #if defined(__i386__) || defined(__x86_64__)   /* Patch up x86 BIOS bug. */
                if (last_irq)
index 93346f00486ba78ddf719a23de6ab71fd9058ea7..79aef681ac85db206d65f0a31944c7ee1a6a3374 100644 (file)
@@ -133,8 +133,8 @@ struct rfd_struct
   unsigned char  last;         /* Bit15,Last Frame on List / Bit14,suspend */
   unsigned short next;         /* linkoffset to next RFD */
   unsigned short rbd_offset;   /* pointeroffset to RBD-buffer */
-  unsigned char  dest[6];      /* ethernet-address, destination */
-  unsigned char  source[6];    /* ethernet-address, source */
+  unsigned char  dest[ETH_ALEN];       /* ethernet-address, destination */
+  unsigned char  source[ETH_ALEN];     /* ethernet-address, source */
   unsigned short length;       /* 802.3 frame-length */
   unsigned short zero_dummy;   /* dummy */
 };
index 967bae8b85c5f6c9bb0a720d97b3c0c17c940430..d4cdf4dc4bc41f54238848c681284521bce93994 100644 (file)
@@ -244,7 +244,7 @@ struct myri10ge_priv {
        int fw_ver_minor;
        int fw_ver_tiny;
        int adopted_rx_filter_bug;
-       u8 mac_addr[6];         /* eeprom mac address */
+       u8 mac_addr[ETH_ALEN];          /* eeprom mac address */
        unsigned long serial_number;
        int vendor_specific_offset;
        int fw_multicast_support;
index e88bdb1aa669c1f60c9b161f2b81fb10150c7777..dcfe58fa3b8a1abc9eb858a91a171db7a7bfcc3d 100644 (file)
@@ -922,7 +922,7 @@ static void __init get_mac_address(struct net_device *dev)
 {
        struct w90p910_ether *ether = netdev_priv(dev);
        struct platform_device *pdev;
-       char addr[6];
+       char addr[ETH_ALEN];
 
        pdev = ether->pdev;
 
@@ -934,7 +934,7 @@ static void __init get_mac_address(struct net_device *dev)
        addr[5] = 0xa8;
 
        if (is_valid_ether_addr(addr))
-               memcpy(dev->dev_addr, &addr, 0x06);
+               memcpy(dev->dev_addr, &addr, ETH_ALEN);
        else
                dev_err(&pdev->dev, "invalid mac address\n");
 }
index a5f0b5da614933c74ebbf005edba139b4bb74b88..f21ae7b6c766c3353e33faeab6fd32a14270e0cf 100644 (file)
@@ -191,7 +191,7 @@ static int pasemi_get_mac_addr(struct pasemi_mac *mac)
        struct device_node *dn = pci_device_to_OF_node(pdev);
        int len;
        const u8 *maddr;
-       u8 addr[6];
+       u8 addr[ETH_ALEN];
 
        if (!dn) {
                dev_dbg(&pdev->dev,
@@ -201,8 +201,8 @@ static int pasemi_get_mac_addr(struct pasemi_mac *mac)
 
        maddr = of_get_property(dn, "local-mac-address", &len);
 
-       if (maddr && len == 6) {
-               memcpy(mac->mac_addr, maddr, 6);
+       if (maddr && len == ETH_ALEN) {
+               memcpy(mac->mac_addr, maddr, ETH_ALEN);
                return 0;
        }
 
@@ -219,14 +219,15 @@ static int pasemi_get_mac_addr(struct pasemi_mac *mac)
                return -ENOENT;
        }
 
-       if (sscanf(maddr, "%hhx:%hhx:%hhx:%hhx:%hhx:%hhx", &addr[0],
-                  &addr[1], &addr[2], &addr[3], &addr[4], &addr[5]) != 6) {
+       if (sscanf(maddr, "%hhx:%hhx:%hhx:%hhx:%hhx:%hhx",
+                  &addr[0], &addr[1], &addr[2], &addr[3], &addr[4], &addr[5])
+           != ETH_ALEN) {
                dev_warn(&pdev->dev,
                         "can't parse mac address, not configuring\n");
                return -EINVAL;
        }
 
-       memcpy(mac->mac_addr, addr, 6);
+       memcpy(mac->mac_addr, addr, ETH_ALEN);
 
        return 0;
 }
index e2f4efa8ad46a5d79384a2afd6e441a1552f02a0..f2749d46c125f738ce38e406958015fc37246640 100644 (file)
@@ -83,7 +83,7 @@ struct pasemi_mac {
 #define MAC_TYPE_GMAC  1
 #define MAC_TYPE_XAUI  2
 
-       u8              mac_addr[6];
+       u8              mac_addr[ETH_ALEN];
 
        struct net_lro_mgr      lro_mgr;
        struct net_lro_desc     lro_desc[MAX_LRO_DESCRIPTORS];
index 9fbb1cdbfa4764b48d010f5ca43a851b33234e20..8375cbde996976047475b9c5e6c16f33fd365df4 100644 (file)
@@ -536,10 +536,10 @@ static void netxen_p2_nic_set_multi(struct net_device *netdev)
 {
        struct netxen_adapter *adapter = netdev_priv(netdev);
        struct netdev_hw_addr *ha;
-       u8 null_addr[6];
+       u8 null_addr[ETH_ALEN];
        int i;
 
-       memset(null_addr, 0, 6);
+       memset(null_addr, 0, ETH_ALEN);
 
        if (netdev->flags & IFF_PROMISC) {
 
index 7e8d68263963aa5928f5ef1f145ad27624c14ce2..89943377846699e1e2719be8b2da16b07b144d09 100644 (file)
@@ -2149,7 +2149,7 @@ struct ql_adapter {
        struct timer_list timer;
        atomic_t lb_count;
        /* Keep local copy of current mac address. */
-       char current_mac_addr[6];
+       char current_mac_addr[ETH_ALEN];
 };
 
 /*