diff options
| author | David S. Miller <davem@davemloft.net> | 2010-11-18 11:49:25 -0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-11-18 11:49:25 -0800 |
| commit | 30dfe2c05037fbc021121c037872c09956938c2f (patch) | |
| tree | 2db9f69aaddf6109946810a53d837a25ae595757 /drivers/atm | |
| parent | 57e1ab6eaddc9f2c358cd4afb497cda6e3c6821a (diff) | |
| download | lwn-30dfe2c05037fbc021121c037872c09956938c2f.tar.gz lwn-30dfe2c05037fbc021121c037872c09956938c2f.zip | |
atm: fore200e: Fix build warning.
GCC (rightfully) complains that:
drivers/atm/fore200e.c:614:5: warning: operation on 'cmdq->head' may be undefined
This is due to the FORE200E_NEXT_ENTRY macro, which essentially
evaluates to:
i = ++i % m
Make it what's explicitly intended here which is:
i = (i + 1) % m
and the warning goes away.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/atm')
| -rw-r--r-- | drivers/atm/fore200e.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/atm/fore200e.c b/drivers/atm/fore200e.c index c8fc69c85a06..c09761959354 100644 --- a/drivers/atm/fore200e.c +++ b/drivers/atm/fore200e.c @@ -92,7 +92,7 @@ #define FORE200E_INDEX(virt_addr, type, index) (&((type *)(virt_addr))[ index ]) -#define FORE200E_NEXT_ENTRY(index, modulo) (index = ++(index) % (modulo)) +#define FORE200E_NEXT_ENTRY(index, modulo) (index = ((index) + 1) % (modulo)) #if 1 #define ASSERT(expr) if (!(expr)) { \ |