libertas: clean up lbs_interrupt()
Make it take struct lbs_private as argument; that's all it wants anyway, and all callers were starting off from that. Don't wake the netif queues, because those should be handled elsewhere. And sort out the locking, with a big nasty warning for those who don't have the driver_lock locked when they call it. Oh, and fix if_cs.c to lock the driver_lock before calling it. Signed-off-by: David Woodhouse <dwmw2@infradead.org> Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
committed by
David S. Miller
parent
1309b55b4d
commit
4f67949656
@@ -632,7 +632,7 @@ static inline void process_cmdrequest(int recvlength, u8 *recvbuff,
|
||||
priv->upld_len);
|
||||
|
||||
kfree_skb(skb);
|
||||
lbs_interrupt(priv->dev);
|
||||
lbs_interrupt(priv);
|
||||
spin_unlock(&priv->driver_lock);
|
||||
|
||||
lbs_deb_usbd(&cardp->udev->dev,
|
||||
@@ -705,7 +705,7 @@ static void if_usb_receive(struct urb *urb)
|
||||
cardp->usb_event_cause <<= 3;
|
||||
cardp->usb_int_cause |= MRVDRV_CARDEVENT;
|
||||
kfree_skb(skb);
|
||||
lbs_interrupt(priv->dev);
|
||||
lbs_interrupt(priv);
|
||||
spin_unlock(&priv->driver_lock);
|
||||
goto rx_exit;
|
||||
default:
|
||||
|
Reference in New Issue
Block a user