Skip to content

adding rp2350 adalogger #10

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

Merged
merged 7 commits into from
Apr 15, 2025
Merged

adding rp2350 adalogger #10

merged 7 commits into from
Apr 15, 2025

Conversation

BlitzCityDIY
Copy link

adding rp2350 adalogger feather

earlephilhower and others added 7 commits April 7, 2025 10:17
* Convert to SDK RP2350A/B determination

Fixes earlephilhower#2878

The SDK uses `defined(PICO_RP2350) && !PICO_RP2350A` to indicate an RP2350B
chip, not the define PICO_RP2350B.

Match the SDK's usage by converting from `defined(PICO_RP2350B)` to
`defined(PICO_RP2350) && !PICO_RP2350A` and update the chip variants
accordingly.

* Need to explicitly override PICO_RP2350A for all

The *SDK*'s board definition file hardcodes a PICO_RP2350A value for
all boards, but we use the same board file for both A and B variants.
Override the SDK board definition in the variant definition file.

* Generic RP2350 needs 2-stage PICO_RP2350A setting

Also ensure SDK board definition included before pins_arduino.h for
clearing up redefinition errors.

* Factor out undef PICO_RP2350A

* Update Arduino.h
Update the PICO_RP2350A macro usage
adding rp2350 adalogger
@BlitzCityDIY
Copy link
Author

@ladyada sync with upstream and add feather rp2350 adalogger

@ladyada ladyada merged commit 88c717d into master Apr 15, 2025
26 checks passed
@ladyada ladyada deleted the rp2350_adalogger branch April 15, 2025 15:27
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.

3 participants