diff options
| author | Alan Cox <alan@linux.intel.com> | 2011-02-01 13:19:07 -0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-02-01 13:19:07 -0800 |
| commit | f97f3057fd2febbd7f34a60f09a2cb9cef8bf403 (patch) | |
| tree | f1ddc24cfdd6252c5c631fbd2dbab01a0a919027 | |
| parent | 2ba5e1feaab2762cdf45b490e3f342fedb33f57b (diff) | |
| download | op-kernel-dev-f97f3057fd2febbd7f34a60f09a2cb9cef8bf403.zip op-kernel-dev-f97f3057fd2febbd7f34a60f09a2cb9cef8bf403.tar.gz | |
depca: Fix warnings
Replace the rather weird use of ++ with + 1 as the value is being assigned
Signed-off-by: Alan Cox <alan@linux.intel.com>
| -rw-r--r-- | drivers/net/depca.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/net/depca.c b/drivers/net/depca.c index 1b48b68..8b0084d 100644 --- a/drivers/net/depca.c +++ b/drivers/net/depca.c @@ -1094,7 +1094,7 @@ static int depca_rx(struct net_device *dev) } } /* Change buffer ownership for this last frame, back to the adapter */ - for (; lp->rx_old != entry; lp->rx_old = (++lp->rx_old) & lp->rxRingMask) { + for (; lp->rx_old != entry; lp->rx_old = (lp->rx_old + 1) & lp->rxRingMask) { writel(readl(&lp->rx_ring[lp->rx_old].base) | R_OWN, &lp->rx_ring[lp->rx_old].base); } writel(readl(&lp->rx_ring[entry].base) | R_OWN, &lp->rx_ring[entry].base); @@ -1103,7 +1103,7 @@ static int depca_rx(struct net_device *dev) /* ** Update entry information */ - lp->rx_new = (++lp->rx_new) & lp->rxRingMask; + lp->rx_new = (lp->rx_new + 1) & lp->rxRingMask; } return 0; @@ -1148,7 +1148,7 @@ static int depca_tx(struct net_device *dev) } /* Update all the pointers */ - lp->tx_old = (++lp->tx_old) & lp->txRingMask; + lp->tx_old = (lp->tx_old + 1) & lp->txRingMask; } return 0; |
