drivers/net/oki-semi: Remove the definition of PCH_GBE_ETH_ALEN
authorRongQing.Li <roy.qing.li@gmail.com>
Thu, 26 Apr 2012 21:01:12 +0000 (21:01 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sun, 29 Apr 2012 01:59:17 +0000 (21:59 -0400)
PCH_GBE_ETH_ALEN is equal to ETH_ALEN, so we can replace it with
ETH_ALEN.

If they are not equal, it must be a bug, since this is ethernet,
and the address has been already stored to mc_addr_list as ETH_ALEN
bytes when call pch_gbe_mac_mc_addr_list_update.

Signed-off-by: RongQing.Li <roy.qing.li@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/oki-semi/pch_gbe/pch_gbe_main.c

index 89c6bcf4bca27239d3ecdeea7de32d8f628dc883..835e3bbdac18357412776f97a410caa34a8d8b13 100644 (file)
@@ -79,7 +79,6 @@ const char pch_driver_version[] = DRV_VERSION;
 #define        PCH_GBE_PAUSE_PKT4_VALUE    0x01000888
 #define        PCH_GBE_PAUSE_PKT5_VALUE    0x0000FFFF
 
-#define PCH_GBE_ETH_ALEN            6
 
 /* This defines the bits that are set in the Interrupt Mask
  * Set/Read Register.  Each bit is documented below:
@@ -519,7 +518,7 @@ static void pch_gbe_mac_mc_addr_list_update(struct pch_gbe_hw *hw,
                if (mc_addr_count) {
                        pch_gbe_mac_mar_set(hw, mc_addr_list, i);
                        mc_addr_count--;
-                       mc_addr_list += PCH_GBE_ETH_ALEN;
+                       mc_addr_list += ETH_ALEN;
                } else {
                        /* Clear MAC address mask */
                        adrmask = ioread32(&hw->reg->ADDR_MASK);