diff options
author | Roger Quadros <rogerq@ti.com> | 2015-09-21 11:08:36 +0300 |
---|---|---|
committer | Felipe Balbi <balbi@ti.com> | 2015-09-21 14:42:36 -0500 |
commit | a66c275b3d5d8467d770dabd30927f5d5e857294 (patch) | |
tree | 7745302518c1cff3f44ac773300e054fcfb3bd51 /drivers/usb | |
parent | b431ba8803666e56c1d178a421b3cbc36e8d3d33 (diff) | |
download | lwn-a66c275b3d5d8467d770dabd30927f5d5e857294.tar.gz lwn-a66c275b3d5d8467d770dabd30927f5d5e857294.zip |
usb: dwc3: gadget: Fix BUG in RT config
Using spin_lock() in hard irq handler is pointless
and causes a BUG() in RT (real-time) configuration
so get rid of it.
The reason it's pointless is because the driver is
basically accessing register which is, anyways,
atomic.
Signed-off-by: Roger Quadros <rogerq@ti.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
Diffstat (limited to 'drivers/usb')
-rw-r--r-- | drivers/usb/dwc3/gadget.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c index 0c25704dcb6b..1e8bdf817811 100644 --- a/drivers/usb/dwc3/gadget.c +++ b/drivers/usb/dwc3/gadget.c @@ -2665,8 +2665,6 @@ static irqreturn_t dwc3_interrupt(int irq, void *_dwc) int i; irqreturn_t ret = IRQ_NONE; - spin_lock(&dwc->lock); - for (i = 0; i < dwc->num_event_buffers; i++) { irqreturn_t status; @@ -2675,8 +2673,6 @@ static irqreturn_t dwc3_interrupt(int irq, void *_dwc) ret = status; } - spin_unlock(&dwc->lock); - return ret; } |