Commit 73926db3 authored by Daniel Mack's avatar Daniel Mack Committed by Greg Kroah-Hartman

usb: musb: do not sleep in atomic context

musb_port_reset() is called from musb_hub_control() which in turn holds
a spinlock, so musb_port_reset() is not allowed to call msleep().

With the asynchronous work helpers in place, this is fortunately easy to
fix by rescheduling the reset deassertion function to after the time
when the wait period is finished.

Note, however, that the MUSB_POWER_RESUME bit is only set on AM33xx
processors under rare conditions such as when to another driver
reporting an error during suspend. Hence, this didn't hit me yet in
normal operation.
Signed-off-by: default avatarDaniel Mack <zonque@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 12df84d4
...@@ -477,6 +477,8 @@ static irqreturn_t musb_stage0_irq(struct musb *musb, u8 int_usb, ...@@ -477,6 +477,8 @@ static irqreturn_t musb_stage0_irq(struct musb *musb, u8 int_usb,
musb->port1_status |= musb->port1_status |=
(USB_PORT_STAT_C_SUSPEND << 16) (USB_PORT_STAT_C_SUSPEND << 16)
| MUSB_PORT_STAT_RESUME; | MUSB_PORT_STAT_RESUME;
musb->rh_timer = jiffies
+ msecs_to_jiffies(20);
schedule_delayed_work( schedule_delayed_work(
&musb->finish_resume_work, 20); &musb->finish_resume_work, 20);
......
...@@ -158,7 +158,6 @@ void musb_port_reset(struct musb *musb, bool do_reset) ...@@ -158,7 +158,6 @@ void musb_port_reset(struct musb *musb, bool do_reset)
*/ */
power = musb_readb(mbase, MUSB_POWER); power = musb_readb(mbase, MUSB_POWER);
if (do_reset) { if (do_reset) {
/* /*
* If RESUME is set, we must make sure it stays minimum 20 ms. * If RESUME is set, we must make sure it stays minimum 20 ms.
* Then we must clear RESUME and wait a bit to let musb start * Then we must clear RESUME and wait a bit to let musb start
...@@ -167,11 +166,22 @@ void musb_port_reset(struct musb *musb, bool do_reset) ...@@ -167,11 +166,22 @@ void musb_port_reset(struct musb *musb, bool do_reset)
* detected". * detected".
*/ */
if (power & MUSB_POWER_RESUME) { if (power & MUSB_POWER_RESUME) {
while (time_before(jiffies, musb->rh_timer)) long remain = (unsigned long) musb->rh_timer - jiffies;
msleep(1);
if (musb->rh_timer > 0 && remain > 0) {
/* take into account the minimum delay after resume */
schedule_delayed_work(
&musb->deassert_reset_work,
jiffies_to_msecs(remain));
return;
}
musb_writeb(mbase, MUSB_POWER, musb_writeb(mbase, MUSB_POWER,
power & ~MUSB_POWER_RESUME); power & ~MUSB_POWER_RESUME);
msleep(1);
/* Give the core 1 ms to clear MUSB_POWER_RESUME */
schedule_delayed_work(&musb->deassert_reset_work, 1);
return;
} }
power &= 0xf0; power &= 0xf0;
......
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