diff options
| author | Jiri Pirko <jpirko@redhat.com> | 2010-02-23 10:54:58 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-02-26 11:08:33 +0100 |
| commit | fbc450b137e59205b98a172326415cd981c9caa6 (patch) | |
| tree | e9bb0a901ed69c7699069bd323b098b216f57499 | |
| parent | net: convert multiple drivers to use netdev_for_each_mc_addr, part6 (diff) | |
| download | linux-fbc450b137e59205b98a172326415cd981c9caa6.tar.xz linux-fbc450b137e59205b98a172326415cd981c9caa6.zip | |
octeon: convert to use netdev_for_each_mc_addr
Hmm so actually my original patch including this bit was correct,
"list = list->next;" confused me :) - will send patch correcting that in a few.
Signed-off-by: Jiri Pirko <jpirko@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | drivers/net/octeon/octeon_mgmt.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/drivers/net/octeon/octeon_mgmt.c b/drivers/net/octeon/octeon_mgmt.c index 3a0f910924a5..be368e5cbf75 100644 --- a/drivers/net/octeon/octeon_mgmt.c +++ b/drivers/net/octeon/octeon_mgmt.c @@ -467,7 +467,6 @@ static void octeon_mgmt_set_rx_filtering(struct net_device *netdev) { struct octeon_mgmt *p = netdev_priv(netdev); int port = p->port; - int i; union cvmx_agl_gmx_rxx_adr_ctl adr_ctl; union cvmx_agl_gmx_prtx_cfg agl_gmx_prtx; unsigned long flags; @@ -511,12 +510,8 @@ static void octeon_mgmt_set_rx_filtering(struct net_device *netdev) } } if (multicast_mode == 0) { - i = netdev_mc_count(netdev); - list = netdev->mc_list; - while (i--) { + netdev_for_each_mc_addr(list, netdev) octeon_mgmt_cam_state_add(&cam_state, list->da_addr); - list = list->next; - } } |