diff options
author | Jiri Pirko <jpirko@redhat.com> | 2010-02-22 04:22:26 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-02-22 18:45:52 -0500 |
commit | 48e2f183cb1709600012265a2e723f45a350d5fe (patch) | |
tree | 608368bda6f159342c307deca3b03998d25434c8 /drivers/net/forcedeth.c | |
parent | 7a81e9f3ca712db82344ea3ab2a5879241f59c48 (diff) |
net: convert multiple drivers to use netdev_for_each_mc_addr, part4
Signed-off-by: Jiri Pirko <jpirko@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/forcedeth.c')
-rw-r--r-- | drivers/net/forcedeth.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/net/forcedeth.c b/drivers/net/forcedeth.c index 3eb713b014f9..ca05e5662029 100644 --- a/drivers/net/forcedeth.c +++ b/drivers/net/forcedeth.c | |||
@@ -3095,7 +3095,7 @@ static void nv_set_multicast(struct net_device *dev) | |||
3095 | } else { | 3095 | } else { |
3096 | pff |= NVREG_PFF_MYADDR; | 3096 | pff |= NVREG_PFF_MYADDR; |
3097 | 3097 | ||
3098 | if (dev->flags & IFF_ALLMULTI || dev->mc_list) { | 3098 | if (dev->flags & IFF_ALLMULTI || !netdev_mc_empty(dev)) { |
3099 | u32 alwaysOff[2]; | 3099 | u32 alwaysOff[2]; |
3100 | u32 alwaysOn[2]; | 3100 | u32 alwaysOn[2]; |
3101 | 3101 | ||
@@ -3105,8 +3105,7 @@ static void nv_set_multicast(struct net_device *dev) | |||
3105 | } else { | 3105 | } else { |
3106 | struct dev_mc_list *walk; | 3106 | struct dev_mc_list *walk; |
3107 | 3107 | ||
3108 | walk = dev->mc_list; | 3108 | netdev_for_each_mc_addr(walk, dev) { |
3109 | while (walk != NULL) { | ||
3110 | u32 a, b; | 3109 | u32 a, b; |
3111 | a = le32_to_cpu(*(__le32 *) walk->dmi_addr); | 3110 | a = le32_to_cpu(*(__le32 *) walk->dmi_addr); |
3112 | b = le16_to_cpu(*(__le16 *) (&walk->dmi_addr[4])); | 3111 | b = le16_to_cpu(*(__le16 *) (&walk->dmi_addr[4])); |
@@ -3114,7 +3113,6 @@ static void nv_set_multicast(struct net_device *dev) | |||
3114 | alwaysOff[0] &= ~a; | 3113 | alwaysOff[0] &= ~a; |
3115 | alwaysOn[1] &= b; | 3114 | alwaysOn[1] &= b; |
3116 | alwaysOff[1] &= ~b; | 3115 | alwaysOff[1] &= ~b; |
3117 | walk = walk->next; | ||
3118 | } | 3116 | } |
3119 | } | 3117 | } |
3120 | addr[0] = alwaysOn[0]; | 3118 | addr[0] = alwaysOn[0]; |