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

Fix some race conditions #2630

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

Conversation

HiFiPhile
Copy link
Collaborator

@HiFiPhile HiFiPhile commented May 8, 2024

Describe the PR

  • In cdc_device if read* functions called before tud_ready() transfer will be queued to EP0 cause strange issues.
  • Clear _usbd_dev first to ensure tud_ready() return false before driver clean up.
  • Introduced by work around possible RP2040 erratum #2492, if both DCD_EVENT_BUS_RESET and DCD_EVENT_SETUP_RECEIVED are in the event queue, setup_count is cleared in bus reset, result _usbd_dev.setup_count-- underflow. I think the best fix would be clear the queue on bus reset.
  • Fix STM32F7 FS port build using HS define.

@HiFiPhile HiFiPhile changed the title Fix 2 race conditions Fix some race conditions May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant