Skip to content

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

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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

Signed-off-by: Brian Baltz <brian.a.baltz@intel.com>
@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.

Signed-off-by: Brian Baltz <brian.a.baltz@intel.com>
@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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants