Commit a40a0031 authored by Heiko Stübner's avatar Heiko Stübner Committed by Felipe Balbi

usb: dwc2: add shutdown callback to platform variant

In specific conditions (involving usb hubs) dwc2 devices can create a
lot of interrupts, even to the point of overwhelming devices running
at low frequencies. Some devices need to do special clock handling
at shutdown-time which may bring the system clock below the threshold
of being able to handle the dwc2 interrupts. Disabling dwc2-irqs
in a shutdown callbacks prevents reboots/poweroffs from getting stuck
in such cases.

The hsotg struct already contains an unused irq element, so we can
just use it to store the irq number for the shutdown callback.
Reviewed-by: default avatarDouglas Anderson <dianders@chromium.org>
Acked-by: default avatarJohn Youn <johnyoun@synopsys.com>
Signed-off-by: default avatarHeiko Stuebner <heiko.stuebner@collabora.com>
Signed-off-by: default avatarFelipe Balbi <balbi@ti.com>
parent ea4a8cb1
...@@ -405,6 +405,25 @@ static int dwc2_driver_remove(struct platform_device *dev) ...@@ -405,6 +405,25 @@ static int dwc2_driver_remove(struct platform_device *dev)
return 0; return 0;
} }
/**
* dwc2_driver_shutdown() - Called on device shutdown
*
* @dev: Platform device
*
* In specific conditions (involving usb hubs) dwc2 devices can create a
* lot of interrupts, even to the point of overwhelming devices running
* at low frequencies. Some devices need to do special clock handling
* at shutdown-time which may bring the system clock below the threshold
* of being able to handle the dwc2 interrupts. Disabling dwc2-irqs
* prevents reboots/poweroffs from getting stuck in such cases.
*/
static void dwc2_driver_shutdown(struct platform_device *dev)
{
struct dwc2_hsotg *hsotg = platform_get_drvdata(dev);
disable_irq(hsotg->irq);
}
static const struct of_device_id dwc2_of_match_table[] = { static const struct of_device_id dwc2_of_match_table[] = {
{ .compatible = "brcm,bcm2835-usb", .data = &params_bcm2835 }, { .compatible = "brcm,bcm2835-usb", .data = &params_bcm2835 },
{ .compatible = "hisilicon,hi6220-usb", .data = &params_hi6220 }, { .compatible = "hisilicon,hi6220-usb", .data = &params_hi6220 },
...@@ -435,7 +454,6 @@ static int dwc2_driver_probe(struct platform_device *dev) ...@@ -435,7 +454,6 @@ static int dwc2_driver_probe(struct platform_device *dev)
struct dwc2_hsotg *hsotg; struct dwc2_hsotg *hsotg;
struct resource *res; struct resource *res;
int retval; int retval;
int irq;
match = of_match_device(dwc2_of_match_table, &dev->dev); match = of_match_device(dwc2_of_match_table, &dev->dev);
if (match && match->data) { if (match && match->data) {
...@@ -490,15 +508,15 @@ static int dwc2_driver_probe(struct platform_device *dev) ...@@ -490,15 +508,15 @@ static int dwc2_driver_probe(struct platform_device *dev)
dwc2_set_all_params(hsotg->core_params, -1); dwc2_set_all_params(hsotg->core_params, -1);
irq = platform_get_irq(dev, 0); hsotg->irq = platform_get_irq(dev, 0);
if (irq < 0) { if (hsotg->irq < 0) {
dev_err(&dev->dev, "missing IRQ resource\n"); dev_err(&dev->dev, "missing IRQ resource\n");
return irq; return hsotg->irq;
} }
dev_dbg(hsotg->dev, "registering common handler for irq%d\n", dev_dbg(hsotg->dev, "registering common handler for irq%d\n",
irq); hsotg->irq);
retval = devm_request_irq(hsotg->dev, irq, retval = devm_request_irq(hsotg->dev, hsotg->irq,
dwc2_handle_common_intr, IRQF_SHARED, dwc2_handle_common_intr, IRQF_SHARED,
dev_name(hsotg->dev), hsotg); dev_name(hsotg->dev), hsotg);
if (retval) if (retval)
...@@ -523,14 +541,14 @@ static int dwc2_driver_probe(struct platform_device *dev) ...@@ -523,14 +541,14 @@ static int dwc2_driver_probe(struct platform_device *dev)
dwc2_force_dr_mode(hsotg); dwc2_force_dr_mode(hsotg);
if (hsotg->dr_mode != USB_DR_MODE_HOST) { if (hsotg->dr_mode != USB_DR_MODE_HOST) {
retval = dwc2_gadget_init(hsotg, irq); retval = dwc2_gadget_init(hsotg, hsotg->irq);
if (retval) if (retval)
goto error; goto error;
hsotg->gadget_enabled = 1; hsotg->gadget_enabled = 1;
} }
if (hsotg->dr_mode != USB_DR_MODE_PERIPHERAL) { if (hsotg->dr_mode != USB_DR_MODE_PERIPHERAL) {
retval = dwc2_hcd_init(hsotg, irq); retval = dwc2_hcd_init(hsotg, hsotg->irq);
if (retval) { if (retval) {
if (hsotg->gadget_enabled) if (hsotg->gadget_enabled)
dwc2_hsotg_remove(hsotg); dwc2_hsotg_remove(hsotg);
...@@ -597,6 +615,7 @@ static struct platform_driver dwc2_platform_driver = { ...@@ -597,6 +615,7 @@ static struct platform_driver dwc2_platform_driver = {
}, },
.probe = dwc2_driver_probe, .probe = dwc2_driver_probe,
.remove = dwc2_driver_remove, .remove = dwc2_driver_remove,
.shutdown = dwc2_driver_shutdown,
}; };
module_platform_driver(dwc2_platform_driver); module_platform_driver(dwc2_platform_driver);
......
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