usb: gadget: udc-core: fix wrong call order
authorFelipe Balbi <balbi@ti.com>
Fri, 27 Apr 2012 08:02:15 +0000 (11:02 +0300)
committerFelipe Balbi <balbi@ti.com>
Fri, 27 Apr 2012 08:02:15 +0000 (11:02 +0300)
commit 6d258a4 (usb: gadget: udc-core: stop UDC on device-initiated
disconnect) introduced another case of asymmetric calls when issuing
a device-initiated disconnect. Fix it.

Reported-by: Ben Hutchings <ben@decadent.org.uk>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/gadget/udc-core.c

index 2fa9865babedb0d76e28d6cfa2ac2ad6cf859628..df1bd87232ddb7e41a59bd62274e8124893fdebd 100644 (file)
@@ -415,9 +415,9 @@ static ssize_t usb_udc_softconn_store(struct device *dev,
                        usb_gadget_udc_start(udc->gadget, udc->driver);
                usb_gadget_connect(udc->gadget);
        } else if (sysfs_streq(buf, "disconnect")) {
+               usb_gadget_disconnect(udc->gadget);
                if (udc_is_newstyle(udc))
                        usb_gadget_udc_stop(udc->gadget, udc->driver);
-               usb_gadget_disconnect(udc->gadget);
        } else {
                dev_err(dev, "unsupported command '%s'\n", buf);
                return -EINVAL;