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

bugfix(ch32-hs-dcd): fix ch32 DATAx managment and long packet transmit #2392

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Krasutski
Copy link

@Krasutski Krasutski commented Dec 28, 2023

Old implementations have some issues:

  • EP0 OUT transfer just copied data from the buffer without any checking
  • TOG1 bit set not correctly for OUT xfers
  • Long packets(more than ep size) xfer incorrectly

Fixed this issue #1891 also other examples now work stable

UPD:
Pushed one more commit, fixed iso transfers, probably this issue fixed: #2325

@Krasutski Krasutski force-pushed the bugfix/ch32-hs-dcd branch 3 times, most recently from 9d9a427 to 6a13b0a Compare December 29, 2023 00:05
@zx96
Copy link
Contributor

zx96 commented Jan 21, 2024

This MR seems to resolve issues I was having getting writes to work reliably with the MSC device class. 👍

@WilheJo
Copy link

WilheJo commented Jan 22, 2024

My problem in #2377 is also fixed with this MR!

@Krasutski
Copy link
Author

@hathach Could you check and merge PR?

Copy link
Owner

@hathach hathach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

still haven't got time to test this out. I am off for TET holidays for a couple of weeks. Will try to review afterwards, thank you for your patient.

@Krasutski
Copy link
Author

Krasutski commented Apr 9, 2024

@hathach how can I help to complete this PR?

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

Successfully merging this pull request may close these issues.

None yet

4 participants