usb: dwc3: gadget: Don't clear flags before transfer ended
authorThinh Nguyen <Thinh.Nguyen@synopsys.com>
Thu, 5 Mar 2020 21:23:49 +0000 (13:23 -0800)
committerFelipe Balbi <balbi@kernel.org>
Sun, 15 Mar 2020 09:19:37 +0000 (11:19 +0200)
We track END_TRANSFER command completion. Don't clear transfer
started/ended flag prematurely. Otherwise, we'd run into the problem
with restarting transfer before END_TRANSFER command finishes.

Fixes: 6d8a019614f3 ("usb: dwc3: gadget: check for Missed Isoc from event status")
Signed-off-by: Thinh Nguyen <thinhn@synopsys.com>
Signed-off-by: Felipe Balbi <balbi@kernel.org>
drivers/usb/dwc3/gadget.c

index 1e00bf2..b032e62 100644 (file)
@@ -2570,10 +2570,8 @@ static void dwc3_gadget_endpoint_transfer_in_progress(struct dwc3_ep *dep,
 
        dwc3_gadget_ep_cleanup_completed_requests(dep, event, status);
 
-       if (stop) {
+       if (stop)
                dwc3_stop_active_transfer(dep, true, true);
-               dep->flags = DWC3_EP_ENABLED;
-       }
 
        /*
         * WORKAROUND: This is the 2nd half of U1/U2 -> U0 workaround.