Commit f97f3057fd2febbd7f34a60f09a2cb9cef8bf403
Committed by
David S. Miller
1 parent
2ba5e1feaa
Exists in
master
and in
39 other branches
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>
Showing 1 changed file with 3 additions and 3 deletions Side-by-side Diff
drivers/net/depca.c
... | ... | @@ -1094,7 +1094,7 @@ |
1094 | 1094 | } |
1095 | 1095 | } |
1096 | 1096 | /* Change buffer ownership for this last frame, back to the adapter */ |
1097 | - for (; lp->rx_old != entry; lp->rx_old = (++lp->rx_old) & lp->rxRingMask) { | |
1097 | + for (; lp->rx_old != entry; lp->rx_old = (lp->rx_old + 1) & lp->rxRingMask) { | |
1098 | 1098 | writel(readl(&lp->rx_ring[lp->rx_old].base) | R_OWN, &lp->rx_ring[lp->rx_old].base); |
1099 | 1099 | } |
1100 | 1100 | writel(readl(&lp->rx_ring[entry].base) | R_OWN, &lp->rx_ring[entry].base); |
... | ... | @@ -1103,7 +1103,7 @@ |
1103 | 1103 | /* |
1104 | 1104 | ** Update entry information |
1105 | 1105 | */ |
1106 | - lp->rx_new = (++lp->rx_new) & lp->rxRingMask; | |
1106 | + lp->rx_new = (lp->rx_new + 1) & lp->rxRingMask; | |
1107 | 1107 | } |
1108 | 1108 | |
1109 | 1109 | return 0; |
... | ... | @@ -1148,7 +1148,7 @@ |
1148 | 1148 | } |
1149 | 1149 | |
1150 | 1150 | /* Update all the pointers */ |
1151 | - lp->tx_old = (++lp->tx_old) & lp->txRingMask; | |
1151 | + lp->tx_old = (lp->tx_old + 1) & lp->txRingMask; | |
1152 | 1152 | } |
1153 | 1153 | |
1154 | 1154 | return 0; |