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

Support max3421e (usb host shield) for nrf52 and samd21/samd51 #334

Merged
merged 9 commits into from
Sep 18, 2023

Conversation

hathach
Copy link
Member

@hathach hathach commented Sep 14, 2023

  • Added max3421e (usb host shield) controller
  • tested with nrf52 and samd21/samd51, work relatively well, there is a few hipcups, but it is due to tinyusb core (connection debouncing issue, will fix later)
  • update all DualRole examples: CDC/serial_host_bridge, Simple/device_info, MassStorage/msc_file_explorer, HID/hid_device_report. Tested all feather nrf52840, metro m0 & m4 express, all seems to work well.
  • Fix MSC host wait_for_io() issue when running with freeRTOS (e.g nrf52840).

PS: not tested with hub + device, hub + lowspeed device

@lyusupov
Copy link
Contributor

lyusupov commented Sep 14, 2023

I am curious of why external MAX3421 USB Host is required for a SAMD x1 SoC when the latter can do the Host feature with built-in hardware components ?
Please, see this library as a known good example: https://github.com/gdsports/USB_Host_Library_SAMD

@hathach
Copy link
Member Author

hathach commented Sep 18, 2023

we like to keep usb device for uploading and debugging. Feel free to use other library if you don't want to use max3421 for samd

@hathach hathach merged commit d4c72bf into master Sep 18, 2023
18 checks passed
@hathach hathach deleted the add-max3421e-support branch September 18, 2023 17:49
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.

2 participants