usb: dwc3: gadget: drop unnecessary loop when cleaning up TRBs

Now that we're using XFERINPROGRESS for all endpoint
types (except Control), we will *always* be completing
one TRB at a time, so it's safe to remove the loop
from dwc3_cleanup_done_reqs.

Signed-off-by: Felipe Balbi <balbi@ti.com>

1 files changed, 16 insertions, 21 deletions

diff --git a/drivers/usb/dwc3/gadget.c b/drivers/usb/dwc3/gadget.c
index a03a485205c7..8946c32047e9 100644
--- a/drivers/usb/dwc3/gadget.c
+++ b/drivers/usb/dwc3/gadget.c
@@ -1855,32 +1855,27 @@ static int dwc3_cleanup_done_reqs(struct dwc3 *dwc, struct dwc3_ep *dep,
         unsigned int            i;
         int                     ret;
 
+        req = next_request(&dep->req_queued);
+        if (!req) {
+                WARN_ON_ONCE(1);
+                return 1;
+        }
+        i = 0;
         do {
-                req = next_request(&dep->req_queued);
-                if (!req) {
-                        WARN_ON_ONCE(1);
-                        return 1;
-                }
-                i = 0;
-                do {
-                        slot = req->start_slot + i;
-                        if ((slot == DWC3_TRB_NUM - 1) &&
+                slot = req->start_slot + i;
+                if ((slot == DWC3_TRB_NUM - 1) &&
                                 usb_endpoint_xfer_isoc(dep->endpoint.desc))
-                                slot++;
-                        slot %= DWC3_TRB_NUM;
-                        trb = &dep->trb_pool[slot];
-
-                        ret = __dwc3_cleanup_done_trbs(dwc, dep, req, trb,
-                                        event, status);
-                        if (ret)
-                                break;
-                }while (++i < req->request.num_mapped_sgs);
-
-                dwc3_gadget_giveback(dep, req, status);
+                        slot++;
+                slot %= DWC3_TRB_NUM;
+                trb = &dep->trb_pool[slot];
 
+                ret = __dwc3_cleanup_done_trbs(dwc, dep, req, trb,
+                                event, status);
                 if (ret)
                         break;
-        } while (1);
+        } while (++i < req->request.num_mapped_sgs);
+
+        dwc3_gadget_giveback(dep, req, status);
 
         if (usb_endpoint_xfer_isoc(dep->endpoint.desc) &&
                         list_empty(&dep->req_queued)) {