Skip to content

Conversation

nui4328
Copy link
Contributor

@nui4328 nui4328 commented Oct 1, 2025

This PR adds support for the MyMakers RP2350B board to the arduino-pico core. Changes include:

  • Modified tools/makeboards.py to generate boards.txt with MyMakers RP2350B
  • Added JSON configuration in json/mymakers_rp2350b.json for PlatformIO
  • Created variants/mymakers_rp2350b/pins_arduino.h for pin mappings
  • Updated README.md to include MyMakers RP2350B in Supported Boards
  • Tested with Arduino IDE:
    • Compiled and uploaded Blink sketch successfully
    • Verified Serial output and GPIO functionality (e.g., LED on GPIO25)
  • Board specifications:

Please review and let me know if any changes are needed.

Copy link
Owner

@earlephilhower earlephilhower left a comment

Choose a reason for hiding this comment

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

It looks like you've got a bad or very old version of the core for the patch. There should only be your single line addition to makeboards.py but you have changed/removed hundreds of lines with this PR, and 10,000 lines in boards.txt.

Please start from the latest core for your changes, and make sure to not edit boards.txt at all, it should be generated completely by makeboards.py but is showing hand edits in this PR, too.

@earlephilhower earlephilhower self-requested a review October 2, 2025 01:29
Copy link
Owner

@earlephilhower earlephilhower left a comment

Choose a reason for hiding this comment

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

Looking better, but there are now 2 variant.h and variant.json files for a single board. Please remove the unneeded set.

@nui4328
Copy link
Contributor Author

nui4328 commented Oct 2, 2025 via email

Copy link
Owner

@earlephilhower earlephilhower left a comment

Choose a reason for hiding this comment

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

Thanks, looks good now.

@earlephilhower earlephilhower merged commit 6aad689 into earlephilhower:master Oct 2, 2025
32 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