STM32 baremetal support step1 (F0/F1/F3/H7/L0) #12992
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,54 +1,52 @@ | ||
| #! armcc -E | ||
| ; Scatter-Loading Description File | ||
| ; | ||
| ; SPDX-License-Identifier: BSD-3-Clause | ||
| ;****************************************************************************** | ||
| ;* @attention | ||
| ;* | ||
| ;* Copyright (c) 2014-2020 STMicroelectronics. | ||
| ;* All rights reserved. | ||
| ;* | ||
| ;* This software component is licensed by ST under BSD 3-Clause license, | ||
| ;* the "License"; You may not use this file except in compliance with the | ||
| ;* License. You may obtain a copy of the License at: | ||
| ;* opensource.org/licenses/BSD-3-Clause | ||
| ;* | ||
| ;****************************************************************************** | ||
|
|
||
| #if !defined(MBED_APP_START) | ||
| #define MBED_APP_START MBED_ROM_START | ||
| #endif | ||
|
|
||
| #if !defined(MBED_APP_SIZE) | ||
| #define MBED_APP_SIZE MBED_ROM_SIZE | ||
| #endif | ||
|
|
||
| #if !defined(MBED_BOOT_STACK_SIZE) | ||
| /* This value is normally defined by the tools to 0x1000 for bare metal and 0x400 for RTOS */ | ||
| #define MBED_BOOT_STACK_SIZE 0x400 | ||
| #endif | ||
|
|
||
| /* Round up VECTORS_SIZE to 8 bytes */ | ||
| #define NVIC_NUM_VECTORS 48 | ||
| #define VECTORS_SIZE (((NVIC_NUM_VECTORS * 4) + 7) AND ~7) | ||
|
|
||
| LR_IROM1 MBED_APP_START MBED_APP_SIZE { | ||
|
|
||
| ER_IROM1 MBED_APP_START MBED_APP_SIZE { | ||
| *.o (RESET, +First) | ||
| *(InRoot$$Sections) | ||
| .ANY (+RO) | ||
| } | ||
|
|
||
| RW_IRAM1 (MBED_RAM_START + VECTORS_SIZE) { ; RW data | ||
| .ANY (+RW +ZI) | ||
| } | ||
|
|
||
| ARM_LIB_HEAP AlignExpr(+0, 16) EMPTY (MBED_RAM_START + MBED_RAM_SIZE - MBED_BOOT_STACK_SIZE - AlignExpr(ImageLimit(RW_IRAM1), 16)) { ; Heap growing up | ||
| } | ||
|
|
||
| ARM_LIB_STACK (MBED_RAM_START + MBED_RAM_SIZE) EMPTY -MBED_BOOT_STACK_SIZE { ; Stack region growing down | ||
| } | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,36 +1,59 @@ | ||
| /* Linker script to configure memory regions. | ||
| * | ||
| * SPDX-License-Identifier: BSD-3-Clause | ||
| ****************************************************************************** | ||
| * @attention | ||
| * | ||
| * Copyright (c) 2016-2020 STMicroelectronics. | ||
| * All rights reserved. | ||
| * | ||
| * This software component is licensed by ST under BSD 3-Clause license, | ||
| * the "License"; You may not use this file except in compliance with the | ||
| * License. You may obtain a copy of the License at: | ||
| * opensource.org/licenses/BSD-3-Clause | ||
| * | ||
| ****************************************************************************** | ||
| */ | ||
| /* Device specific values */ | ||
|
|
||
| /* Tools provide -DMBED_ROM_START=xxx -DMBED_ROM_SIZE=xxx -DMBED_RAM_START=xxx -DMBED_RAM_SIZE=xxx */ | ||
|
|
||
| define symbol VECTORS = 48; /* This value must match NVIC_NUM_VECTORS in cmsis_nvic.h */ | ||
| define symbol HEAP_SIZE = 0x1000; | ||
|
|
||
| /* Common - Do not change */ | ||
|
|
||
| if (!isdefinedsymbol(MBED_APP_START)) { | ||
| define symbol MBED_APP_START = MBED_ROM_START; | ||
| } | ||
|
|
||
| if (!isdefinedsymbol(MBED_APP_SIZE)) { | ||
| define symbol MBED_APP_SIZE = MBED_ROM_SIZE; | ||
| } | ||
|
|
||
| if (!isdefinedsymbol(MBED_BOOT_STACK_SIZE)) { | ||
| /* This value is normally defined by the tools | ||
| to 0x1000 for bare metal and 0x400 for RTOS */ | ||
| define symbol MBED_BOOT_STACK_SIZE = 0x400; | ||
| } | ||
|
|
||
| /* Round up VECTORS_SIZE to 8 bytes */ | ||
| define symbol VECTORS_SIZE = ((VECTORS * 4) + 7) & ~7; | ||
| define symbol RAM_REGION_START = MBED_RAM_START + VECTORS_SIZE; | ||
| define symbol RAM_REGION_SIZE = MBED_RAM_SIZE - VECTORS_SIZE; | ||
|
|
||
| define memory mem with size = 4G; | ||
| define region ROM_region = mem:[from MBED_APP_START size MBED_APP_SIZE]; | ||
| define region RAM_region = mem:[from RAM_REGION_START size RAM_REGION_SIZE]; | ||
|
|
||
| define block CSTACK with alignment = 8, size = MBED_BOOT_STACK_SIZE { }; | ||
| define block HEAP with alignment = 8, size = HEAP_SIZE { }; | ||
|
|
||
| initialize by copy { readwrite }; | ||
| do not initialize { section .noinit }; | ||
|
|
||
| place at address mem: MBED_APP_START { readonly section .intvec }; | ||
|
|
||
| place in ROM_region { readonly }; | ||
| place in RAM_region { readwrite, | ||
| block CSTACK, block HEAP }; | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is already defined in
cmsis_nvic.h.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, but cmsis_nvic.h for M0 core is not so simple as other cores.
So redefining value here is a good workaround.
I ill make it better later
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont understand, we can't get the value from the header file to be in here ? Or at least to check if its not already defined ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, we can, but no with a "quick" patch.
So I used the method described in https://os.mbed.com/docs/mbed-os/v5.15/porting/porting-bootstrap.html
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is it also in
cmsis_nvic.h?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
because mbed needs it...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
True. Linker script should do
#include "../cmsis_nvic.h"and not define it again. This is a bug in our docs.@hugueskamba can you create an issue if you agree? the docs should be updated. The default place in our targets is cmsis_nvic.h file, and linker script should include it (or we need to change this and refactor most of our targets).