Skip to content

rp2xxx: Fix Adafruit boards#960

Merged
mattnite merged 1 commit into
mainfrom
rp2xxx_fix_adafruit_xosc
Jun 13, 2026
Merged

rp2xxx: Fix Adafruit boards#960
mattnite merged 1 commit into
mainfrom
rp2xxx_fix_adafruit_xosc

Conversation

@Grazfather

Copy link
Copy Markdown
Collaborator

I noticed that my Adafruit Feather rp2350 worked when I first flashed it, but not after a reset or power cycle.

Digging through some code, I found a commit that added support for the Adafruit Feather to the arduino-pico core:

earlephilhower/arduino-pico@163b209d#diff-d0d48a8d47fd5d31e3bc70b12159bdcb23a5ab82460bb5972cf61776f4d6768aR34-R42

It sets PICO_XOSC_STARTUP_DELAY_MULTIPLIER=64.

Seems that these boards have shakier clocks.

This change adds support for an optional
xosc_startup_delay_multiplier, which increases the startup_delay_ms when setting up the external oscillator.

I noticed that my Adafruit Feather rp2350 worked when I first flashed
it, but not after a reset or power cycle.

Digging through some code, I found a commit that added support for the
Adafruit Feather to the arduino-pico core:

earlephilhower/arduino-pico@163b209d#diff-d0d48a8d47fd5d31e3bc70b12159bdcb23a5ab82460bb5972cf61776f4d6768aR34-R42

It sets `PICO_XOSC_STARTUP_DELAY_MULTIPLIER=64`.

Seems that these boards have shakier clocks.

This change adds support for an optional
`xosc_startup_delay_multiplier`, which increases the startup_delay_ms
when setting up the external oscillator.

@mattnite mattnite left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great find, and addition! smh I hope it was worth it adafruit

@mattnite mattnite merged commit 6750f70 into main Jun 13, 2026
59 checks passed
@mattnite mattnite deleted the rp2xxx_fix_adafruit_xosc branch June 13, 2026 21:13
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