From ee60c8ec323167a02de357e9d9b44af850052ee3 Mon Sep 17 00:00:00 2001 From: Mark Einon Date: Sun, 14 Sep 2014 16:59:07 +0100 Subject: [PATCH] staging: et131x: Use braces on all arms of if/else statements In some places in et131x.c, one arm of am if/else statement has braces and the other not - put braces on both arms where this happens. Signed-off-by: Mark Einon Signed-off-by: Greg Kroah-Hartman --- drivers/staging/et131x/et131x.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/staging/et131x/et131x.c b/drivers/staging/et131x/et131x.c index 12bde76..3f40247 100644 --- a/drivers/staging/et131x/et131x.c +++ b/drivers/staging/et131x/et131x.c @@ -2742,8 +2742,9 @@ static int nic_send_packet(struct et131x_adapter *adapter, struct tcb *tcb) tcb->index = NUM_DESC_PER_RING_TX - 1; else tcb->index = ET_DMA10_WRAP|(NUM_DESC_PER_RING_TX - 1); - } else + } else { tcb->index = tx_ring->send_idx - 1; + } spin_lock(&adapter->tcb_send_qlock); @@ -4021,8 +4022,9 @@ static int et131x_set_packet_filter(struct et131x_adapter *adapter) if (filter & ET131X_PACKET_TYPE_BROADCAST) { pf_ctrl |= 1; /* Broadcast filter bit */ ctrl &= ~0x04; - } else + } else { pf_ctrl &= ~1; + } /* Setup the receive mac configuration registers - Packet * Filter control + the enable / disable for packet filter @@ -4070,8 +4072,9 @@ static void et131x_multicast(struct net_device *netdev) if (netdev_mc_count(netdev) < 1) { adapter->packet_filter &= ~ET131X_PACKET_TYPE_ALL_MULTICAST; adapter->packet_filter &= ~ET131X_PACKET_TYPE_MULTICAST; - } else + } else { adapter->packet_filter |= ET131X_PACKET_TYPE_MULTICAST; + } /* Set values in the private adapter struct */ i = 0; -- 2.7.4