Re: [PATCH v3] usb: dwc3: gadget: don't error on dequeue of a completed request

From: Cole Munz

Date: Fri Sep 04 2026 - 20:03:17 EST


On Fri, Sep 04, 2026, Thinh Nguyen wrote:
> NAK.
>
> This is not a fix. This changes the dequeue() behavior. You're breaking
> the documented behavior of usb_ep_dequeue():

You're right, the return value is documented and I shouldn't change it.

The part that actually hurts is the dev_err. f_fs dequeues ep0req on
every teardown on purpose since ce405d561b02, so every unbind logs

dwc3 23000000.usb: request 00000000ad92f1c4 was not queued to ep0out

for a request that completed normally, and that noise buries real
errors. Would you take a version that keeps the -EINVAL return and only
lowers the message to dev_dbg when the request is COMPLETED on its own
endpoint? The never-queued and wrong-endpoint cases would keep the
dev_err.

If that's not worth doing either, I'll drop it.

Regards,
Cole