Skip to content

ports/rp2/boards/SPARKFUN_IOTREDBOARD_RP2350: Add support for board. #16856

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

Conversation

SFE-Brudnerd
Copy link
Contributor

Summary

Added support for an upcoming SparkFun board

Testing

I compiled and uploaded to my board, then tested the features of the board.
Note: This board will benefit from #15620

Signed-off-by: Alex Brudner <alex.brudner@sparkfun.com>
Copy link

github-actions bot commented Mar 4, 2025

Code size report:

   bare-arm:    +0 +0.000% 
minimal x86:    +0 +0.000% 
   unix x64:    +0 +0.000% standard
      stm32:    +0 +0.000% PYBV10
     mimxrt:    +0 +0.000% TEENSY40
        rp2:    +0 +0.000% RPI_PICO_W
       samd:    +0 +0.000% ADAFRUIT_ITSYBITSY_M4_EXPRESS
  qemu rv32:    +0 +0.000% VIRT_RV32

@dpgeorge dpgeorge added port-rp2 board-definition New or updated board definition files. Combine with a port- label. labels Mar 4, 2025
@SFE-Brudnerd
Copy link
Contributor Author

Image added in this PR: micropython/micropython-media#84

@SFE-Brudnerd SFE-Brudnerd marked this pull request as ready for review March 4, 2025 17:20
Signed-off-by: Alex Brudner <alex.brudner@sparkfun.com>
Signed-off-by: Alex Brudner <alex.brudner@sparkfun.com>
@SFE-Brudnerd SFE-Brudnerd requested a review from dpgeorge March 10, 2025 16:55
Signed-off-by: Malcolm McKellips <malcolm.mckellips@sparkfun.com>
@SFE-Brudnerd
Copy link
Contributor Author

Hi @dpgeorge have you had a chance to review this? I made the requested changes, and they were pretty simple. Unless there's further issues is there anything stopping this from merge? If you don't have the time to re-review, I totally get it, this is public and a lot of work is involved here. Is there someone else that can review it for merge?

@dpgeorge
Copy link
Member

Thanks for updating. It now builds, and the configuration looks good.

@dpgeorge
Copy link
Member

Squashed, rebased and merged in 9a070fe

@dpgeorge dpgeorge closed this Mar 13, 2025
@SFE-Brudnerd
Copy link
Contributor Author

Thanks so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
board-definition New or updated board definition files. Combine with a port- label. port-rp2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants