Skip to content

mbed_rtx.h file cleanup #9626

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

Closed
jeromecoutant opened this issue Feb 6, 2019 · 6 comments
Closed

mbed_rtx.h file cleanup #9626

jeromecoutant opened this issue Feb 6, 2019 · 6 comments

Comments

@jeromecoutant
Copy link
Collaborator

Description

Proposition is to clean every targets/TARGET_xxx/mbed_rtx.h files

Issue request type

[ ] Question
[x] Enhancement
[ ] Bug
@ciarmcom
Copy link
Member

ciarmcom commented Feb 6, 2019

Internal Jira reference: https://jira.arm.com/browse/MBOCUSTRIA-843

@0xc0170
Copy link
Contributor

0xc0170 commented Feb 7, 2019

cc @mprse

@mprse
Copy link
Contributor

mprse commented Feb 7, 2019

@deepikabhavnani prepares the general solution for moving to 2 region memory model. Handling of INITIAL_SP and HEAP_START/HEAP_SIZE/ISR_STACK_START/ISR_STACK_SIZE macros is a part of this process.

@deepikabhavnani
Copy link

@jeromecoutant
Agree with this enhancement, INITIAL_SP is not used in Mbed OS 5

* -Value INITIAL_SP is ignored

But we cannot remove it to maintain backward compatibility with Mbed 2 + CMSIS RTOS version 2. We will pick up this enhancement in future versions.

Along with this cleanup of linker files is needed as well. We have duplicate exports like __heap_limit and __HeapLimit

@cmonr
Copy link
Contributor

cmonr commented Feb 11, 2019

@SenRamakri Fyi

@ciarmcom
Copy link
Member

Thank you for raising this issue. Please note we have updated our policies and
now only defects should be raised directly in GitHub. Going forward questions and
enhancements will be considered in our forums, https://forums.mbed.com/ . If this
issue is still relevant please re-raise it there.
This GitHub issue will now be closed.

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

7 participants