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

add RF.Guru RP2040 board #9095

Merged
merged 8 commits into from
Mar 27, 2024
Merged

add RF.Guru RP2040 board #9095

merged 8 commits into from
Mar 27, 2024

Conversation

ure
Copy link

@ure ure commented Mar 26, 2024

@ure
Copy link
Author

ure commented Mar 27, 2024

@dhalbert i added your xtal start issue prevention sugention ;) Many thx !

@dhalbert
Copy link
Collaborator

@dhalbert i added your xtal start issue prevention sugention ;) Many thx !

You're welcome! For background on this see:
raspberrypi/pico-sdk#457
and raspberrypi/pico-sdk#401 (which takes a while to reach the right conclusion).

@ure
Copy link
Author

ure commented Mar 27, 2024

@dhalbert thx ... interesting reading material ;) We use the RP2040 chip in many of our projects (10+ and counting ;)) and never ran into this issue ... luck must be on our side, hard fault on serial disconnect that was the first thing that really broke things ... but that's been solved in 9 also ... something to test in the weekend ;)

@dhalbert
Copy link
Collaborator

We thought it had to do with our choice of capacitor and crystal but redoing the capacitor values did not seem to fix things. Not that many boards had the trouble, but it was a noticeable small fraction util we add the startup delay.

Copy link
Collaborator

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

Looks good!

@dhalbert dhalbert merged commit 7553c5b into adafruit:main Mar 27, 2024
15 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