diff options
author | Ben Hutchings <bhutchings@solarflare.com> | 2009-01-19 19:43:59 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-01-21 17:33:50 -0500 |
commit | 288379f050284087578b77e04f040b57db3db3f8 (patch) | |
tree | ac5f5c83e2778a1966327d87316fc94067363b45 /drivers/net/tg3.c | |
parent | 627af770c63acddc2402dd19fec70df5c3ad8ab7 (diff) |
net: Remove redundant NAPI functions
Following the removal of the unused struct net_device * parameter from
the NAPI functions named *netif_rx_* in commit 908a7a1, they are
exactly equivalent to the corresponding *napi_* functions and are
therefore redundant.
Signed-off-by: Ben Hutchings <bhutchings@solarflare.com>
Acked-by: Neil Horman <nhorman@tuxdriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/tg3.c')
-rw-r--r-- | drivers/net/tg3.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/drivers/net/tg3.c b/drivers/net/tg3.c index 8b3f84685387..5fa65acb68e5 100644 --- a/drivers/net/tg3.c +++ b/drivers/net/tg3.c | |||
@@ -4460,7 +4460,7 @@ static int tg3_poll(struct napi_struct *napi, int budget) | |||
4460 | sblk->status &= ~SD_STATUS_UPDATED; | 4460 | sblk->status &= ~SD_STATUS_UPDATED; |
4461 | 4461 | ||
4462 | if (likely(!tg3_has_work(tp))) { | 4462 | if (likely(!tg3_has_work(tp))) { |
4463 | netif_rx_complete(napi); | 4463 | napi_complete(napi); |
4464 | tg3_restart_ints(tp); | 4464 | tg3_restart_ints(tp); |
4465 | break; | 4465 | break; |
4466 | } | 4466 | } |
@@ -4470,7 +4470,7 @@ static int tg3_poll(struct napi_struct *napi, int budget) | |||
4470 | 4470 | ||
4471 | tx_recovery: | 4471 | tx_recovery: |
4472 | /* work_done is guaranteed to be less than budget. */ | 4472 | /* work_done is guaranteed to be less than budget. */ |
4473 | netif_rx_complete(napi); | 4473 | napi_complete(napi); |
4474 | schedule_work(&tp->reset_task); | 4474 | schedule_work(&tp->reset_task); |
4475 | return work_done; | 4475 | return work_done; |
4476 | } | 4476 | } |
@@ -4519,7 +4519,7 @@ static irqreturn_t tg3_msi_1shot(int irq, void *dev_id) | |||
4519 | prefetch(&tp->rx_rcb[tp->rx_rcb_ptr]); | 4519 | prefetch(&tp->rx_rcb[tp->rx_rcb_ptr]); |
4520 | 4520 | ||
4521 | if (likely(!tg3_irq_sync(tp))) | 4521 | if (likely(!tg3_irq_sync(tp))) |
4522 | netif_rx_schedule(&tp->napi); | 4522 | napi_schedule(&tp->napi); |
4523 | 4523 | ||
4524 | return IRQ_HANDLED; | 4524 | return IRQ_HANDLED; |
4525 | } | 4525 | } |
@@ -4544,7 +4544,7 @@ static irqreturn_t tg3_msi(int irq, void *dev_id) | |||
4544 | */ | 4544 | */ |
4545 | tw32_mailbox(MAILBOX_INTERRUPT_0 + TG3_64BIT_REG_LOW, 0x00000001); | 4545 | tw32_mailbox(MAILBOX_INTERRUPT_0 + TG3_64BIT_REG_LOW, 0x00000001); |
4546 | if (likely(!tg3_irq_sync(tp))) | 4546 | if (likely(!tg3_irq_sync(tp))) |
4547 | netif_rx_schedule(&tp->napi); | 4547 | napi_schedule(&tp->napi); |
4548 | 4548 | ||
4549 | return IRQ_RETVAL(1); | 4549 | return IRQ_RETVAL(1); |
4550 | } | 4550 | } |
@@ -4586,7 +4586,7 @@ static irqreturn_t tg3_interrupt(int irq, void *dev_id) | |||
4586 | sblk->status &= ~SD_STATUS_UPDATED; | 4586 | sblk->status &= ~SD_STATUS_UPDATED; |
4587 | if (likely(tg3_has_work(tp))) { | 4587 | if (likely(tg3_has_work(tp))) { |
4588 | prefetch(&tp->rx_rcb[tp->rx_rcb_ptr]); | 4588 | prefetch(&tp->rx_rcb[tp->rx_rcb_ptr]); |
4589 | netif_rx_schedule(&tp->napi); | 4589 | napi_schedule(&tp->napi); |
4590 | } else { | 4590 | } else { |
4591 | /* No work, shared interrupt perhaps? re-enable | 4591 | /* No work, shared interrupt perhaps? re-enable |
4592 | * interrupts, and flush that PCI write | 4592 | * interrupts, and flush that PCI write |
@@ -4632,7 +4632,7 @@ static irqreturn_t tg3_interrupt_tagged(int irq, void *dev_id) | |||
4632 | tw32_mailbox_f(MAILBOX_INTERRUPT_0 + TG3_64BIT_REG_LOW, 0x00000001); | 4632 | tw32_mailbox_f(MAILBOX_INTERRUPT_0 + TG3_64BIT_REG_LOW, 0x00000001); |
4633 | if (tg3_irq_sync(tp)) | 4633 | if (tg3_irq_sync(tp)) |
4634 | goto out; | 4634 | goto out; |
4635 | if (netif_rx_schedule_prep(&tp->napi)) { | 4635 | if (napi_schedule_prep(&tp->napi)) { |
4636 | prefetch(&tp->rx_rcb[tp->rx_rcb_ptr]); | 4636 | prefetch(&tp->rx_rcb[tp->rx_rcb_ptr]); |
4637 | /* Update last_tag to mark that this status has been | 4637 | /* Update last_tag to mark that this status has been |
4638 | * seen. Because interrupt may be shared, we may be | 4638 | * seen. Because interrupt may be shared, we may be |
@@ -4640,7 +4640,7 @@ static irqreturn_t tg3_interrupt_tagged(int irq, void *dev_id) | |||
4640 | * if tg3_poll() is not scheduled. | 4640 | * if tg3_poll() is not scheduled. |
4641 | */ | 4641 | */ |
4642 | tp->last_tag = sblk->status_tag; | 4642 | tp->last_tag = sblk->status_tag; |
4643 | __netif_rx_schedule(&tp->napi); | 4643 | __napi_schedule(&tp->napi); |
4644 | } | 4644 | } |
4645 | out: | 4645 | out: |
4646 | return IRQ_RETVAL(handled); | 4646 | return IRQ_RETVAL(handled); |