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/airo.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/airo.c')
-rw-r--r-- | drivers/net/wireless/airo.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/wireless/airo.c b/drivers/net/wireless/airo.c index 9eabf4d1f2e7..dfb30b9c4267 100644 --- a/drivers/net/wireless/airo.c +++ b/drivers/net/wireless/airo.c | |||
@@ -2193,7 +2193,8 @@ static int airo_start_xmit11(struct sk_buff *skb, struct net_device *dev) { | |||
2193 | if (test_bit(FLAG_MPI, &priv->flags)) { | 2193 | if (test_bit(FLAG_MPI, &priv->flags)) { |
2194 | /* Not implemented yet for MPI350 */ | 2194 | /* Not implemented yet for MPI350 */ |
2195 | netif_stop_queue(dev); | 2195 | netif_stop_queue(dev); |
2196 | return -ENETDOWN; | 2196 | dev_kfree_skb_any(skb); |
2197 | return NETDEV_TX_OK; | ||
2197 | } | 2198 | } |
2198 | 2199 | ||
2199 | if ( skb == NULL ) { | 2200 | if ( skb == NULL ) { |