ixgbe: Drop unnecessary napi_schedule_prep and spare blank line from ixgbe_intr
authorAlexander Duyck <alexander.h.duyck@intel.com>
Wed, 8 Feb 2012 07:49:54 +0000 (07:49 +0000)
committerJeff Kirsher <jeffrey.t.kirsher@intel.com>
Tue, 13 Mar 2012 03:34:30 +0000 (20:34 -0700)
This patch is a minor cleanup to address the unnecessary use of
napi_schedule_prep in ixgbe_intr and to also remove a blank line that is
not needed since it is separating a comment from the line it is explaining.

Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>
Tested-by: Stephen Ko <stephen.s.ko@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c

index 80ae91902aab2d061e1bcb36f56cd97118877422..b1f53eda8e3fe5674439b9dcdbc2671d0aba006b 100644 (file)
@@ -2401,16 +2401,13 @@ static irqreturn_t ixgbe_intr(int irq, void *data)
 
        ixgbe_check_fan_failure(adapter, eicr);
 
-       if (napi_schedule_prep(&(q_vector->napi))) {
-               /* would disable interrupts here but EIAM disabled it */
-               __napi_schedule(&(q_vector->napi));
-       }
+       /* would disable interrupts here but EIAM disabled it */
+       napi_schedule(&q_vector->napi);
 
        /*
         * re-enable link(maybe) and non-queue interrupts, no flush.
         * ixgbe_poll will re-enable the queue interrupts
         */
-
        if (!test_bit(__IXGBE_DOWN, &adapter->state))
                ixgbe_irq_enable(adapter, false, false);