Commit 3deb649e authored by Ben Nizette's avatar Ben Nizette Committed by Dmitry Torokhov

Input: ucb1400 - use disable_irq_nosync() in irq handler

disable_irq() waits for all running handlers to complete before
returning.  As such, if it's used to disable an interrupt from
that interrupt's handler it will deadlock.  This replaces the
dangerous instances with the _nosync() variant which doesn't have
this problem.
Signed-off-by: default avatarBen Nizette <bn@niasdigital.com>
Signed-off-by: default avatarDmitry Torokhov <dtor@mail.ru>
parent 29fa98bd
...@@ -256,7 +256,7 @@ static irqreturn_t ucb1400_hard_irq(int irqnr, void *devid) ...@@ -256,7 +256,7 @@ static irqreturn_t ucb1400_hard_irq(int irqnr, void *devid)
struct ucb1400_ts *ucb = devid; struct ucb1400_ts *ucb = devid;
if (irqnr == ucb->irq) { if (irqnr == ucb->irq) {
disable_irq(ucb->irq); disable_irq_nosync(ucb->irq);
ucb->irq_pending = 1; ucb->irq_pending = 1;
wake_up(&ucb->ts_wait); wake_up(&ucb->ts_wait);
return IRQ_HANDLED; return IRQ_HANDLED;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment