usb: dwc3: gadget: stop touching HWO TRBs
authorFelipe Balbi <felipe.balbi@linux.intel.com>
Tue, 25 Oct 2016 10:50:46 +0000 (13:50 +0300)
committerFelipe Balbi <felipe.balbi@linux.intel.com>
Thu, 3 Nov 2016 08:38:41 +0000 (10:38 +0200)
Say we have three requests prepared to the HW (reqA,
reqB, and reqC). All of them are composed of
SG-lists with several entries and they all requests
interrupt only on last TRBs of the SG-list.

When we get interrupt for reqA, it could be that
reqB is already half-way transferred and some of its
TRBs will have HWO already cleared.

It's okay to free up TRBs without HWO bit set, but
we need to guarantee we don't giveback a request
that's half-way transferred as that will confuse
gadget drivers.

Signed-off-by: Felipe Balbi <felipe.balbi@linux.intel.com>
drivers/usb/dwc3/gadget.c

index 317bb9aed4a47277bd386668b3315c54212d1091..4643c25702aebe8feaf86a3f9628b43bb93a9c31 100644 (file)
@@ -2104,6 +2104,9 @@ static int dwc3_cleanup_done_reqs(struct dwc3 *dwc, struct dwc3_ep *dep,
                        for_each_sg(sg, s, pending, i) {
                                trb = &dep->trb_pool[dep->trb_dequeue];
 
+                               if (trb->ctrl & DWC3_TRB_CTRL_HWO)
+                                       break;
+
                                req->sg = sg_next(s);
                                req->num_pending_sgs--;