Skip to content
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

Add cp sapling rev b #4677

Merged
merged 8 commits into from Apr 30, 2021
Merged

Add cp sapling rev b #4677

merged 8 commits into from Apr 30, 2021

Conversation

skerr92
Copy link

@skerr92 skerr92 commented Apr 27, 2021

Creating a draft PR for the CP Sapling Rev B board. Will update to open once I can confirm all changes.

@skerr92 skerr92 marked this pull request as ready for review April 28, 2021 00:23
@skerr92
Copy link
Author

skerr92 commented Apr 28, 2021

PID pending on Pid.Codes --> PID Codes PID PR

@skerr92
Copy link
Author

skerr92 commented Apr 28, 2021

This is failing on the Japanese language build. Not sure why it ends up being bigger than the on board flash.. will need to investigate this...

@ladyada
Copy link
Member

ladyada commented Apr 28, 2021

you can add a CFLAGS_INLINE_LIMIT = 35 for japanese too

@skerr92
Copy link
Author

skerr92 commented Apr 30, 2021

you can add a CFLAGS_INLINE_LIMIT = 35 for japanese too

Not sure why when I added that flag it was still failing. I ended up looking at the Qt Py M0 mpconfigboard.mk file and matching that and it's built without issue. not sure what the reasoning for that ends up being.

Copy link
Member

@gamblor21 gamblor21 left a comment

Choose a reason for hiding this comment

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

Looks good to me

@gamblor21 gamblor21 merged commit 5679eb4 into adafruit:main Apr 30, 2021
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.

None yet

3 participants