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

mbed_retarget: Add an internal class to provide basic console functionality #4

Closed
wants to merge 1 commit into from

Conversation

hugueskamba
Copy link
Owner

@hugueskamba hugueskamba commented Oct 18, 2019

Description

The retarget code allocates an array of FileHandle* for console and file
handling (filehandles). A tiny target only needs a console (putc/getc).
There is no need for file handling.

The POSIX layer and the array of FileHandle* is not required for small
targets that only need a console ; this code is optionally compiled
out if the configuration parameter platform.stdio-minimal-console-only is
set to "true".

Pull request type

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

Reviewers

Release Notes

mbed-os-example-blinky (master) for K64F

Toolchain GCC_ARM

Total Static RAM memory (data + bss): 12096 bytes
Total Flash memory (text + data): 66428 bytes

Toolchain ARM

Total Static RAM memory (data + bss): 205879 bytes
Total Flash memory (text + data): 47625 bytes

platform/source/mbed_retarget.cpp Outdated Show resolved Hide resolved
platform/mbed_lib.json Outdated Show resolved Hide resolved
platform/source/mbed_retarget.cpp Outdated Show resolved Hide resolved
platform/source/mbed_retarget.cpp Show resolved Hide resolved
platform/source/mbed_retarget.cpp Show resolved Hide resolved
platform/source/mbed_retarget.cpp Show resolved Hide resolved
platform/source/mbed_retarget.cpp Outdated Show resolved Hide resolved
platform/source/mbed_retarget.cpp Show resolved Hide resolved
platform/source/mbed_retarget.cpp Outdated Show resolved Hide resolved
platform/source/mbed_retarget.cpp Show resolved Hide resolved
@hugueskamba hugueskamba force-pushed the hk-add-minimal-console-to-retarget branch 10 times, most recently from 51afce6 to 6b68737 Compare October 28, 2019 11:04
platform/mbed_retarget.h Outdated Show resolved Hide resolved
@hugueskamba hugueskamba force-pushed the hk-add-minimal-console-to-retarget branch 5 times, most recently from f3f1c0f to 043ec8b Compare October 31, 2019 10:05
…nality

The retarget code allocates an array of FileHandle* for console and file
handling (filehandles). A tiny target only needs a console (putc/getc).
There is no need for file handling.

The POSIX layer and the array of FileHandle* is not required for small
targets that only need a console ; this code is optionally compiled
out if the configuration parameter platform.stdio-minimal-console-only is
set to `"true"`.
@hugueskamba hugueskamba force-pushed the hk-add-minimal-console-to-retarget branch from 043ec8b to c204bf1 Compare October 31, 2019 11:54
@hugueskamba
Copy link
Owner Author

This force-push adds doxygen documentation for minimal_console_write and minimal_console_read.

Copy link
Collaborator

@evedon evedon left a comment

Choose a reason for hiding this comment

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

Go ahead and make a PR to mbed-os

@hugueskamba hugueskamba closed this Nov 1, 2019
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