ESP32-S3 bulk transfer issues

[qnex0]

Operating System

Linux

Commit SHA

espressif/tinyusb (0.18.0~2)

Board

ESP32-S3 based board

Firmware

tinyusb_firmware.zip

libusb_test.zip

What happened ?

I'm trying to implement a custom vendor class driver. I noticed that when using usbd_edpt_xfer to receive data from the OUT EP, the ZLP is misinterpreted and causes the transfer to complete with an xfer size equal to the request size regardless of the actual size. I tried some older versions of tinyusb and noticed this behavior started occurring after version "0.15.0~10." Additionally, when I queue for 32768 bytes in my example, when more than 8191 bytes are sent to the device, the transfer is also completed with the incorrect size.

How to reproduce ?

I use the libusb_test program attached to the firmware section of this issue to send a variety of different data sizes to the bulk endpoint and view the results using the firmware with debugging enabled.

Debug Log as txt file (LOG/CFG_TUSB_DEBUG=2)

N/A

Screenshots

I have checked existing issues, dicussion and documentation

• I confirm I have checked existing issues, dicussion and documentation.

[dpgeorge]

Just for reference, it looks like this commit db7670a was responsible for this issue (it seems that ZLP was correctly handled prior to that commit).

[hathach]

@dpgeorge it seems to be fixed by #3293, let me know if that works for you.

[dpgeorge]

@hathach yes, that PR fixes the issue. We have needed to update our TinyUSB submodule to include that patch which is not yet in a release (see micropython/micropython#18406) and also fork the Espressif fork of TinyUSB to patch that as well (see micropython/micropython#18407).

[hathach]

@dpgeorge I will make an minor release for this soon enough (in a day or two)

[dpgeorge]

I will make an minor release for this soon enough (in a day or two)

That would be great, thank you!

[hathach]

0.20.0 is release, please give it a try to see if there is any other new issues.

[dpgeorge]

Very good, thank you! I will test it.

[dpgeorge]

I tested TinyUSB 0.20.0 and found an issue with samd21, see micropython/micropython#18406 (comment)