Re: [PATCH] usb:cdns3: Fix for CV CH9 running with g_zero driver.
From: Felipe Balbi
Date: Tue Oct 08 2019 - 08:59:06 EST
Hi,
Peter Chen <peter.chen@xxxxxxx> writes:
> On 19-10-07 07:39:11, Pawel Laszczak wrote:
>> Patch fixes issue with Halt Endnpoint Test observed
>
> %s/Endnpoint/Endpoint
>
>>
>> during using g_zero
>> driver as DUT. Bug occurred only on some testing board.
>
> g_zero is legacy, please use configfs function source_sink or loopback
> instead.
We still want fixes for those ;-)
>> Endpoint can defer transition to Halted state if endpoint has pending
>> requests.
>
> The implementation of halt handling is a little complicated, you may
> consider return -EAGAIN for functional stall through usb_ep_set_halt
> from function driver if the requests are pending, it doesn't need to
> defer such kinds of functional stall.
-EAGAIN, IIRC is only supposed to be used for IN endpoint stalls.
--
balbi
Attachment:
signature.asc
Description: PGP signature