chelsio: Fix build warning.
authorDavid S. Miller <davem@davemloft.net>
Tue, 13 Apr 2010 10:07:17 +0000 (03:07 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 13 Apr 2010 10:07:17 +0000 (03:07 -0700)
GCC warns that:

drivers/net/chelsio/sge.c:463:11: warning: operation on 's->port' may be undefined

Better to eliminate the side effects in the calculation and
express what was intended here.

Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/chelsio/sge.c

index 475304f..a8ffc1e 100644 (file)
@@ -460,7 +460,7 @@ static struct sk_buff *sched_skb(struct sge *sge, struct sk_buff *skb,
 
 again:
        for (i = 0; i < MAX_NPORTS; i++) {
-               s->port = ++s->port & (MAX_NPORTS - 1);
+               s->port = (s->port + 1) & (MAX_NPORTS - 1);
                skbq = &s->p[s->port].skbq;
 
                skb = skb_peek(skbq);