Skip to content

Conversation

@FoamyGuy
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@TheKitty TheKitty left a comment

Choose a reason for hiding this comment

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

Spelling: If already enumerated, additional class driverr

PIO USB require CPU clock must be multiple of 120 Mhz - is that true or 12 MHz?

@FoamyGuy
Copy link
Collaborator Author

PIO USB require CPU clock must be multiple of 120 Mhz - is that true or 12 MHz?

I think it's 120 Mhz, but honestly I am not 100%, nor do I know the reason for the limitation. The code here seems to enforce it though: https://github.com/FoamyGuy/Adafruit_Learning_System_Guides/blob/7edbbf2401b81734db129807f714c056cba07048/USB_SNES_Gamepad/Arduino_USB_Host/usbh_helper.h#L68

I reused that file from here: https://github.com/adafruit/Adafruit_Learning_System_Guides/blob/main/USB_Host_Turbo_Button_Gamepad/Arduino_USB_Host_Turbo_Button_Gamepad/usbh_helper.h

@TheKitty
Copy link
Collaborator

TheKitty commented Apr 17, 2025

PIO USB require CPU clock must be multiple of 120 Mhz - is that true or 12 MHz?

I think it's 120 Mhz, but honestly I am not 100%, nor do I know the reason for the limitation. The code here seems to enforce it though: https://github.com/FoamyGuy/Adafruit_Learning_System_Guides/blob/7edbbf2401b81734db129807f714c056cba07048/USB_SNES_Gamepad/Arduino_USB_Host/usbh_helper.h#L68

I reused that file from here: https://github.com/adafruit/Adafruit_Learning_System_Guides/blob/main/USB_Host_Turbo_Button_Gamepad/Arduino_USB_Host_Turbo_Button_Gamepad/usbh_helper.h

Please check with Scott, I was thinking this was an evolving thing

Also look to use a .test.only file for Arduino projects https://learn.adafruit.com/submitting-arduino-code-to-the-adafruit-learning-system

@FoamyGuy FoamyGuy merged commit 3f0ed86 into adafruit:main Apr 17, 2025
51 checks passed
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