diff options
| author | Peter Chen <peter.chen@freescale.com> | 2013-03-04 17:30:08 +0800 |
|---|---|---|
| committer | Peter Chen <peter.chen@freescale.com> | 2013-03-06 09:21:33 +0800 |
| commit | 5c554bfc4bc3b0908ce8165c18234eccb70ad722 (patch) | |
| tree | 66dc8ae25b3109f73b9265d5f0d97361c0bb0dc9 /drivers/usb | |
| parent | f6ec2b094d2ed5ee03488f031a93c45da86748eb (diff) | |
ENGR00251209-5 usb: fix the system hang when unload gadget module
- For gadget, the clock will be closed at platform code, so
the driver should not be close clock beforehand.
- For host, the PHY should not be low power mode when the host
controller has still possibilities to use. Besides, the 5v should
be off at last as some platform uses 5v for PHY's power.
Signed-off-by: Peter Chen <peter.chen@freescale.com>
Diffstat (limited to 'drivers/usb')
| -rwxr-xr-x | drivers/usb/gadget/arcotg_udc.c | 1 | ||||
| -rwxr-xr-x | drivers/usb/host/ehci-arc.c | 9 |
2 files changed, 5 insertions, 5 deletions
diff --git a/drivers/usb/gadget/arcotg_udc.c b/drivers/usb/gadget/arcotg_udc.c index d1a2e27bcd56..9d9417f6cce4 100755 --- a/drivers/usb/gadget/arcotg_udc.c +++ b/drivers/usb/gadget/arcotg_udc.c @@ -3299,7 +3299,6 @@ static int fsl_udc_remove(struct platform_device *pdev) release_mem_region(res->start, resource_size(res)); } #endif - dr_clk_gate(false); device_unregister(&udc_controller->gadget.dev); /* free udc --wait for the release() finished */ wait_for_completion(&done); diff --git a/drivers/usb/host/ehci-arc.c b/drivers/usb/host/ehci-arc.c index 4a2768a65c2c..4a7c606f23fd 100755 --- a/drivers/usb/host/ehci-arc.c +++ b/drivers/usb/host/ehci-arc.c @@ -350,9 +350,6 @@ static void usb_hcd_fsl_remove(struct usb_hcd *hcd, } } - /* DDD shouldn't we turn off the power here? */ - fsl_platform_set_vbus_power(pdata, 0); - if (ehci->transceiver) { (void)otg_set_host(ehci->transceiver, 0); otg_put_transceiver(ehci->transceiver); @@ -361,12 +358,16 @@ static void usb_hcd_fsl_remove(struct usb_hcd *hcd, } /*disable the host wakeup and put phy to low power mode */ usb_host_set_wakeup(hcd->self.controller, false); - fsl_usb_lowpower_mode(pdata, true); /*free the ehci_fsl_pre_irq */ free_irq(hcd->irq, (void *)pdev); usb_remove_hcd(hcd); usb_put_hcd(hcd); + fsl_usb_lowpower_mode(pdata, true); + + /* DDD shouldn't we turn off the power here? */ + fsl_platform_set_vbus_power(pdata, 0); + /* * do platform specific un-initialization: * release iomux pins clocks, etc. |