Commit f730f205 authored by Tony Lindgren's avatar Tony Lindgren Committed by Greg Kroah-Hartman

usb: musb: Split PM runtime between wrapper IP and musb core

Let's not tinker with the PM runtime of musb core from the omap2430
wrapper. This allows us to initialize PM runtime for musb core later
on instead of doing it in stages. And omap2430 wrapper has no need
to for accessing musb core at this point.

Note that this does not remove all the PM runtime calls from the
glue layer, those will get removed in a later patch.
Signed-off-by: default avatarTony Lindgren <tony@atomide.com>
Signed-off-by: default avatarBin Liu <b-liu@ti.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 7099dbc5
...@@ -2005,17 +2005,6 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl) ...@@ -2005,17 +2005,6 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl)
musb_readl = musb_default_readl; musb_readl = musb_default_readl;
musb_writel = musb_default_writel; musb_writel = musb_default_writel;
/*
* We need musb_read/write functions initialized for PM.
* Note that at least 2430 glue needs autosuspend delay
* somewhere above 300 ms for the hardware to idle properly
* after disconnecting the cable in host mode. Let's use
* 500 ms for some margin.
*/
pm_runtime_use_autosuspend(musb->controller);
pm_runtime_set_autosuspend_delay(musb->controller, 500);
pm_runtime_enable(musb->controller);
/* The musb_platform_init() call: /* The musb_platform_init() call:
* - adjusts musb->mregs * - adjusts musb->mregs
* - sets the musb->isr * - sets the musb->isr
...@@ -2117,6 +2106,16 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl) ...@@ -2117,6 +2106,16 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl)
if (musb->ops->phy_callback) if (musb->ops->phy_callback)
musb_phy_callback = musb->ops->phy_callback; musb_phy_callback = musb->ops->phy_callback;
/*
* We need musb_read/write functions initialized for PM.
* Note that at least 2430 glue needs autosuspend delay
* somewhere above 300 ms for the hardware to idle properly
* after disconnecting the cable in host mode. Let's use
* 500 ms for some margin.
*/
pm_runtime_use_autosuspend(musb->controller);
pm_runtime_set_autosuspend_delay(musb->controller, 500);
pm_runtime_enable(musb->controller);
pm_runtime_get_sync(musb->controller); pm_runtime_get_sync(musb->controller);
status = usb_phy_init(musb->xceiv); status = usb_phy_init(musb->xceiv);
...@@ -2251,6 +2250,7 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl) ...@@ -2251,6 +2250,7 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl)
err_usb_phy_init: err_usb_phy_init:
pm_runtime_dont_use_autosuspend(musb->controller); pm_runtime_dont_use_autosuspend(musb->controller);
pm_runtime_put_sync(musb->controller); pm_runtime_put_sync(musb->controller);
pm_runtime_disable(musb->controller);
fail2: fail2:
if (musb->irq_wake) if (musb->irq_wake)
...@@ -2258,7 +2258,6 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl) ...@@ -2258,7 +2258,6 @@ musb_init_controller(struct device *dev, int nIrq, void __iomem *ctrl)
musb_platform_exit(musb); musb_platform_exit(musb);
fail1: fail1:
pm_runtime_disable(musb->controller);
dev_err(musb->controller, dev_err(musb->controller,
"musb_init_controller failed with status %d\n", status); "musb_init_controller failed with status %d\n", status);
...@@ -2306,6 +2305,9 @@ static int musb_remove(struct platform_device *pdev) ...@@ -2306,6 +2305,9 @@ static int musb_remove(struct platform_device *pdev)
*/ */
musb_exit_debugfs(musb); musb_exit_debugfs(musb);
cancel_work_sync(&musb->irq_work);
cancel_delayed_work_sync(&musb->finish_resume_work);
cancel_delayed_work_sync(&musb->deassert_reset_work);
pm_runtime_get_sync(musb->controller); pm_runtime_get_sync(musb->controller);
musb_host_cleanup(musb); musb_host_cleanup(musb);
musb_gadget_cleanup(musb); musb_gadget_cleanup(musb);
...@@ -2314,21 +2316,14 @@ static int musb_remove(struct platform_device *pdev) ...@@ -2314,21 +2316,14 @@ static int musb_remove(struct platform_device *pdev)
musb_generic_disable(musb); musb_generic_disable(musb);
spin_unlock_irqrestore(&musb->lock, flags); spin_unlock_irqrestore(&musb->lock, flags);
musb_writeb(musb->mregs, MUSB_DEVCTL, 0); musb_writeb(musb->mregs, MUSB_DEVCTL, 0);
pm_runtime_dont_use_autosuspend(musb->controller);
pm_runtime_put_sync(musb->controller);
pm_runtime_disable(musb->controller);
musb_platform_exit(musb); musb_platform_exit(musb);
musb_phy_callback = NULL; musb_phy_callback = NULL;
if (musb->dma_controller) if (musb->dma_controller)
musb_dma_controller_destroy(musb->dma_controller); musb_dma_controller_destroy(musb->dma_controller);
usb_phy_shutdown(musb->xceiv); usb_phy_shutdown(musb->xceiv);
cancel_work_sync(&musb->irq_work);
cancel_delayed_work_sync(&musb->finish_resume_work);
cancel_delayed_work_sync(&musb->deassert_reset_work);
pm_runtime_dont_use_autosuspend(musb->controller);
pm_runtime_put_sync(musb->controller);
pm_runtime_disable(musb->controller);
musb_free(musb); musb_free(musb);
device_init_wakeup(dev, 0); device_init_wakeup(dev, 0);
return 0; return 0;
......
...@@ -400,13 +400,6 @@ static int omap2430_musb_init(struct musb *musb) ...@@ -400,13 +400,6 @@ static int omap2430_musb_init(struct musb *musb)
if (status < 0) if (status < 0)
goto err1; goto err1;
status = pm_runtime_get_sync(dev);
if (status < 0) {
dev_err(dev, "pm_runtime_get_sync FAILED %d\n", status);
pm_runtime_put_sync(glue->dev);
goto err1;
}
l = musb_readl(musb->mregs, OTG_INTERFSEL); l = musb_readl(musb->mregs, OTG_INTERFSEL);
if (data->interface_type == MUSB_INTERFACE_UTMI) { if (data->interface_type == MUSB_INTERFACE_UTMI) {
...@@ -434,9 +427,6 @@ static int omap2430_musb_init(struct musb *musb) ...@@ -434,9 +427,6 @@ static int omap2430_musb_init(struct musb *musb)
phy_init(musb->phy); phy_init(musb->phy);
phy_power_on(musb->phy); phy_power_on(musb->phy);
pm_runtime_mark_last_busy(musb->controller);
pm_runtime_put_autosuspend(musb->controller);
pm_runtime_put(glue->dev); pm_runtime_put(glue->dev);
return 0; return 0;
......
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