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