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

Max RAM indication not present #200

Closed
Testato opened this issue Aug 13, 2016 · 11 comments
Closed

Max RAM indication not present #200

Testato opened this issue Aug 13, 2016 · 11 comments

Comments

@Testato
Copy link
Contributor

Testato commented Aug 13, 2016

On Arduino at the end of a build, in consolle appear the used/maximum flash size and the used/maximum RAM size

On STM32duino the maximum RAM do not appear

Sketch uses 12,316 bytes (11%) of program storage space. Maximum is 110,592 bytes.
Global variables use 2,560 bytes of dynamic memory.
@Testato Testato changed the title Max RAMindication not present Max RAM indication not present Aug 13, 2016
@rogerclarkmelbourne
Copy link
Owner

Please can you suggest a solution and submit via PR

@rogerclarkmelbourne
Copy link
Owner

FYI.
This appears to be a data parameter that's optional in boards.txt

upload.maximum_data_size

Introduced in

arduino/Arduino@7c87db3#diff-4ead488680453362ffec01d74609315c

Please can you update boards.txt to add this value to all boards (including F1 and F4 repos)

Thanks

@Testato
Copy link
Contributor Author

Testato commented Aug 14, 2016

Yes i can prepare a PR,
But i checked also on Arduino Zero and this bug is present too.
Very strange

@rogerclarkmelbourne
Copy link
Owner

You could do a quick test by updating one board in boards.txt and check it works.

I'm not sure why they omitted this for the Zero, I suspect they did what we did, and used an old file which doesn't have that data in it.

It doesnt seem to be documented in the Arduino Third Party Boards specification.
https://github.com/arduino/Arduino/wiki/Arduino-IDE-1.5-3rd-party-Hardware-specification

(I'm not surprised as Arduino don't seem to spend much time on documentation :-( )

BTW. While I remember. I think some of the sizes defined in boards.txt are wrong, I can't remember if its just the Flash size thats wrong or whether the Ram size is wrong as well

@rogerclarkmelbourne
Copy link
Owner

closed due to inactivity

@Testato
Copy link
Contributor Author

Testato commented Dec 16, 2016

i agree that this issue have inactivity, but i think it will be a very good upgrade for the core.
It is very usefull have this info during compilation.
This is an example from the Uno

Sketch uses 444 bytes (1%) of program storage space. Maximum is 32,256 bytes.
Global variables use 9 bytes (0%) of dynamic memory, leaving 2,039 bytes for local variables. Maximum is 2,048 bytes.

So may you remain it open ?

@rogerclarkmelbourne
Copy link
Owner

@Testato

Please send a PR for this

Testato added a commit to Testato/Arduino_STM32 that referenced this issue Dec 17, 2016
Testato added a commit to Testato/Arduino_STM32 that referenced this issue Dec 17, 2016
PR for issue rogerclarkmelbourne#200

- replaced `upload.flash.maximum_size` and `upload.ram.maximum_size` by the new arduino standard `upload.maximum_size` and `upload.maximum_data_size`
- corrected Flash and Ram value
- Tested by builded
Testato added a commit to Testato/Arduino_STM32 that referenced this issue Dec 17, 2016
PR for issue rogerclarkmelbourne#200

- replaced `upload.flash.maximum_size` and `upload.ram.maximum_size` by the new arduino standard `upload.maximum_size` and `upload.maximum_data_size`
- corrected Flash and Ram value
- Tested by builded every MCU on every Variant
@Testato
Copy link
Contributor Author

Testato commented Dec 18, 2016

@rogerclarkmelbourne
i created 3 separated PR for the 3 board.txt correction

@Testato
Copy link
Contributor Author

Testato commented Mar 23, 2017

Is there problems whit my PRs ?

@rogerclarkmelbourne
Copy link
Owner

Not had time to test

@rogerclarkmelbourne
Copy link
Owner

Merged (ages ago)

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

No branches or pull requests

2 participants