diff options
author | Rémi Denis-Courmont <remi.denis-courmont@nokia.com> | 2009-05-31 21:18:55 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2009-06-02 03:17:44 -0400 |
commit | fa202592796499dade5b3854643eaf0eed4d34ba (patch) | |
tree | 6064c17ffcc6e74ddd5f7db586226b16821d49f7 /drivers/usb | |
parent | bbd5898d39ab9f855c732f6f07e40e95b05cf52a (diff) |
f_phonet: dev_kfree_skb instead of dev_kfree_skb_any in TX callback
Network device TX is never run in IRQ context, and skb is freed outside
of the IRQ-disabling spin lock. So checking for IRQ was a waste of time
here.
Signed-off-by: Rémi Denis-Courmont <remi.denis-courmont@nokia.com>
Signed-off-by: Felipe Balbi <felipe.balbi@nokia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/usb')
-rw-r--r-- | drivers/usb/gadget/f_phonet.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/gadget/f_phonet.c b/drivers/usb/gadget/f_phonet.c index c1abeb89b413..54451ee02298 100644 --- a/drivers/usb/gadget/f_phonet.c +++ b/drivers/usb/gadget/f_phonet.c | |||
@@ -255,7 +255,7 @@ out_unlock: | |||
255 | spin_unlock_irqrestore(&port->lock, flags); | 255 | spin_unlock_irqrestore(&port->lock, flags); |
256 | out: | 256 | out: |
257 | if (unlikely(skb)) { | 257 | if (unlikely(skb)) { |
258 | dev_kfree_skb_any(skb); | 258 | dev_kfree_skb(skb); |
259 | dev->stats.tx_dropped++; | 259 | dev->stats.tx_dropped++; |
260 | } | 260 | } |
261 | return 0; | 261 | return 0; |