usb: dwc3: ep0: fix for possible early delayed_status
authorFelipe Balbi <balbi@ti.com>
Thu, 8 Dec 2011 11:56:27 +0000 (13:56 +0200)
committerFelipe Balbi <balbi@ti.com>
Mon, 12 Dec 2011 09:48:47 +0000 (11:48 +0200)
There is a very small possibility (previously unimagined
by us) that the whole Mass Storage delayed status happens
rather early, before we even get our XferNotReady event.

In that case, we will be queueing a request to ep0 while
we're still on Setup Phase and we would be waiting for
another usb_ep_queue() forever.

Handle such cases by clearing dwc->delayed_status so that
we start control status from the next XferNotReady like
there was no wait for Delayed Status.

Tested against Linux 3.2-rc3 and USB30CV tool from USB-IF
(on a Windows XP with USB3 PCIe card).

Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/dwc3/ep0.c

index d6bfc73dedbd952cd92b2b83693dc4a236913d6f..2f51de57593a11ebc617085707d22e67735165de 100644 (file)
@@ -165,10 +165,13 @@ static int __dwc3_gadget_ep0_queue(struct dwc3_ep *dep,
                                req->request.dma, req->request.length, type);
                dep->flags &= ~(DWC3_EP_PENDING_REQUEST |
                                DWC3_EP0_DIR_IN);
-
-       } else if (dwc->delayed_status && (dwc->ep0state == EP0_STATUS_PHASE)) {
+       } else if (dwc->delayed_status) {
                dwc->delayed_status = false;
-               dwc3_ep0_do_control_status(dwc, 1);
+
+               if (dwc->ep0state == EP0_STATUS_PHASE)
+                       dwc3_ep0_do_control_status(dwc, 1);
+               else
+                       dev_dbg(dwc->dev, "too early for delayed status\n");
        }
 
        return ret;