The 'if' blocks have braces, so the 'else' blocks should too. Signed-off-by: Mark Einon <mark.einon@xxxxxxxxx> --- drivers/staging/et131x/et131x.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index a000581..a219ba3 100644 --- a/drivers/staging/et131x/et131x.c +++ b/drivers/staging/et131x/et131x.c @@ -2494,9 +2494,10 @@ static int et131x_handle_recv_pkts(struct et131x_adapter *adapter, int budget) rx_ring->unfinished_receives = true; writel(PARM_TX_TIME_INT_DEF * NANO_IN_A_MICRO, &adapter->regs->global.watchdog_timer); - } else + } else { /* Watchdog timer will disable itself if appropriate. */ rx_ring->unfinished_receives = false; + } return count; } @@ -2679,9 +2680,10 @@ static int nic_send_packet(struct et131x_adapter *adapter, struct tcb *tcb) } else { /* Last element */ desc[frag - 1].flags = TXDESC_FLAG_LASTPKT; } - } else + } else { desc[frag - 1].flags = TXDESC_FLAG_INTPROC | TXDESC_FLAG_LASTPKT; + } desc[0].flags |= TXDESC_FLAG_FIRSTPKT; -- 2.1.0 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel