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/netxen | |
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/netxen')
-rw-r--r-- | drivers/net/netxen/netxen_nic_hw.c | 19 |
1 files changed, 7 insertions, 12 deletions
diff --git a/drivers/net/netxen/netxen_nic_hw.c b/drivers/net/netxen/netxen_nic_hw.c index 25f4414cc33e..a945591298a8 100644 --- a/drivers/net/netxen/netxen_nic_hw.c +++ b/drivers/net/netxen/netxen_nic_hw.c | |||
@@ -539,7 +539,7 @@ void netxen_p2_nic_set_multi(struct net_device *netdev) | |||
539 | struct netxen_adapter *adapter = netdev_priv(netdev); | 539 | struct netxen_adapter *adapter = netdev_priv(netdev); |
540 | struct dev_mc_list *mc_ptr; | 540 | struct dev_mc_list *mc_ptr; |
541 | u8 null_addr[6]; | 541 | u8 null_addr[6]; |
542 | int index = 0; | 542 | int i; |
543 | 543 | ||
544 | memset(null_addr, 0, 6); | 544 | memset(null_addr, 0, 6); |
545 | 545 | ||
@@ -570,16 +570,13 @@ void netxen_p2_nic_set_multi(struct net_device *netdev) | |||
570 | 570 | ||
571 | netxen_nic_enable_mcast_filter(adapter); | 571 | netxen_nic_enable_mcast_filter(adapter); |
572 | 572 | ||
573 | for (mc_ptr = netdev->mc_list; mc_ptr; mc_ptr = mc_ptr->next, index++) | 573 | i = 0; |
574 | netxen_nic_set_mcast_addr(adapter, index, mc_ptr->dmi_addr); | 574 | netdev_for_each_mc_addr(mc_ptr, netdev) |
575 | 575 | netxen_nic_set_mcast_addr(adapter, i++, mc_ptr->dmi_addr); | |
576 | if (index != netdev_mc_count(netdev)) | ||
577 | printk(KERN_WARNING "%s: %s multicast address count mismatch\n", | ||
578 | netxen_nic_driver_name, netdev->name); | ||
579 | 576 | ||
580 | /* Clear out remaining addresses */ | 577 | /* Clear out remaining addresses */ |
581 | for (; index < adapter->max_mc_count; index++) | 578 | while (i < adapter->max_mc_count) |
582 | netxen_nic_set_mcast_addr(adapter, index, null_addr); | 579 | netxen_nic_set_mcast_addr(adapter, i++, null_addr); |
583 | } | 580 | } |
584 | 581 | ||
585 | static int | 582 | static int |
@@ -710,10 +707,8 @@ void netxen_p3_nic_set_multi(struct net_device *netdev) | |||
710 | } | 707 | } |
711 | 708 | ||
712 | if (!netdev_mc_empty(netdev)) { | 709 | if (!netdev_mc_empty(netdev)) { |
713 | for (mc_ptr = netdev->mc_list; mc_ptr; | 710 | netdev_for_each_mc_addr(mc_ptr, netdev) |
714 | mc_ptr = mc_ptr->next) { | ||
715 | nx_p3_nic_add_mac(adapter, mc_ptr->dmi_addr, &del_list); | 711 | nx_p3_nic_add_mac(adapter, mc_ptr->dmi_addr, &del_list); |
716 | } | ||
717 | } | 712 | } |
718 | 713 | ||
719 | send_fw_cmd: | 714 | send_fw_cmd: |