Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Speed up tinyusb CDC serial transfer rate on ESP32-S3 (IEC-93) (IEC-232) #54

Open
2 tasks done
Superberti opened this issue Mar 4, 2024 · 5 comments
Open
2 tasks done

Comments

@Superberti
Copy link

Answers checklist.

  • I have read the documentation of the component in question and the issue is not addressed there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report

Hi,

I'm using an ESP32-S3 on ESP-IDF on the release/5.2 branch.
I made some tests with the tinyusb CDC serial profile but I was not able to get read transfer speeds on the native port (pins 19/20) over 50 KiB/s. The theoretical maximum rate should be something about 1 MiB/s in this mode.
Regardless what I'm doing (larger buffers and FIFOs, using tinyusb with or without tusb_config-h), it had no influence on the transfer speed. So it seems that the speed is limited in some other way. The USB port is connected to a C# test program which sends out the data as fast as it can, so this should not be a problem at all.
Any ideas how to speedup the CDC read speed?

Thanks and bye,
Oliver

@github-actions github-actions bot changed the title Speed up tinyusb CDC serial transfer rate on ESP32-S3 Speed up tinyusb CDC serial transfer rate on ESP32-S3 (IEC-93) Mar 4, 2024
@tore-espressif
Copy link
Collaborator

Hello @Superberti Thank you much for the report.

We can reproduce the issue and see similar performance.
The bottleneck is in the DCD layer of TinyUSB, this will requires significant refactoring. We plan to work this in Q2/2024

@Superberti
Copy link
Author

Thank you for clarification on this subject!

@tore-espressif tore-espressif transferred this issue from espressif/idf-extra-components Jul 31, 2024
@tayvs
Copy link

tayvs commented Nov 1, 2024

@tore-espressif Hey.
Are there any updates? Is there an open issue in tinyUSB that provides latest updates?

@espressif-bot espressif-bot added the Status: Opened Issue is new label Nov 1, 2024
@github-actions github-actions bot changed the title Speed up tinyusb CDC serial transfer rate on ESP32-S3 (IEC-93) Speed up tinyusb CDC serial transfer rate on ESP32-S3 (IEC-93) (IEC-232) Nov 1, 2024
@tore-espressif
Copy link
Collaborator

The ticket in upstream TinyUSB is here hathach/tinyusb#2576 , it was merged a month ago.

It is already tested on S2 and S3, right now we are preparing P4 support and then we can release new version

@emaayan
Copy link

emaayan commented Dec 4, 2024

hi ,will this effect the NCM performance throughput as well?

@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: Duplicate This issue or pull request already exists and removed Status: Opened Issue is new labels Dec 16, 2024
@tore-espressif tore-espressif removed Resolution: Duplicate This issue or pull request already exists Status: Done Issue is done internally labels Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants