Skip to content

Fixing sketch size calculation (text + ctors + rodata + datas #141

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

Merged
merged 2 commits into from
Mar 8, 2016

Conversation

bbaltz505
Copy link
Contributor

Reopening closed PR #126 to fix sketch size.

@bbaltz505
Copy link
Contributor Author

@pricopb @SidLeung

@bbaltz505
Copy link
Contributor Author

...\Blink.ino.elf  :
section     size         addr
text       13600   1073954816 <-- 
ctors         12   1073968416 <-- 
rodata       440   1073968428 <-- 
datas       3156   2818629632 <--
bss         1916   2818632788
heap        8192   2818634704
stack       2560   2818642896
.comment     118            0
Total      29994

The IDE then reports:

Sketch uses 17,208 bytes (8%) of program storage space. Maximum is 196,608 bytes.

@SidLeung
Copy link
Contributor

SidLeung commented Mar 8, 2016

@bbaltz505 We only have 152 KB of Flash storage for the current setup. Can change that in future releases.

@bbaltz505
Copy link
Contributor Author

Ok. i can update boards.txt. What is the exact # of bytes? 155,648?

@SidLeung
Copy link
Contributor

SidLeung commented Mar 8, 2016

Yes, it's 155,648 bytes.

@bbaltz505
Copy link
Contributor Author

@SidLeung Corrected the max sketch size.

@SidLeung
Copy link
Contributor

SidLeung commented Mar 8, 2016

@calvinatintel Looks good. Ready to merge. Thank you, Brian.

bbaltz505 pushed a commit that referenced this pull request Mar 8, 2016
Fixing sketch size calculation (text + ctors + rodata + datas
@bbaltz505 bbaltz505 merged commit e50ef90 into arduino:master Mar 8, 2016
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