-
Notifications
You must be signed in to change notification settings - Fork 3k
Add MBED_SECTION attribute #3676
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
Conversation
Allow the section that code is placed in to be explicitly specified with the attribute MBED_SECTION.
@mbed-bot: TEST |
/morph test |
@mbed-bot: TEST HOST_OSES=ALL |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
[Build 1244] |
Not to be a stickler, but this should come in with a test for compiler support |
Since there is no guarantee on section names by the current linker files in the code base I can't think of a good way to test this. When bootloader support comes it will test this. Furthermore if linker script generation with named sections becomes a thing this will be easily testable. If you can think of a clean way to test this in its current form then I would be glad to add tests. |
Ah interesting, fair enough, go for it 👍 |
@c1728p9 What is the use case of this extension for the end user if it is not possible to provide custom linker script ? Is it for internal mbed-os internal use only ? |
@pan- this feature will be used by the bootloader to put data in a dedicated no-init ram region. It is possible to specify a custom linker script from the command line if someone want to use it for another purpose though. |
@pan- Basically
It is possible
|
@c1728p9 @theotherjimmy Thanks for the clarification. There's something I've missed apparently 😆 . |
Allow the section that code is placed in to be explicitly specified with the attribute MBED_SECTION.