diff options
| author | Stephen Hemminger <shemminger@osdl.org> | 2006-03-23 14:07:25 -0500 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2006-03-23 17:13:54 -0500 |
| commit | 5a01144717ee9dacd45e1b0861a2c593796bcead (patch) | |
| tree | ca401ae28092ee83c943c3b5e4a538803c81edd4 | |
| parent | b5d56ddc3f839e94e97a3eb8afb4d0d64a9f2ef8 (diff) | |
[PATCH] skge: rx_reuse called twice
In the error case we call skge_rx_reuse twice. This is harmless
but unnecessary.
Signed-off-by: Stephen Hemminger <shemminger@osdl.org>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
| -rw-r--r-- | drivers/net/skge.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/net/skge.c b/drivers/net/skge.c index b0aa150bbd8a..72148f0fbfcc 100644 --- a/drivers/net/skge.c +++ b/drivers/net/skge.c | |||
| @@ -781,7 +781,7 @@ static void skge_rx_setup(struct skge_port *skge, struct skge_element *e, | |||
| 781 | * Note: DMA address is not changed by chip. | 781 | * Note: DMA address is not changed by chip. |
| 782 | * MTU not changed while receiver active. | 782 | * MTU not changed while receiver active. |
| 783 | */ | 783 | */ |
| 784 | static void skge_rx_reuse(struct skge_element *e, unsigned int size) | 784 | static inline void skge_rx_reuse(struct skge_element *e, unsigned int size) |
| 785 | { | 785 | { |
| 786 | struct skge_rx_desc *rd = e->desc; | 786 | struct skge_rx_desc *rd = e->desc; |
| 787 | 787 | ||
| @@ -2719,8 +2719,7 @@ static int skge_poll(struct net_device *dev, int *budget) | |||
| 2719 | netif_receive_skb(skb); | 2719 | netif_receive_skb(skb); |
| 2720 | 2720 | ||
| 2721 | ++work_done; | 2721 | ++work_done; |
| 2722 | } else | 2722 | } |
| 2723 | skge_rx_reuse(e, skge->rx_buf_size); | ||
| 2724 | } | 2723 | } |
| 2725 | ring->to_clean = e; | 2724 | ring->to_clean = e; |
| 2726 | 2725 | ||
