Re: [PATCH] usb:cdns3: Fix for CV CH9 running with g_zero driver.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Pawel,

On 09/10/2019 06:58, Pawel Laszczak wrote:
Hi,


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 ;-)

So, if my understanding is correct in new production systems
using legacy gadget drivers is not recommended and we are not going to fix
any more bugs there.

We can't really control who uses what. So we need to support legacy
drivers as well.

The issue detected by using g_zero is a controller driver issue and not an
issue with g_zero itself.

cheers,
-roger


The legacy gadget drivers remains in the kernel only to support the old
products.

So because we introduce new driver we should not worry  about legacy
gadget drivers.

Is my understanding correct ?

pawell

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

--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux