diff options
| author | David S. Miller <davem@davemloft.net> | 2010-04-13 12:07:17 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-04-13 12:07:17 +0200 |
| commit | 18d777a5bcf31491021f527d32b72dd16f911fe0 (patch) | |
| tree | 128f7d733523ab54b1cd158e9276f428964da31b /drivers/net/chelsio | |
| parent | e1000e: use static params to save stack space (diff) | |
| download | linux-18d777a5bcf31491021f527d32b72dd16f911fe0.tar.xz linux-18d777a5bcf31491021f527d32b72dd16f911fe0.zip | |
chelsio: Fix build warning.
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>
Diffstat (limited to 'drivers/net/chelsio')
| -rw-r--r-- | drivers/net/chelsio/sge.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/chelsio/sge.c b/drivers/net/chelsio/sge.c index 475304fe4628..a8ffc1e11c5f 100644 --- a/drivers/net/chelsio/sge.c +++ b/drivers/net/chelsio/sge.c @@ -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); |