diff options
author | Patrick McHardy <kaber@trash.net> | 2009-06-12 00:08:02 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-06-13 04:18:37 -0400 |
commit | 4153e77596c4caaf52293b0c6b2207d73ed8f1eb (patch) | |
tree | 5920561315cfea1b89e287d264fdac6314c670ec /drivers/net/wireless/zd1201.c | |
parent | 4bd73ae2682d9069746bb049a416d9ab90c6684b (diff) |
net: fix network drivers ndo_start_xmit() return values (part 3)
net: fix network drivers ndo_start_xmit() return values (part 3)
Fix up wireless drivers that return an errno value to qdisc_restart(), causing
qdisc_restart() to print a warning an requeue/retransmit the skb.
- airo: transmission not implemented for chip, intention is to free and abort
- ipw2200: transmission not implemented for promiscous mode, intention is to
drop
- prism54: intention is to drop
- wl3501_cs: intention appears to be to drop
- zd1201: error counter indicates intention is to drop
All drivers compile tested.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wireless/zd1201.c')
-rw-r--r-- | drivers/net/wireless/zd1201.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/net/wireless/zd1201.c b/drivers/net/wireless/zd1201.c index 5fabd9c0f07a..4430b8d92e21 100644 --- a/drivers/net/wireless/zd1201.c +++ b/drivers/net/wireless/zd1201.c | |||
@@ -819,11 +819,11 @@ static int zd1201_hard_start_xmit(struct sk_buff *skb, struct net_device *dev) | |||
819 | if (err) { | 819 | if (err) { |
820 | dev->stats.tx_errors++; | 820 | dev->stats.tx_errors++; |
821 | netif_start_queue(dev); | 821 | netif_start_queue(dev); |
822 | return err; | 822 | } else { |
823 | dev->stats.tx_packets++; | ||
824 | dev->stats.tx_bytes += skb->len; | ||
825 | dev->trans_start = jiffies; | ||
823 | } | 826 | } |
824 | dev->stats.tx_packets++; | ||
825 | dev->stats.tx_bytes += skb->len; | ||
826 | dev->trans_start = jiffies; | ||
827 | kfree_skb(skb); | 827 | kfree_skb(skb); |
828 | 828 | ||
829 | return 0; | 829 | return 0; |