Skip to content

Add RAM/ROM memory statistics to system stats structure #8661

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

Add RAM/ROM memory statistics to system stats structure #8661

wants to merge 2 commits into from

Conversation

deepikabhavnani
Copy link

@deepikabhavnani deepikabhavnani commented Nov 6, 2018 •
edited
Loading

Description

Internal RAM / ROM memory size and start address of target can be fetched using mbed_stats_sys_get() API.

TODO

Pull request type

[ ] Fix
[ ] Refactor
[ ] Target update
[X] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

CC @SenRamakri

@deepikabhavnani deepikabhavnani mentioned this pull request Nov 6, 2018
Merged
@cmonr cmonr requested review from SenRamakri and kegilbert November 7, 2018 03:17
kegilbert
kegilbert reviewed Nov 7, 2018
View reviewed changes
kegilbert
kegilbert approved these changes Nov 7, 2018
View reviewed changes
Copy link
Contributor

@kegilbert kegilbert left a comment

Choose a reason for hiding this comment

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

Very cool!

@deepikabhavnani deepikabhavnani mentioned this pull request Nov 7, 2018
Merged
Deepika added 2 commits November 7, 2018 09:06
Add RAM/ROM memory statistics to system stats structure
c0d7177 
Internal RAM / ROM memory size and start address of target can be
fetched using `mbed_stats_sys_get()` API.
Add tests to verify RAM/ROM sizes
a559f1a
@cmonr cmonr added the rollup PR label Nov 7, 2018
@cmonr cmonr mentioned this pull request Nov 7, 2018
Closed
@cmonr
Copy link
Contributor

cmonr commented Nov 7, 2018

Info: This PR is being tested as a part of a rollup PR (#8669).

@cmonr
Copy link
Contributor

cmonr commented Nov 8, 2018

@deepikabhavnani Please refer to the following build error: #8669 (comment)

@deepikabhavnani
Copy link
Author

Added commits from this PR to #8607

@deepikabhavnani deepikabhavnani deleted the update_ram_size branch November 14, 2018 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kegilbert kegilbert kegilbert approved these changes

@SenRamakri SenRamakri SenRamakri approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@deepikabhavnani @cmonr @kegilbert @SenRamakri @studavekar