Use ETH_ALEN
Subsystems (8)
cover Use ETH_ALEN
Remove other #defines and uses in favor of ETH_ALEN
Joe Perches (5):
ethernet: Convert MAC_ADDR_LEN uses to ETH_ALEN
ethernet: Convert ETHER_ADDR_LEN uses to ETH_ALEN
bna: Convert MAC_ADDRLEN uses to ETH_ALEN
amd8111e: Convert ETH_ADDR_LEN uses to ETH_ALEN
ucc_geth: Convert ENET_NUM_OCTETS_PER_ADDRESS uses to ETH_ALEN
drivers/net/ethernet/8390/8390.h | 2 - drivers/net/ethernet/8390/apne.c | 2 +- drivers/net/ethernet/8390/ax88796.c | 5 +- drivers/net/ethernet/8390/es3210.c | 2 +- drivers/net/ethernet/8390/hp-plus.c | 2 +- drivers/net/ethernet/8390/hp.c | 2 +- drivers/net/ethernet/8390/hydra.c | 2 +- drivers/net/ethernet/8390/lne390.c | 4 +- drivers/net/ethernet/8390/ne-h8300.c | 2 +- drivers/net/ethernet/8390/ne.c | 4 +- drivers/net/ethernet/8390/ne2.c | 2 +- drivers/net/ethernet/8390/ne3210.c | 2 +- drivers/net/ethernet/8390/stnic.c | 2 +- drivers/net/ethernet/8390/zorro8390.c | 2 +- drivers/net/ethernet/amd/amd8111e.c | 6 +- drivers/net/ethernet/amd/amd8111e.h | 1 - drivers/net/ethernet/amd/nmclan_cs.c | 12 ++--- drivers/net/ethernet/broadcom/sb1250-mac.c | 3 +- drivers/net/ethernet/brocade/bna/cna.h | 3 +- drivers/net/ethernet/freescale/gianfar.c | 12 +++--- drivers/net/ethernet/freescale/gianfar.h | 3 - drivers/net/ethernet/freescale/ucc_geth.c | 2 +- drivers/net/ethernet/freescale/ucc_geth.h | 6 +- drivers/net/ethernet/micrel/ksz884x.c | 59 +++++++++++++-------------- drivers/net/ethernet/realtek/r8169.c | 5 +-- drivers/net/ethernet/sis/sis190.c | 8 +-- 26 files changed, 68 insertions(+), 87 deletions(-)
Signature · 1 line
1.7.6.405.gc1be0
PATCH 1/5 ethernet: Convert MAC_ADDR_LEN uses to ETH_ALEN
Reduce the number of #defines, use the normal #define from if_ether.h
--- drivers/net/ethernet/freescale/gianfar.c | 12 +++--- drivers/net/ethernet/freescale/gianfar.h | 3 -- drivers/net/ethernet/micrel/ksz884x.c | 59 ++++++++++++++---------------- drivers/net/ethernet/realtek/r8169.c | 5 +-- drivers/net/ethernet/sis/sis190.c | 8 ++--- 5 files changed, 38 insertions(+), 49 deletions(-)
drivers/net/ethernet/freescale/gianfar.c +6 -6
@@ -734,7 +734,7 @@ static int gfar_of_init(struct platform_device *ofdev, struct net_device **pdev) mac_addr =of_get_mac_address (np);if (mac_addr) -memcpy (dev->dev_addr, mac_addr,MAC_ADDR_LEN ); +memcpy (dev->dev_addr, mac_addr,ETH_ALEN );if (model && !strcasecmp (model,"TSEC" )) priv->device_flags =
@@ -3114,7 +3114,7 @@ static void gfar_set_multi(struct net_device *dev)static void gfar_clear_exact_match (struct net_device *dev) {int idx; -static const u8 zero_arr[MAC_ADDR_LEN ] = {0 ,0 ,0 ,0 ,0 ,0 }; +static const u8 zero_arr[ETH_ALEN ] = {0 ,0 ,0 ,0 ,0 ,0 };for (idx =1 ;idx <GFAR_EM_NUM +1 ;idx++)gfar_set_mac_for_addr (dev, idx, zero_arr);
@@ -3137,7 +3137,7 @@ static void gfar_set_hash_for_addr(struct net_device *dev, u8 *addr) {u32 tempval;struct gfar_private *priv =netdev_priv (dev); -u32 result =ether_crc (MAC_ADDR_LEN , addr); +u32 result =ether_crc (ETH_ALEN , addr);int width = priv->hash_width;u8 whichbit = (result >> (32 - width)) &0x1f ;u8 whichreg = result >> (32 - width +5 );
@@ -3158,7 +3158,7 @@ static void gfar_set_mac_for_addr(struct net_device *dev, int num,struct gfar_private *priv =netdev_priv (dev);struct gfar __iomem *regs = priv->gfargrp[0 ].regs;int idx; -char tmpbuf[MAC_ADDR_LEN ]; +char tmpbuf[ETH_ALEN ];u32 tempval;u32 __iomem *macptr = ®s->macstnaddr1;
@@ -3166,8 +3166,8 @@ static void gfar_set_mac_for_addr(struct net_device *dev, int num,/* Now copy it into the mac registers backwards, cuz */ /* little endian is silly */ -for (idx =0 ; idx <MAC_ADDR_LEN ; idx++) - tmpbuf[MAC_ADDR_LEN -1 - idx] = addr[idx]; +for (idx =0 ; idx <ETH_ALEN ; idx++) + tmpbuf[ETH_ALEN -1 - idx] = addr[idx];gfar_write (macptr, *((u32 *) (tmpbuf)));
drivers/net/ethernet/freescale/gianfar.h +0 -3
@@ -74,9 +74,6 @@ struct ethtool_rx_list {* will be the next highest multiple of 512 bytes. */ #define INCREMENTAL_BUFFER_SIZE 512 - -#define MAC_ADDR_LEN 6 -#define PHY_INIT_TIMEOUT 100000#define GFAR_PHY_CHANGE_TIME 2
drivers/net/ethernet/micrel/ksz884x.c +28 -31
174 lines not on this page · raw message
drivers/net/ethernet/realtek/r8169.c +1 -4
@@ -69,9 +69,6 @@The RTL chips use a 64 element hash table based on the Ethernet CRC. */ static const int multicast_filter_limit =32 ; -/* MAC address length */ -#define MAC_ADDR_LEN 6 -#define MAX_READ_REQUEST_SHIFT 12#define TX_DMA_BURST 6/* Maximum PCI burst, '6' is 1024 */ #define SafeMtu 0x1c20/* ... actually life sucks beyond ~7k */
@@ -4099,7 +4096,7 @@ rtl8169_init_one(struct pci_dev *pdev, const struct pci_device_id *ent)spin_lock_init (&tp->lock);/* Get MAC address */ -for (i =0 ; i <MAC_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++) dev->dev_addr[i] =RTL_R8 (MAC0 + i);memcpy (dev->perm_addr, dev->dev_addr, dev->addr_len);
drivers/net/ethernet/sis/sis190.c +3 -5
@@ -47,8 +47,6 @@#define sis190_rx_skb netif_rx#define sis190_rx_quota (count, quota) count -#define MAC_ADDR_LEN 6 -#define NUM_TX_DESC 64/* [8..1024] */ #define NUM_RX_DESC 64/* [8..8192] */ #define TX_RING_BYTES (NUM_TX_DESC * sizeof(struct TxDesc))
@@ -1601,7 +1599,7 @@ static int __devinit sis190_get_mac_addr_from_eeprom(struct pci_dev *pdev, }/* Get MAC address from EEPROM */ -for (i =0 ; i <MAC_ADDR_LEN /2 ; i++) { +for (i =0 ; i <ETH_ALEN /2 ; i++) {u16 w =sis190_read_eeprom (ioaddr, EEPROMMACAddr + i); ((__le16 *)dev->dev_addr)[i] =cpu_to_le16 (w);
@@ -1653,7 +1651,7 @@ static int __devinit sis190_get_mac_addr_from_apc(struct pci_dev *pdev,udelay (50 );pci_read_config_byte (isa_bridge,0x48 , ®); -for (i =0 ; i <MAC_ADDR_LEN ; i++) { +for (i =0 ; i <ETH_ALEN ; i++) {outb (0x9 + i,0x78 ); dev->dev_addr[i] =inb (0x79 ); }
@@ -1692,7 +1690,7 @@ static inline void sis190_init_rxfilter(struct net_device *dev)*/ SIS_W16 (RxMacControl, ctl & ~0x0f00 ); -for (i =0 ; i <MAC_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++)SIS_W8 (RxMacAddr + i, dev->dev_addr[i]);SIS_W16 (RxMacControl, ctl);
Signature · 1 line
1.7.6.405.gc1be0
PATCH 2/5 ethernet: Convert ETHER_ADDR_LEN uses to ETH_ALEN
In reply to Joe PerchesReduce the number of #defines, use the normal #define from if_ether.h
--- drivers/net/ethernet/8390/8390.h | 2 -- drivers/net/ethernet/8390/apne.c | 2 +- drivers/net/ethernet/8390/ax88796.c | 5 ++--- drivers/net/ethernet/8390/es3210.c | 2 +- drivers/net/ethernet/8390/hp-plus.c | 2 +- drivers/net/ethernet/8390/hp.c | 2 +- drivers/net/ethernet/8390/hydra.c | 2 +- drivers/net/ethernet/8390/lne390.c | 4 ++-- drivers/net/ethernet/8390/ne-h8300.c | 2 +- drivers/net/ethernet/8390/ne.c | 4 ++-- drivers/net/ethernet/8390/ne2.c | 2 +- drivers/net/ethernet/8390/ne3210.c | 2 +- drivers/net/ethernet/8390/stnic.c | 2 +- drivers/net/ethernet/8390/zorro8390.c | 2 +- drivers/net/ethernet/amd/nmclan_cs.c | 12 +++++------- drivers/net/ethernet/broadcom/sb1250-mac.c | 3 +-- 16 files changed, 22 insertions(+), 28 deletions(-)
drivers/net/ethernet/8390/8390.h +0 -2
@@ -14,8 +14,6 @@#define TX_PAGES 12/* Two Tx slots */ -#define ETHER_ADDR_LEN 6 -/* The 8390 specific per-packet-header format. */ struct e8390_pkt_hdr {unsigned char status;/* status */
drivers/net/ethernet/8390/apne.c +1 -1
@@ -318,7 +318,7 @@ static int __init apne_probe1(struct net_device *dev, int ioaddr) i =request_irq (dev->irq, apne_interrupt,IRQF_SHARED ,DRV_NAME , dev);if (i)return i; -for (i =0 ; i <ETHER_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++) dev->dev_addr[i] = SA_prom[i];printk (" %pM\n" , dev->dev_addr);
drivers/net/ethernet/8390/ax88796.c +2 -3
@@ -735,15 +735,14 @@ static int ax_init_dev(struct net_device *dev)if (ax->plat->flags &AXFLG_MAC_FROMDEV ) {ei_outb (E8390_NODMA +E8390_PAGE1 +E8390_STOP , ei_local->mem +E8390_CMD );/* 0x61 */ -for (i =0 ; i <ETHER_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++) dev->dev_addr[i] =ei_inb (ioaddr +EN1_PHYS_SHIFT (i)); }if ((ax->plat->flags &AXFLG_MAC_FROMPLATFORM ) && ax->plat->mac_addr) -memcpy (dev->dev_addr, ax->plat->mac_addr, -ETHER_ADDR_LEN ); +memcpy (dev->dev_addr, ax->plat->mac_addr,ETH_ALEN );ax_reset_8390 (dev);
drivers/net/ethernet/8390/es3210.c +1 -1
@@ -195,7 +195,7 @@ static int __init es_probe1(struct net_device *dev, int ioaddr) goto out; } -for (i =0 ; i <ETHER_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++) dev->dev_addr[i] =inb (ioaddr +ES_SA_PROM + i);/* Check the Racal vendor ID as well. */
drivers/net/ethernet/8390/hp-plus.c +1 -1
@@ -202,7 +202,7 @@ static int __init hpp_probe1(struct net_device *dev, int ioaddr)/* Retrieve and checksum the station address. */ outw (MAC_Page, ioaddr +HP_PAGING ); -for (i =0 ; i <ETHER_ADDR_LEN ; i++) { +for (i =0 ; i <ETH_ALEN ; i++) {unsigned char inval =inb (ioaddr +8 + i); dev->dev_addr[i] = inval; checksum += inval;
drivers/net/ethernet/8390/hp.c +1 -1
@@ -156,7 +156,7 @@ static int __init hp_probe1(struct net_device *dev, int ioaddr)printk ("%s: %s (ID %02x) at %#3x," , dev->name, name, board_id, ioaddr); -for (i =0 ; i <ETHER_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++) dev->dev_addr[i] =inb (ioaddr + i);printk (" %pM" , dev->dev_addr);
drivers/net/ethernet/8390/hydra.c +1 -1
@@ -129,7 +129,7 @@ static int __devinit hydra_init(struct zorro_dev *z)if (!dev)return -ENOMEM ; -for (j =0 ; j <ETHER_ADDR_LEN ; j++) +for (j =0 ; j <ETH_ALEN ; j++) dev->dev_addr[j] = *((u8 *)(board +HYDRA_ADDRPROM +2 *j));/* We must set the 8390 for word mode. */
drivers/net/ethernet/8390/lne390.c +2 -2
@@ -191,14 +191,14 @@ static int __init lne390_probe1(struct net_device *dev, int ioaddr) || inb(ioaddr + LNE390_SA_PROM + 1) != LNE390_ADDR1 || inb(ioaddr + LNE390_SA_PROM + 2) != LNE390_ADDR2 ) {printk ("lne390.c: card not found" ); -for (i =0 ; i <ETHER_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++)printk (" %02x" ,inb (ioaddr +LNE390_SA_PROM + i));printk (" (invalid prefix).\n" );return -ENODEV ; }#endif -for (i =0 ; i <ETHER_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++) dev->dev_addr[i] =inb (ioaddr +LNE390_SA_PROM + i);printk ("lne390.c: LNE390%X in EISA slot %d, address %pM.\n" ,0xa +revision, ioaddr/0x1000 , dev->dev_addr);
drivers/net/ethernet/8390/ne-h8300.c +1 -1
@@ -312,7 +312,7 @@ static int __init ne_probe1(struct net_device *dev, int ioaddr) dev->base_addr = ioaddr; -for (i =0 ; i <ETHER_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++) dev->dev_addr[i] = SA_prom[i];printk (" %pM\n" , dev->dev_addr);
drivers/net/ethernet/8390/ne.c +2 -2
@@ -503,12 +503,12 @@ static int __init ne_probe1(struct net_device *dev, unsigned long ioaddr)#ifdef CONFIG_PLAT_MAPPI outb_p (E8390_NODMA +E8390_PAGE1 +E8390_STOP , ioaddr +E8390_CMD );/* 0x61 */ -for (i =0 ; i <ETHER_ADDR_LEN ; i++) { +for (i =0 ; i <ETH_ALEN ; i++) { dev->dev_addr[i] = SA_prom[i] =inb_p (ioaddr +EN1_PHYS_SHIFT (i)); }#else -for (i =0 ; i <ETHER_ADDR_LEN ; i++) { +for (i =0 ; i <ETH_ALEN ; i++) { dev->dev_addr[i] = SA_prom[i]; }#endif
drivers/net/ethernet/8390/ne2.c +1 -1
@@ -460,7 +460,7 @@ static int __init ne2_probe1(struct net_device *dev, int slot) dev->base_addr = base_addr; -for (i =0 ; i <ETHER_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++) dev->dev_addr[i] = SA_prom[i];printk (" %pM\n" , dev->dev_addr);
drivers/net/ethernet/8390/ne3210.c +1 -1
@@ -125,7 +125,7 @@ static int __init ne3210_eisa_probe (struct device *device)#endif port_index =inb (ioaddr +NE3210_CFG2 ) >>6 ; -for (i =0 ; i <ETHER_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++) dev->dev_addr[i] =inb (ioaddr +NE3210_SA_PROM + i);printk ("ne3210.c: NE3210 in EISA slot %d, media: %s, addr: %pM.\n" , edev->slot, ifmap[port_index], dev->dev_addr);
drivers/net/ethernet/8390/stnic.c +1 -1
@@ -114,7 +114,7 @@ static int __init stnic_probe(void)#ifdef CONFIG_SH_STANDARD_BIOS sh_bios_get_node_addr (stnic_eadr);#endif -for (i =0 ; i <ETHER_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++) dev->dev_addr[i] = stnic_eadr[i];/* Set the base address to point to the NIC, not the "real" base! */
drivers/net/ethernet/8390/zorro8390.c +1 -1
@@ -365,7 +365,7 @@ static int __devinit zorro8390_init(struct net_device *dev,if (i)return i; -for (i =0 ; i <ETHER_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++) dev->dev_addr[i] = SA_prom[i];pr_debug ("Found ethernet address: %pM\n" , dev->dev_addr);
drivers/net/ethernet/amd/nmclan_cs.c +5 -7
@@ -160,8 +160,6 @@ Include FilesDefines ---------------------------------------------------------------------------- */ -#define ETHER_ADDR_LEN ETH_ALEN -/* 6 bytes in an Ethernet Address */ #define MACE_LADRF_LEN 8/* 8 bytes in Logical Address Filter */
@@ -600,7 +598,7 @@ static int mace_init(mace_private *lp, unsigned int ioaddr, char *enet_addr) } }/* Set PADR register */ -for (i =0 ; i <ETHER_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++)mace_write (lp, ioaddr,MACE_PADR , enet_addr[i]);/* MAC Configuration Control Register should be written last */
@@ -639,11 +637,11 @@ static int nmclan_config(struct pcmcia_device *link)/* Read the ethernet address from the CIS. */ len =pcmcia_get_tuple (link,0x80 , &buf); -if (!buf || len <ETHER_ADDR_LEN ) { +if (!buf || len <ETH_ALEN ) {kfree (buf); goto failed; } -memcpy (dev->dev_addr, buf,ETHER_ADDR_LEN ); +memcpy (dev->dev_addr, buf,ETH_ALEN );kfree (buf);/* Verify configuration by reading the MACE ID. */
@@ -1420,7 +1418,7 @@ Outputstatic void set_multicast_list (struct net_device *dev) {mace_private *lp =netdev_priv (dev); -int adr[ETHER_ADDR_LEN ] = {0 };/* Ethernet address */ +int adr[ETH_ALEN ] = {0 };/* Ethernet address */ struct netdev_hw_addr *ha;#ifdef PCMCIA_DEBUG
@@ -1442,7 +1440,7 @@ static void set_multicast_list(struct net_device *dev)/* Calculate multicast logical address filter */ memset (lp->multicast_ladrf,0 ,MACE_LADRF_LEN );netdev_for_each_mc_addr (ha, dev) { -memcpy (adr, ha->addr,ETHER_ADDR_LEN ); +memcpy (adr, ha->addr,ETH_ALEN );BuildLAF (lp->multicast_ladrf, adr); } }
drivers/net/ethernet/broadcom/sb1250-mac.c +1 -2
@@ -163,7 +163,6 @@ enum sbmac_state {#define SBMAC_MAX_TXDESCR 256#define SBMAC_MAX_RXDESCR 256 -#define ETHER_ADDR_LEN 6#define ENET_PACKET_SIZE 1518/*#define ENET_PACKET_SIZE 9216 */
@@ -266,7 +265,7 @@ struct sbmac_softc {int sbm_pause;/* current pause setting */ int sbm_link;/* current link state */ -unsigned char sbm_hwaddr[ETHER_ADDR_LEN ]; +unsigned char sbm_hwaddr[ETH_ALEN ];struct sbmacdma sbm_txdma;/* only channel 0 for now */ struct sbmacdma sbm_rxdma;
Signature · 1 line
1.7.6.405.gc1be0
PATCH 3/5 bna: Convert MAC_ADDRLEN uses to ETH_ALEN
In reply to Joe PerchesReduce the number of #defines, use the normal #define from if_ether.h
--- drivers/net/ethernet/brocade/bna/cna.h | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-)
drivers/net/ethernet/brocade/bna/cna.h +1 -2
@@ -43,8 +43,7 @@ extern char bfa_version[];#pragma pack(1) -#define MAC_ADDRLEN (6) -typedef struct mac {u8 mac[MAC_ADDRLEN ]; }mac_t ; +typedef struct mac {u8 mac[ETH_ALEN ]; }mac_t ;#pragma pack()
Signature · 1 line
1.7.6.405.gc1be0
RE: [PATCH net-next 3/5] bna: Convert MAC_ADDRLEN uses to ETH_ALEN
Quoted 4 lines
From: Joe Perches [mailto:[email protected]]
Sent: Wednesday, November 16, 2011 11:38 AMReduce the number of #defines, use the normal #define from
if_ether.h
Patch looks fine.
Thanks,
Rasesh
PATCH 4/5 amd8111e: Convert ETH_ADDR_LEN uses to ETH_ALEN
In reply to Joe PerchesReduce the number of #defines, use the normal #define from if_ether.h
--- drivers/net/ethernet/amd/amd8111e.c | 6 +++--- drivers/net/ethernet/amd/amd8111e.h | 1 - 2 files changed, 3 insertions(+), 4 deletions(-)
drivers/net/ethernet/amd/amd8111e.c +3 -3
@@ -499,7 +499,7 @@ static int amd8111e_restart(struct net_device *dev)writel (VAL0 |APAD_XMT |REX_RTRY , mmio +CMD2 );/* Setting the MAC address to the device */ -for (i =0 ; i <ETH_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++)writeb ( dev->dev_addr[i], mmio +PADR + i );/* Enable interrupt coalesce */
@@ -1550,7 +1550,7 @@ static int amd8111e_set_mac_address(struct net_device *dev, void *p)memcpy (dev->dev_addr, addr->sa_data, dev->addr_len);spin_lock_irq (&lp->lock);/* Setting the MAC address to the device */ -for (i =0 ; i <ETH_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++)writeb ( dev->dev_addr[i], lp->mmio +PADR + i );spin_unlock_irq (&lp->lock);
@@ -1886,7 +1886,7 @@ static int __devinit amd8111e_probe_one(struct pci_dev *pdev, }/* Initializing MAC address */ -for (i =0 ; i <ETH_ADDR_LEN ; i++) +for (i =0 ; i <ETH_ALEN ; i++) dev->dev_addr[i] =readb (lp->mmio +PADR + i);/* Setting user defined parametrs */
drivers/net/ethernet/amd/amd8111e.h +0 -1
@@ -586,7 +586,6 @@ typedef enum {#define PKT_BUFF_SZ 1536#define MIN_PKT_LEN 60 -#define ETH_ADDR_LEN 6#define AMD8111E_TX_TIMEOUT (3 * HZ)/* 3 sec */ #define SOFT_TIMER_FREQ 0xBEBC/* 0.5 sec */
Signature · 1 line
1.7.6.405.gc1be0
PATCH 5/5 ucc_geth: Convert ENET_NUM_OCTETS_PER_ADDRESS uses to ETH_ALEN
In reply to Joe PerchesReduce the number of #defines, use the normal #define from if_ether.h
--- drivers/net/ethernet/freescale/ucc_geth.c | 2 +- drivers/net/ethernet/freescale/ucc_geth.h | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-)
drivers/net/ethernet/freescale/ucc_geth.c +1 -1
@@ -443,7 +443,7 @@ static void hw_add_addr_in_hash(struct ucc_geth_private *ugeth,static inline int compare_addr (u8 **addr1,u8 **addr2) { -return memcmp (addr1, addr2,ENET_NUM_OCTETS_PER_ADDRESS ); +return memcmp (addr1, addr2,ETH_ALEN ); }#ifdef DEBUG
drivers/net/ethernet/freescale/ucc_geth.h +3 -3
@@ -20,6 +20,7 @@#include <linux/kernel.h> #include <linux/list.h> +#include <linux/if_ether.h> #include <asm/immap_qe.h> #include <asm/qe.h>
@@ -881,7 +882,6 @@ struct ucc_geth_hardware_statistics {#define TX_RING_MOD_MASK (size) (size-1)#define RX_RING_MOD_MASK (size) (size-1) -#define ENET_NUM_OCTETS_PER_ADDRESS 6#define ENET_GROUP_ADDR 0x01/* Group address mask for ethernet addresses */
@@ -1051,7 +1051,7 @@ enum ucc_geth_num_of_station_addresses {/* UCC GETH 82xx Ethernet Address Container */ struct enet_addr_container { -u8 address[ENET_NUM_OCTETS_PER_ADDRESS ];/* ethernet address */ +u8 address[ETH_ALEN ];/* ethernet address */ enum ucc_geth_enet_address_recognition_location location;/* location in 82xx address recognition
@@ -1194,7 +1194,7 @@ struct ucc_geth_private {u16 cpucount[NUM_TX_QUEUES ];u16 __iomem *p_cpucount[NUM_TX_QUEUES ];int indAddrRegUsed[NUM_OF_PADDRS ]; -u8 paddr[NUM_OF_PADDRS ][ENET_NUM_OCTETS_PER_ADDRESS ];/* ethernet address */ +u8 paddr[NUM_OF_PADDRS ][ETH_ALEN ];/* ethernet address */ u8 numGroupAddrInHash;u8 numIndAddrInHash;u8 numIndAddrInReg;
Signature · 1 line
1.7.6.405.gc1be0
David Miller
maintainerFrom: Joe Perches <[email protected]>
Date: Wed, 16 Nov 2011 11:38:01 -0800
Quoted 8 lines
Remove other #defines and uses in favor of
ETH_ALENJoe Perches (5):
ethernet: ConvertMAC_ADDR_LENuses toETH_ALEN
ethernet: ConvertETHER_ADDR_LENuses toETH_ALEN
bna: ConvertMAC_ADDRLENuses toETH_ALEN
amd8111e: ConvertETH_ADDR_LENuses toETH_ALEN
ucc_geth: ConvertENET_NUM_OCTETS_PER_ADDRESSuses toETH_ALEN
All applied, thanks Joe.