schedule_delayed_work() takes the delay in jiffies, not msecs.

This bug slipped in with the recent reset logic cleanup
(8ed1fb790ea: "usb: musb: finish suspend/reset work independently from
musb_hub_control()").

Signed-off-by: Daniel Mack <dan...@zonque.org>
---
 drivers/usb/musb/musb_core.c    |  3 ++-
 drivers/usb/musb/musb_virthub.c | 12 +++++++-----
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/drivers/usb/musb/musb_core.c b/drivers/usb/musb/musb_core.c
index c147d66..d55350e 100644
--- a/drivers/usb/musb/musb_core.c
+++ b/drivers/usb/musb/musb_core.c
@@ -481,7 +481,8 @@ static irqreturn_t musb_stage0_irq(struct musb *musb, u8 
int_usb,
                                musb->rh_timer = jiffies
                                                 + msecs_to_jiffies(20);
                                schedule_delayed_work(
-                                       &musb->finish_resume_work, 20);
+                                       &musb->finish_resume_work,
+                                       msecs_to_jiffies(20));
 
                                musb->xceiv->state = OTG_STATE_A_HOST;
                                musb->is_active = 1;
diff --git a/drivers/usb/musb/musb_virthub.c b/drivers/usb/musb/musb_virthub.c
index 64a4ee7..5780254 100644
--- a/drivers/usb/musb/musb_virthub.c
+++ b/drivers/usb/musb/musb_virthub.c
@@ -136,7 +136,8 @@ void musb_port_suspend(struct musb *musb, bool do_suspend)
 
                /* later, GetPortStatus will stop RESUME signaling */
                musb->port1_status |= MUSB_PORT_STAT_RESUME;
-               schedule_delayed_work(&musb->finish_resume_work, 20);
+               schedule_delayed_work(&musb->finish_resume_work,
+                                     msecs_to_jiffies(20));
        }
 }
 
@@ -172,8 +173,7 @@ void musb_port_reset(struct musb *musb, bool do_reset)
                        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));
+                                       &musb->deassert_reset_work, remain);
                                return;
                        }
 
@@ -181,7 +181,8 @@ void musb_port_reset(struct musb *musb, bool do_reset)
                                    power & ~MUSB_POWER_RESUME);
 
                        /* Give the core 1 ms to clear MUSB_POWER_RESUME */
-                       schedule_delayed_work(&musb->deassert_reset_work, 1);
+                       schedule_delayed_work(&musb->deassert_reset_work,
+                                             msecs_to_jiffies(1));
                        return;
                }
 
@@ -191,7 +192,8 @@ void musb_port_reset(struct musb *musb, bool do_reset)
 
                musb->port1_status |= USB_PORT_STAT_RESET;
                musb->port1_status &= ~USB_PORT_STAT_ENABLE;
-               schedule_delayed_work(&musb->deassert_reset_work, 50);
+               schedule_delayed_work(&musb->deassert_reset_work,
+                                     msecs_to_jiffies(50));
        } else {
                dev_dbg(musb->controller, "root port reset stopped\n");
                musb_writeb(mbase, MUSB_POWER,
-- 
1.8.5.3

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to