-
Notifications
You must be signed in to change notification settings - Fork 503
Add support for MyMakers RP2350B board #3164
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
There was a problem hiding this 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.
There was a problem hiding this 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.
I have deleted
เมื่อ พฤ. 2 ต.ค. 2568 เวลา 10:07 Earle F. Philhower, III <
***@***.***> เขียนว่า:
… ***@***.**** requested changes on this pull request.
------------------------------
In tools/json/mymakers_rp2350b.json
<#3164 (comment)>
:
> @@ -0,0 +1,56 @@
+{
This file should not be here, either.
—
Reply to this email directly, view it on GitHub
<#3164 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/APWTIU7U7DGQNE7ZGA634R33VSJGHAVCNFSM6AAAAACH735NOOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTEOJRHA4TCOJVGA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
There was a problem hiding this 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.
This PR adds support for the MyMakers RP2350B board to the arduino-pico core. Changes include:
tools/makeboards.py
to generateboards.txt
with MyMakers RP2350Bjson/mymakers_rp2350b.json
for PlatformIOvariants/mymakers_rp2350b/pins_arduino.h
for pin mappingsREADME.md
to include MyMakers RP2350B in Supported BoardsPlease review and let me know if any changes are needed.