hisilicon: add some missing curly braces
authorDan Carpenter <dan.carpenter@oracle.com>
Wed, 28 Jan 2015 18:58:33 +0000 (21:58 +0300)
committerDavid S. Miller <davem@davemloft.net>
Thu, 29 Jan 2015 07:06:24 +0000 (23:06 -0800)
The if block was supposed to have curly braces.  In the current code we
complain about dropped rx packets when we shouldn't.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/hisilicon/hip04_eth.c

index 525214ef59845f722a8f4bee3767fa0cf42e79dd..c02b81bcfffbe8b6de91fcb53e94acb1f1b42882 100644 (file)
@@ -567,10 +567,11 @@ static irqreturn_t hip04_mac_interrupt(int irq, void *dev_id)
        writel_relaxed(DEF_INT_MASK, priv->base + PPE_RINT);
 
        if (unlikely(ists & DEF_INT_ERR)) {
-               if (ists & (RCV_NOBUF | RCV_DROP))
+               if (ists & (RCV_NOBUF | RCV_DROP)) {
                        stats->rx_errors++;
                        stats->rx_dropped++;
                        netdev_err(ndev, "rx drop\n");
+               }
                if (ists & TX_DROP) {
                        stats->tx_dropped++;
                        netdev_err(ndev, "tx drop\n");