diff options
author | Jiri Pirko <jpirko@redhat.com> | 2010-02-23 18:17:07 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-02-26 05:07:31 -0500 |
commit | 567ec874d15b478c8eda7e9a5d2dcb05f13f1fb5 (patch) | |
tree | e0c22740e44eca6f035e7dccbcff114a72d007a3 /drivers/net/smc91x.c | |
parent | f9dcbcc9e338d08c0f7de7eba4eaafbbb7f81249 (diff) |
net: convert multiple drivers to use netdev_for_each_mc_addr, part6
Signed-off-by: Jiri Pirko <jpirko@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/smc91x.c')
-rw-r--r-- | drivers/net/smc91x.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/net/smc91x.c b/drivers/net/smc91x.c index 66450127c5a1..fc1b5a1a3583 100644 --- a/drivers/net/smc91x.c +++ b/drivers/net/smc91x.c | |||
@@ -1413,7 +1413,6 @@ static void smc_set_multicast_list(struct net_device *dev) | |||
1413 | * within that register. | 1413 | * within that register. |
1414 | */ | 1414 | */ |
1415 | else if (!netdev_mc_empty(dev)) { | 1415 | else if (!netdev_mc_empty(dev)) { |
1416 | int i; | ||
1417 | struct dev_mc_list *cur_addr; | 1416 | struct dev_mc_list *cur_addr; |
1418 | 1417 | ||
1419 | /* table for flipping the order of 3 bits */ | 1418 | /* table for flipping the order of 3 bits */ |
@@ -1422,13 +1421,9 @@ static void smc_set_multicast_list(struct net_device *dev) | |||
1422 | /* start with a table of all zeros: reject all */ | 1421 | /* start with a table of all zeros: reject all */ |
1423 | memset(multicast_table, 0, sizeof(multicast_table)); | 1422 | memset(multicast_table, 0, sizeof(multicast_table)); |
1424 | 1423 | ||
1425 | cur_addr = dev->mc_list; | 1424 | netdev_for_each_mc_addr(cur_addr, dev) { |
1426 | for (i = 0; i < netdev_mc_count(dev); i++, cur_addr = cur_addr->next) { | ||
1427 | int position; | 1425 | int position; |
1428 | 1426 | ||
1429 | /* do we have a pointer here? */ | ||
1430 | if (!cur_addr) | ||
1431 | break; | ||
1432 | /* make sure this is a multicast address - | 1427 | /* make sure this is a multicast address - |
1433 | shouldn't this be a given if we have it here ? */ | 1428 | shouldn't this be a given if we have it here ? */ |
1434 | if (!(*cur_addr->dmi_addr & 1)) | 1429 | if (!(*cur_addr->dmi_addr & 1)) |