diff options
| author | Ye Li <[email protected]> | 2026-05-22 15:55:11 +0800 |
|---|---|---|
| committer | Mattijs Korpershoek <[email protected]> | 2026-07-22 09:16:16 +0200 |
| commit | b79147bbd47cd2d50836dc0a57376f4820663637 (patch) | |
| tree | ddb8ac1a688e08d97301560d9efe08b20846b52e | |
| parent | eed2ed75883bdf029be9325fb9dd9263d921372e (diff) | |
usb: ci_udc: Update usb request status
usb request status is not set in ci_udc driver. However
in ci_ep_dequeue, it checks the request status for executing
complete callback, so upper layer is not able to free these
enqueued requests when calling usb_ep_dequeue.
Signed-off-by: Ye Li <[email protected]>
Fixes: 26cc5129ee64 ("USB: gadaget: add Marvell controller support")
Reviewed-by: Mattijs Korpershoek <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Mattijs Korpershoek <[email protected]>
| -rw-r--r-- | drivers/usb/gadget/ci_udc.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/usb/gadget/ci_udc.c b/drivers/usb/gadget/ci_udc.c index a62e5ea5658..0baad83ef90 100644 --- a/drivers/usb/gadget/ci_udc.c +++ b/drivers/usb/gadget/ci_udc.c @@ -680,6 +680,8 @@ static int ci_ep_queue(struct usb_ep *ep, if (ret) return ret; + req->status = -EINPROGRESS; + DBG("ept%d %s pre-queue req %p, buffer %p\n", num, in ? "in" : "out", ci_req, ci_req->hw_buf); list_add_tail(&ci_req->queue, &ci_ep->queue); @@ -754,6 +756,7 @@ static void handle_ep_complete(struct ci_ep *ci_ep) ci_ep_submit_next_request(ci_ep); ci_req->req.actual = ci_req->req.length - len; + ci_req->req.status = 0; ci_debounce(ci_req, in); DBG("ept%d %s req %p, complete %x\n", |