diff options
author | Tobias Klauser <tklauser@distanz.ch> | 2011-07-07 18:06:26 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-07-08 12:03:20 -0400 |
commit | 498d8e236304a62a2774d7264bdff2c6e8102b5b (patch) | |
tree | c1c5fa5ddf09a76bf0787e09fdc71fb8d5561b1c /drivers/net/hp100.c | |
parent | 40f5d72a4fc098c47068e3888cfb055922f6519f (diff) |
drivers/net: Omit check for multicast bit in netdev_for_each_mc_addr
There is no need to check for the address being a multicast address in
the netdev_for_each_mc_addr loop, so remove it. This patch covers all
remaining network drivers still containing such a check.
Cc: Joe Perches <joe@perches.com>
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/hp100.c')
-rw-r--r-- | drivers/net/hp100.c | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/drivers/net/hp100.c b/drivers/net/hp100.c index c3ecb118c1df..b6519c1ba7e1 100644 --- a/drivers/net/hp100.c +++ b/drivers/net/hp100.c | |||
@@ -2103,20 +2103,18 @@ static void hp100_set_multicast_list(struct net_device *dev) | |||
2103 | #endif | 2103 | #endif |
2104 | netdev_for_each_mc_addr(ha, dev) { | 2104 | netdev_for_each_mc_addr(ha, dev) { |
2105 | addrs = ha->addr; | 2105 | addrs = ha->addr; |
2106 | if ((*addrs & 0x01) == 0x01) { /* multicast address? */ | ||
2107 | #ifdef HP100_DEBUG | 2106 | #ifdef HP100_DEBUG |
2108 | printk("hp100: %s: multicast = %pM, ", | 2107 | printk("hp100: %s: multicast = %pM, ", |
2109 | dev->name, addrs); | 2108 | dev->name, addrs); |
2110 | #endif | 2109 | #endif |
2111 | for (i = idx = 0; i < 6; i++) { | 2110 | for (i = idx = 0; i < 6; i++) { |
2112 | idx ^= *addrs++ & 0x3f; | 2111 | idx ^= *addrs++ & 0x3f; |
2113 | printk(":%02x:", idx); | 2112 | printk(":%02x:", idx); |
2114 | } | 2113 | } |
2115 | #ifdef HP100_DEBUG | 2114 | #ifdef HP100_DEBUG |
2116 | printk("idx = %i\n", idx); | 2115 | printk("idx = %i\n", idx); |
2117 | #endif | 2116 | #endif |
2118 | lp->hash_bytes[idx >> 3] |= (1 << (idx & 7)); | 2117 | lp->hash_bytes[idx >> 3] |= (1 << (idx & 7)); |
2119 | } | ||
2120 | } | 2118 | } |
2121 | } | 2119 | } |
2122 | #else | 2120 | #else |