atl1: handle rx in separate condition
authorTony Zelenoff <antonz@parallels.com>
Fri, 13 Apr 2012 06:09:46 +0000 (06:09 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sun, 15 Apr 2012 16:54:15 +0000 (12:54 -0400)
Remove rx from unlikely optimization in case of rx is very
likely thing for network card. This also reduce code a bit.

Signed-off-by: Tony Zelenoff <antonz@parallels.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/atheros/atlx/atl1.c

index 40ac414..c631528 100644 (file)
@@ -2502,18 +2502,19 @@ static irqreturn_t atl1_intr(int irq, void *data)
                if (status & ISR_CMB_TX)
                        atl1_intr_tx(adapter);
 
+               /* rx event */
+               if (status & ISR_CMB_RX)
+                       alt1_intr_rx(adapter);
+
                /* rx exception */
                if (unlikely(status & (ISR_RXF_OV | ISR_RFD_UNRUN |
                        ISR_RRD_OV | ISR_HOST_RFD_UNRUN |
-                       ISR_HOST_RRD_OV | ISR_CMB_RX))) {
-                       if (status & (ISR_RXF_OV | ISR_RFD_UNRUN |
-                               ISR_RRD_OV | ISR_HOST_RFD_UNRUN |
-                               ISR_HOST_RRD_OV))
-                               if (netif_msg_intr(adapter))
-                                       dev_printk(KERN_DEBUG,
-                                               &adapter->pdev->dev,
-                                               "rx exception, ISR = 0x%x\n",
-                                               status);
+                       ISR_HOST_RRD_OV))) {
+                       if (netif_msg_intr(adapter))
+                               dev_printk(KERN_DEBUG,
+                                       &adapter->pdev->dev,
+                                       "rx exception, ISR = 0x%x\n",
+                                       status);
                        atl1_intr_rx(adapter);
                }