Skip to content

Conversation

@petejohanson
Copy link
Contributor

Add initial Adafruit Metro RP2350 board definition.

@petejohanson petejohanson force-pushed the boards/adafruit-metro-rp2350 branch from bb2150f to 5a9cc90 Compare March 25, 2025 17:40
@petejohanson petejohanson requested a review from nordicjm March 25, 2025 17:42
@petejohanson petejohanson force-pushed the boards/adafruit-metro-rp2350 branch from 5a9cc90 to 58577e3 Compare March 27, 2025 16:09
nordicjm
nordicjm previously approved these changes Mar 28, 2025
@nordicjm nordicjm requested a review from kartben March 28, 2025 07:49
@petejohanson petejohanson force-pushed the boards/adafruit-metro-rp2350 branch from 58577e3 to b0d92c7 Compare March 29, 2025 08:01
@github-actions github-actions bot added the area: ADC Analog-to-Digital Converter (ADC) label Mar 29, 2025
@github-actions github-actions bot requested a review from anangl March 29, 2025 08:01
@petejohanson petejohanson force-pushed the boards/adafruit-metro-rp2350 branch 2 times, most recently from d64fd24 to 131d84b Compare March 29, 2025 22:48
@petejohanson
Copy link
Contributor Author

Rebased and fixed the ADC test build failure, should hopefully be green now.

@petejohanson petejohanson requested a review from nordicjm May 17, 2025 03:30
@soburi soburi self-assigned this Jun 2, 2025
@soburi soburi added the platform: Raspberry Pi Pico Raspberry Pi Pico (RPi Pico) label Jun 2, 2025
@github-actions
Copy link

github-actions bot commented Aug 5, 2025

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Aug 5, 2025
@petejohanson petejohanson force-pushed the boards/adafruit-metro-rp2350 branch 2 times, most recently from e69a781 to 363d4d6 Compare August 5, 2025 18:11
Add initial Adafruit Metro RP2350 board definition.

Signed-off-by: Peter Johanson <[email protected]>
@petejohanson petejohanson force-pushed the boards/adafruit-metro-rp2350 branch from 363d4d6 to f7ff013 Compare August 5, 2025 18:13
@petejohanson petejohanson requested review from nordicjm and soburi August 5, 2025 18:13
@petejohanson
Copy link
Contributor Author

Please remove the Stale label. Thanks!

@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 5, 2025

@github-actions github-actions bot removed the Stale label Aug 6, 2025
Copy link
Member

@soburi soburi left a comment

Choose a reason for hiding this comment

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

LGTM

@jhedberg jhedberg merged commit 6cd8882 into zephyrproject-rtos:main Aug 6, 2025
20 checks passed
Comment on lines +5 to +9

Choo! Choo! This is the RP2350 Metro Line, making all station stops at "Dual Cortex M33 mountain",
"528K RAM round-about" and "16 Megabytes of Flash town". This train is piled high with hardware that
complements the Raspberry Pi RP2350 chip to make it an excellent development board for projects that
want Arduino-shape-compatibility or just need the extra space and debugging ports.
Copy link
Contributor

Choose a reason for hiding this comment

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

For future reference, I don't think we want to use the vendor's marketing blurb (esp. when the tone is clearly geared towards "promotion") verbatim in our own Zephyr docs. Maybe this can be adjusted in a follow-up PR

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fair. I'll work to correct in a follow up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ADC Analog-to-Digital Converter (ADC) platform: Raspberry Pi Pico Raspberry Pi Pico (RPi Pico)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants