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 /drivers/net/skge.c | |
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>
Diffstat (limited to 'drivers/net/skge.c')
-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 | ||