diff options
author | Jiri Pirko <jpirko@redhat.com> | 2010-02-23 04:19:49 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-02-26 05:07:30 -0500 |
commit | f9dcbcc9e338d08c0f7de7eba4eaafbbb7f81249 (patch) | |
tree | 8a1d9a37bc057440220a5ad23231e0fe974b93f6 /drivers/net/ks8851.c | |
parent | 52c793f24054f5dc30d228e37e0e19cc8313f086 (diff) |
net: convert multiple drivers to use netdev_for_each_mc_addr, part5 V2
removed some needless checks and also corrected bug in lp486e (dmi was passed
instead of dmi->dmi_addr)
Signed-off-by: Jiri Pirko <jpirko@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/ks8851.c')
-rw-r--r-- | drivers/net/ks8851.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/net/ks8851.c b/drivers/net/ks8851.c index 9845ab1e5573..b5219cce12ed 100644 --- a/drivers/net/ks8851.c +++ b/drivers/net/ks8851.c | |||
@@ -966,13 +966,12 @@ static void ks8851_set_rx_mode(struct net_device *dev) | |||
966 | rxctrl.rxcr1 = (RXCR1_RXME | RXCR1_RXAE | | 966 | rxctrl.rxcr1 = (RXCR1_RXME | RXCR1_RXAE | |
967 | RXCR1_RXPAFMA | RXCR1_RXMAFMA); | 967 | RXCR1_RXPAFMA | RXCR1_RXMAFMA); |
968 | } else if (dev->flags & IFF_MULTICAST && !netdev_mc_empty(dev)) { | 968 | } else if (dev->flags & IFF_MULTICAST && !netdev_mc_empty(dev)) { |
969 | struct dev_mc_list *mcptr = dev->mc_list; | 969 | struct dev_mc_list *mcptr; |
970 | u32 crc; | 970 | u32 crc; |
971 | int i; | ||
972 | 971 | ||
973 | /* accept some multicast */ | 972 | /* accept some multicast */ |
974 | 973 | ||
975 | for (i = netdev_mc_count(dev); i > 0; i--) { | 974 | netdev_for_each_mc_addr(mcptr, dev) { |
976 | crc = ether_crc(ETH_ALEN, mcptr->dmi_addr); | 975 | crc = ether_crc(ETH_ALEN, mcptr->dmi_addr); |
977 | crc >>= (32 - 6); /* get top six bits */ | 976 | crc >>= (32 - 6); /* get top six bits */ |
978 | 977 | ||