From: Mark Einon Date: Sun, 14 Sep 2014 15:59:07 +0000 (+0100) Subject: staging: et131x: Use braces on all arms of if/else statements X-Git-Tag: v5.15~17125^2~394 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ee60c8ec323167a02de357e9d9b44af850052ee3;p=platform%2Fkernel%2Flinux-starfive.git 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 --- 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;