net/usb: convert to use netdev_for_each_mc_addr
authorJiri Pirko <jpirko@redhat.com>
Thu, 18 Feb 2010 04:02:26 +0000 (04:02 +0000)
committerDavid S. Miller <davem@davemloft.net>
Thu, 18 Feb 2010 22:47:50 +0000 (14:47 -0800)
also removed needless checks in smsc95xx

Signed-off-by: Jiri Pirko <jpirko@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/usb/asix.c
drivers/net/usb/catc.c
drivers/net/usb/mcs7830.c
drivers/net/usb/smsc95xx.c

index f605204de3e5eef414634526e5fac57d55bcb277..20e34608fa4a47ab7d8de9ddba4d9e408801fc9b 100644 (file)
@@ -555,20 +555,18 @@ static void asix_set_multicast(struct net_device *net)
                 * for our 8 byte filter buffer
                 * to avoid allocating memory that
                 * is tricky to free later */
-               struct dev_mc_list *mc_list = net->mc_list;
+               struct dev_mc_list *mc_list;
                u32 crc_bits;
-               int i;
 
                memset(data->multi_filter, 0, AX_MCAST_FILTER_SIZE);
 
                /* Build the multicast hash filter. */
-               for (i = 0; i < netdev_mc_count(net); i++) {
+               netdev_for_each_mc_addr(mc_list, net) {
                        crc_bits =
                            ether_crc(ETH_ALEN,
                                      mc_list->dmi_addr) >> 26;
                        data->multi_filter[crc_bits >> 3] |=
                            1 << (crc_bits & 7);
-                       mc_list = mc_list->next;
                }
 
                asix_write_cmd_async(dev, AX_CMD_WRITE_MULTI_FILTER, 0, 0,
@@ -769,20 +767,18 @@ static void ax88172_set_multicast(struct net_device *net)
                 * for our 8 byte filter buffer
                 * to avoid allocating memory that
                 * is tricky to free later */
-               struct dev_mc_list *mc_list = net->mc_list;
+               struct dev_mc_list *mc_list;
                u32 crc_bits;
-               int i;
 
                memset(data->multi_filter, 0, AX_MCAST_FILTER_SIZE);
 
                /* Build the multicast hash filter. */
-               for (i = 0; i < netdev_mc_count(net); i++) {
+               netdev_for_each_mc_addr(mc_list, net) {
                        crc_bits =
                            ether_crc(ETH_ALEN,
                                      mc_list->dmi_addr) >> 26;
                        data->multi_filter[crc_bits >> 3] |=
                            1 << (crc_bits & 7);
-                       mc_list = mc_list->next;
                }
 
                asix_write_cmd_async(dev, AX_CMD_WRITE_MULTI_FILTER, 0, 0,
index 5a13660ebd17814cd5af445986987b7f8e45b403..96f1ebe0d3484d490fb1bb1d514e6df15c4aa1c0 100644 (file)
@@ -632,7 +632,6 @@ static void catc_set_multicast_list(struct net_device *netdev)
        struct dev_mc_list *mc;
        u8 broadcast[6];
        u8 rx = RxEnable | RxPolarity | RxMultiCast;
-       int i;
 
        memset(broadcast, 0xff, 6);
        memset(catc->multicast, 0, 64);
@@ -648,9 +647,7 @@ static void catc_set_multicast_list(struct net_device *netdev)
        if (netdev->flags & IFF_ALLMULTI) {
                memset(catc->multicast, 0xff, 64);
        } else {
-               for (i = 0, mc = netdev->mc_list;
-                    mc && i < netdev_mc_count(netdev);
-                    i++, mc = mc->next) {
+               netdev_for_each_mc_addr(mc, netdev) {
                        u32 crc = ether_crc_le(6, mc->dmi_addr);
                        if (!catc->is_f5u011) {
                                catc->multicast[(crc >> 3) & 0x3f] |= 1 << (crc & 7);
index 34665137f2c330aabf01b3b83838fa52d035f574..70978219e98afb8005a1519e4120a6f8acd7e23d 100644 (file)
@@ -452,15 +452,13 @@ static void mcs7830_data_set_multicast(struct net_device *net)
                 * for our 8 byte filter buffer
                 * to avoid allocating memory that
                 * is tricky to free later */
-               struct dev_mc_list *mc_list = net->mc_list;
+               struct dev_mc_list *mc_list;
                u32 crc_bits;
-               int i;
 
                /* Build the multicast hash filter. */
-               for (i = 0; i < netdev_mc_count(net); i++) {
+               netdev_for_each_mc_addr(mc_list, net) {
                        crc_bits = ether_crc(ETH_ALEN, mc_list->dmi_addr) >> 26;
                        data->multi_filter[crc_bits >> 3] |= 1 << (crc_bits & 7);
-                       mc_list = mc_list->next;
                }
        }
 }
index 1ada51eb71f7df3409ce77992d2a6d99ddd0b826..df9179a1c93bfb22c1f104ea27233a036da3f49a 100644 (file)
@@ -384,30 +384,20 @@ static void smsc95xx_set_multicast(struct net_device *netdev)
                pdata->mac_cr |= MAC_CR_MCPAS_;
                pdata->mac_cr &= ~(MAC_CR_PRMS_ | MAC_CR_HPFILT_);
        } else if (!netdev_mc_empty(dev->net)) {
-               struct dev_mc_list *mc_list = dev->net->mc_list;
-               int count = 0;
+               struct dev_mc_list *mc_list;
 
                pdata->mac_cr |= MAC_CR_HPFILT_;
                pdata->mac_cr &= ~(MAC_CR_PRMS_ | MAC_CR_MCPAS_);
 
-               while (mc_list) {
-                       count++;
-                       if (mc_list->dmi_addrlen == ETH_ALEN) {
-                               u32 bitnum = smsc95xx_hash(mc_list->dmi_addr);
-                               u32 mask = 0x01 << (bitnum & 0x1F);
-                               if (bitnum & 0x20)
-                                       hash_hi |= mask;
-                               else
-                                       hash_lo |= mask;
-                       } else {
-                               netdev_warn(dev->net, "dmi_addrlen != 6\n");
-                       }
-                       mc_list = mc_list->next;
+               netdev_for_each_mc_addr(mc_list, netdev) {
+                       u32 bitnum = smsc95xx_hash(mc_list->dmi_addr);
+                       u32 mask = 0x01 << (bitnum & 0x1F);
+                       if (bitnum & 0x20)
+                               hash_hi |= mask;
+                       else
+                               hash_lo |= mask;
                }
 
-               if (count != ((u32) netdev_mc_count(dev->net)))
-                       netdev_warn(dev->net, "mc_count != dev->mc_count\n");
-
                netif_dbg(dev, drv, dev->net, "HASHH=0x%08X, HASHL=0x%08X\n",
                                   hash_hi, hash_lo);
        } else {