From: Alexander Duyck Date: Wed, 8 Feb 2012 07:49:54 +0000 (+0000) Subject: ixgbe: Drop unnecessary napi_schedule_prep and spare blank line from ixgbe_intr X-Git-Tag: v3.4-rc1~177^2~64 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b9f6ed2bab0b087bdff870b7173f1d89efd96e54;p=platform%2Fupstream%2Fkernel-adaptation-pc.git ixgbe: Drop unnecessary napi_schedule_prep and spare blank line from ixgbe_intr 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 Tested-by: Stephen Ko Signed-off-by: Jeff Kirsher --- diff --git a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c index 80ae919..b1f53ed 100644 --- a/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c +++ b/drivers/net/ethernet/intel/ixgbe/ixgbe_main.c @@ -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);