net: fix up a few missing hashtable.h conflict resolutions
authorJiri Kosina <jkosina@suse.cz>
Fri, 12 Aug 2016 14:34:49 +0000 (16:34 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sat, 13 Aug 2016 21:51:02 +0000 (14:51 -0700)
There are a couple of leftover symbol conflicts caused by hashtable.h
being included by netdevice.h; those were not caught as build failure
(they're "only" a warning, but in fact real bugs). Fix those up.

Fixes: e87a8f24c ("net: resolve symbol conflicts with generic hashtable.h")
Reported-by: Daniel Borkmann <daniel@iogearbox.net>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/dec/tulip/de4x5.c
drivers/net/ethernet/dec/tulip/de4x5.h
drivers/net/ethernet/freescale/fec_main.c

index f0e9e2ef62a06e3e04a7c71516d2849c251397fa..6620fc861c477eeb38d3c2b9495619b0e65ca63a 100644 (file)
@@ -1966,7 +1966,7 @@ SetMulticastFilter(struct net_device *dev)
     } else if (lp->setup_f == HASH_PERF) {   /* Hash Filtering */
        netdev_for_each_mc_addr(ha, dev) {
                crc = ether_crc_le(ETH_ALEN, ha->addr);
-               hashcode = crc & HASH_BITS;  /* hashcode is 9 LSb of CRC */
+               hashcode = crc & DE4X5_HASH_BITS;  /* hashcode is 9 LSb of CRC */
 
                byte = hashcode >> 3;        /* bit[3-8] -> byte in filter */
                bit = 1 << (hashcode & 0x07);/* bit[0-2] -> bit in byte */
@@ -5043,7 +5043,7 @@ build_setup_frame(struct net_device *dev, int mode)
            *(pa + i) = dev->dev_addr[i];                 /* Host address */
            if (i & 0x01) pa += 2;
        }
-       *(lp->setup_frame + (HASH_TABLE_LEN >> 3) - 3) = 0x80;
+       *(lp->setup_frame + (DE4X5_HASH_TABLE_LEN >> 3) - 3) = 0x80;
     } else {
        for (i=0; i<ETH_ALEN; i++) { /* Host address */
            *(pa + (i&1)) = dev->dev_addr[i];
index ec756eba397bcb011f1af4ded725d65ca759231b..1bfdc9b117f6294913787b67732f9e09c08af38d 100644 (file)
 #define PCI  0
 #define EISA 1
 
-#define HASH_TABLE_LEN   512       /* Bits */
-#define HASH_BITS        0x01ff    /* 9 LS bits */
+#define DE4X5_HASH_TABLE_LEN   512       /* Bits */
+#define DE4X5_HASH_BITS        0x01ff    /* 9 LS bits */
 
 #define SETUP_FRAME_LEN  192       /* Bytes */
 #define IMPERF_PA_OFFSET 156       /* Bytes */
index 01f7e811739bbf215d94a216e966b74b4f2e2c3a..fb5c638813409bef38ea6af3dd7031a26a0e0e71 100644 (file)
@@ -2887,7 +2887,7 @@ fec_enet_close(struct net_device *ndev)
  * this kind of feature?).
  */
 
-#define HASH_BITS      6               /* #bits in hash */
+#define FEC_HASH_BITS  6               /* #bits in hash */
 #define CRC32_POLY     0xEDB88320
 
 static void set_multicast_list(struct net_device *ndev)
@@ -2935,10 +2935,10 @@ static void set_multicast_list(struct net_device *ndev)
                        }
                }
 
-               /* only upper 6 bits (HASH_BITS) are used
+               /* only upper 6 bits (FEC_HASH_BITS) are used
                 * which point to specific bit in he hash registers
                 */
-               hash = (crc >> (32 - HASH_BITS)) & 0x3f;
+               hash = (crc >> (32 - FEC_HASH_BITS)) & 0x3f;
 
                if (hash > 31) {
                        tmp = readl(fep->hwp + FEC_GRP_HASH_TABLE_HIGH);