Re: [11/17] r6040: Fix multicast list iteration when hash filteris used

From: Greg KH
Date: Fri Oct 22 2010 - 15:24:13 EST


On Fri, Oct 22, 2010 at 08:13:27PM +0100, Jack Stone wrote:
> On 22/10/2010 19:39, Greg KH wrote:
> > drivers/net/r6040.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > --- a/drivers/net/r6040.c
> > +++ b/drivers/net/r6040.c
> > @@ -976,6 +976,7 @@ static void r6040_multicast_list(struct
> > iowrite16(hash_table[3], ioaddr + MAR3);
> > }
> > /* Multicast Address 1~4 case */
> > + dmi = dev->mc_list;
> > for (i = 0, dmi; (i < dev->mc_count) && (i < MCAST_MAX); i++) {
> Any reason for the dmi in the above line? As far as I can see it is a
> nop.

Look closer at the for loop please.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/