> From: Felipe Balbi [mailto:balbi@xxxxxx] > Sent: Friday, October 17, 2014 6:23 PM > > that call is completely unnecessary because > usb_del_gadget_udc() already makes sure the > gadget driver is properly unregistered from > the UDC. > > Signed-off-by: Felipe Balbi <balbi@xxxxxx> > --- > > found while reading code > > drivers/usb/dwc2/gadget.c | 7 ------- > 1 file changed, 7 deletions(-) > > diff --git a/drivers/usb/dwc2/gadget.c b/drivers/usb/dwc2/gadget.c > index e55ba85..8f1502f 100644 > --- a/drivers/usb/dwc2/gadget.c > +++ b/drivers/usb/dwc2/gadget.c > @@ -3608,14 +3608,7 @@ static int s3c_hsotg_remove(struct platform_device *pdev) > struct s3c_hsotg *hsotg = platform_get_drvdata(pdev); > > usb_del_gadget_udc(&hsotg->gadget); > - > s3c_hsotg_delete_debug(hsotg); > - > - if (hsotg->driver) { > - /* should have been done already by driver model core */ > - usb_gadget_unregister_driver(hsotg->driver); > - } > - > clk_disable_unprepare(hsotg->clk); > > return 0; Looks good to me. Acked-by: Paul Zimmerman <paulz@xxxxxxxxxxxx> -- Paul -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
