| Message ID | 1425991270-30629-1-git-send-email-m.szyprowski@samsung.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 3/10/2015 5:41 AM, Marek Szyprowski wrote: > If device is configured to work only in HOST or DEVICE mode, there is > no point in initializing both subdrivers. This patch also fixes > resource leakage if host subdriver fails to initialize. > > Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> > --- > drivers/usb/dwc2/core.h | 2 ++ > drivers/usb/dwc2/platform.c | 29 +++++++++++++++++++++-------- > 2 files changed, 23 insertions(+), 8 deletions(-) > > diff --git a/drivers/usb/dwc2/core.h b/drivers/usb/dwc2/core.h > index 7a70a1349334..f93b06daef97 100644 > --- a/drivers/usb/dwc2/core.h > +++ b/drivers/usb/dwc2/core.h > @@ -570,6 +570,8 @@ struct dwc2_hsotg { > struct dwc2_core_params *core_params; > enum usb_otg_state op_state; > enum usb_dr_mode dr_mode; > + unsigned int hcd_enabled:1; > + unsigned int gadget_enabled:1; > > struct phy *phy; > struct usb_phy *uphy; > diff --git a/drivers/usb/dwc2/platform.c b/drivers/usb/dwc2/platform.c > index 6a795aa2ff05..ee0b0b06d0fc 100644 > --- a/drivers/usb/dwc2/platform.c > +++ b/drivers/usb/dwc2/platform.c > @@ -121,8 +121,10 @@ static int dwc2_driver_remove(struct platform_device *dev) > { > struct dwc2_hsotg *hsotg = platform_get_drvdata(dev); > > - dwc2_hcd_remove(hsotg); > - s3c_hsotg_remove(hsotg); > + if (hsotg->hcd_enabled) > + dwc2_hcd_remove(hsotg); > + if (hsotg->gadget_enabled) > + s3c_hsotg_remove(hsotg); > > return 0; > } > @@ -214,12 +216,23 @@ static int dwc2_driver_probe(struct platform_device *dev) > > spin_lock_init(&hsotg->lock); > mutex_init(&hsotg->init_mutex); > - retval = dwc2_gadget_init(hsotg, irq); > - if (retval) > - return retval; > - retval = dwc2_hcd_init(hsotg, irq, params); > - if (retval) > - return retval; > + > + if (hsotg->dr_mode != USB_DR_MODE_HOST) { > + retval = dwc2_gadget_init(hsotg, irq); > + if (retval) > + return retval; > + hsotg->gadget_enabled = 1; > + } > + > + if (hsotg->dr_mode != USB_DR_MODE_PERIPHERAL) { > + retval = dwc2_hcd_init(hsotg, irq, params); > + if (retval) { > + if (hsotg->gadget_enabled) > + s3c_hsotg_remove(hsotg); > + return retval; > + } > + hsotg->hcd_enabled = 1; > + } > > platform_set_drvdata(dev, hsotg); > > Acked-by: John Youn <johnyoun@synopsys.com> -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/usb/dwc2/core.h b/drivers/usb/dwc2/core.h index 7a70a1349334..f93b06daef97 100644 --- a/drivers/usb/dwc2/core.h +++ b/drivers/usb/dwc2/core.h @@ -570,6 +570,8 @@ struct dwc2_hsotg { struct dwc2_core_params *core_params; enum usb_otg_state op_state; enum usb_dr_mode dr_mode; + unsigned int hcd_enabled:1; + unsigned int gadget_enabled:1; struct phy *phy; struct usb_phy *uphy; diff --git a/drivers/usb/dwc2/platform.c b/drivers/usb/dwc2/platform.c index 6a795aa2ff05..ee0b0b06d0fc 100644 --- a/drivers/usb/dwc2/platform.c +++ b/drivers/usb/dwc2/platform.c @@ -121,8 +121,10 @@ static int dwc2_driver_remove(struct platform_device *dev) { struct dwc2_hsotg *hsotg = platform_get_drvdata(dev); - dwc2_hcd_remove(hsotg); - s3c_hsotg_remove(hsotg); + if (hsotg->hcd_enabled) + dwc2_hcd_remove(hsotg); + if (hsotg->gadget_enabled) + s3c_hsotg_remove(hsotg); return 0; } @@ -214,12 +216,23 @@ static int dwc2_driver_probe(struct platform_device *dev) spin_lock_init(&hsotg->lock); mutex_init(&hsotg->init_mutex); - retval = dwc2_gadget_init(hsotg, irq); - if (retval) - return retval; - retval = dwc2_hcd_init(hsotg, irq, params); - if (retval) - return retval; + + if (hsotg->dr_mode != USB_DR_MODE_HOST) { + retval = dwc2_gadget_init(hsotg, irq); + if (retval) + return retval; + hsotg->gadget_enabled = 1; + } + + if (hsotg->dr_mode != USB_DR_MODE_PERIPHERAL) { + retval = dwc2_hcd_init(hsotg, irq, params); + if (retval) { + if (hsotg->gadget_enabled) + s3c_hsotg_remove(hsotg); + return retval; + } + hsotg->hcd_enabled = 1; + } platform_set_drvdata(dev, hsotg);
If device is configured to work only in HOST or DEVICE mode, there is no point in initializing both subdrivers. This patch also fixes resource leakage if host subdriver fails to initialize. Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com> --- drivers/usb/dwc2/core.h | 2 ++ drivers/usb/dwc2/platform.c | 29 +++++++++++++++++++++-------- 2 files changed, 23 insertions(+), 8 deletions(-)