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

nrf5x USB Compliance Verification Test suite #1058

Merged
merged 21 commits into from
Aug 30, 2021
Merged

Conversation

hathach
Copy link
Owner

@hathach hathach commented Aug 30, 2021

Describe the PR

Initial PR to get TinyUSB passed the USB complaince verification test suite (USBCV). The test suite include many corner cases which require changes to both usbd and dcd to pass. This PRs include the update to get nrf52840 ( board feather_nrf52840_express) to pass following:

  • Chapter9 tests suite
  • HID test suite for hid_composite example
  • MSC test suite for cdc_msc/msc_dual_lun examples

Chapter9 test requires an new dcd API dcd_edpt_close_all() to switch between configuration dynamically (mentioned in #1050 and #167). This API is only implemented in nRF5x by this PR, it is rare to see and generally ok to leave empty if the device support only 1 active configuration.

This PR makes msc driver more robust (to response to all weird commands) and fix several hidden issue with nrf5x dcd

  • Explicitly clear stall and reset toggle when open endpoint in case host switch configuration off/on when there is a stalled endpoint
  • Correctly reset data toggle when clear stall

More PRs to get all other MCU passed test suite will follow up in the future. Check out #1059 for compliance status of all ports.

usbcv

@hathach hathach merged commit 38f5aee into master Aug 30, 2021
@hathach hathach deleted the usbcv-compliant-test branch August 30, 2021 10:31
sandeepmistry added a commit to ArmDeveloperEcosystem/microphone-library-for-pico that referenced this pull request Sep 23, 2021
sandeepmistry added a commit to ArmDeveloperEcosystem/microphone-library-for-pico that referenced this pull request Sep 23, 2021
sandeepmistry added a commit to ArmDeveloperEcosystem/microphone-library-for-pico that referenced this pull request Sep 24, 2021
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

3 participants