diff options
author | Mark Einon <mark.einon@gmail.com> | 2014-09-29 07:55:41 (GMT) |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-09-29 17:00:23 (GMT) |
commit | 644bbf95f5e2937f4929866e0d8899a9ed42f691 (patch) | |
tree | df759b78a6bdf96b40bdf2705b1ff693b8f76dbe /drivers | |
parent | ccb32b13c79331cc64d81a00f3cca335b7c26c29 (diff) | |
download | linux-644bbf95f5e2937f4929866e0d8899a9ed42f691.tar.xz |
staging: et131x: Simplify for loop in et131x_init_send()
The for loop in this function increments two variables and has an unusual
starting index of 1 (not 0). Make it look more familiar.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mark Einon <mark.einon@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/staging/et131x/et131x.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index 4973e64..384dc16 100644 --- a/drivers/staging/et131x/et131x.c +++ b/drivers/staging/et131x/et131x.c @@ -1751,7 +1751,7 @@ static void et131x_disable_txrx(struct net_device *netdev) static void et131x_init_send(struct et131x_adapter *adapter) { - u32 ct; + int i; struct tx_ring *tx_ring = &adapter->tx_ring; struct tcb *tcb = tx_ring->tcb_ring; @@ -1759,8 +1759,10 @@ static void et131x_init_send(struct et131x_adapter *adapter) memset(tcb, 0, sizeof(struct tcb) * NUM_TCB); - for (ct = 0; ct++ < NUM_TCB; tcb++) + for (i = 0; i < NUM_TCB; i++) { tcb->next = tcb + 1; + tcb++; + } tcb--; tx_ring->tcb_qtail = tcb; |