Shrink flash and RAM usage even more #627
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Remove MIDI support from the core's main TinyUSB. MIDI is still supported using the Adafruit TinyUSB library, just like before (the core never did use it). Compile the Pico-SDK using `-Os` Remove unneeded warning in UF2 upload Blink.ino shows a savings of 3.5KB flash and 400 bytes of RAM. For comparison, with this PR blink.ino reports: ```` Sketch uses 49908 bytes (3%) of program storage space. Maximum is 1568768 bytes. Global variables use 7024 bytes (2%) of dynamic memory, leaving 255120 bytes for local variables. Maximum is 262144 bytes. ```` As comparison, using the MBED core blink.ino reports: ```` Sketch uses 78882 bytes (0%) of program storage space. Maximum is 16777216 bytes. Global variables use 42780 bytes (15%) of dynamic memory, leaving 227556 bytes for local variables. Maximum is 270336 bytes. ```` So, with this PR we use **37% less flash** and and **87% less RAM** for simple sketches.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Remove MIDI support from the core's main TinyUSB. MIDI is still supported
using the Adafruit TinyUSB library, just like before (the core never
did use it).
Compile the Pico-SDK using
-OsRemove unneeded warning in UF2 upload
Blink.ino shows a savings of 3.5KB flash and 400 bytes of RAM.
For comparison, with this PR blink.ino reports:
As comparison, using the MBED core blink.ino reports:
So, with this PR we use 37% less flash and and 87% less RAM
for simple sketches.