On Mon, May 29, 2023 at 11:48:15PM +0000, Badhri Jagan Sridharan wrote: > This reverts commit 5e1617210aede9f1b91bb9819c93097b6da481f9. > > The regression reported in > https://lore.kernel.org/all/ZF4bMptC3Lf2Hnee@xxxxxxxxxxx/ is being > fixed in > commit 7d7863db7cc0 ("usb: gadget: udc: core: Offload usb_udc_vbus_handler processing"). > Hence reverting the revert. > > Signed-off-by: Badhri Jagan Sridharan <badhri@xxxxxxxxxx> > --- > drivers/usb/gadget/udc/core.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/usb/gadget/udc/core.c b/drivers/usb/gadget/udc/core.c > index 583c339876ab..4641153e9706 100644 > --- a/drivers/usb/gadget/udc/core.c > +++ b/drivers/usb/gadget/udc/core.c > @@ -703,6 +703,9 @@ static int usb_gadget_connect_locked(struct usb_gadget *gadget) > goto out; > } > > + if (gadget->connected) > + goto out; > + > if (gadget->deactivated || !gadget->udc->started) { > /* > * If gadget is deactivated we only save new state. This is silly. There's no need to make this a separate commit; it should be merged in with the preceding patch. There's no good reason for creating a commit that contains a known error. Alan Stern
