hw/mcu/stm32f4: Use common startup code #3205
Merged
Conversation
This file contains 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
kasjer
force-pushed
the
kasjer/stm32f4-common-startup
branch
from
65d208e
to
3328cb9
Compare
sjanc
approved these changes
May 15, 2024
Stack was always placed at the end of region RAM. This adds separate memory region for MCU stack. For some MCUs it is more convenient to place stack in separate memory space and that was not possible without custom linker script. __HeapLimit and __HeapBase assignments are simpler to read __StackTop and __StackLimi as well. dumplicate .bssnz section is removed (NOLOAD) attributes added to .bssnz and .bss .mtb section removed for now Signed-off-by: Jerzy Kasenberg <jerzy.kasenberg@codecoup.pl>
STM32F4 now uses common startup code for Cortex-M4 and autogenerated linker script. Flash cache initialization moved to hal_system_init.c to make system_stm32f4xx.c more like original ST file so future updates will be easier. Unused ld scripts remove Signed-off-by: Jerzy Kasenberg <jerzy.kasenberg@codecoup.pl>
This removes local startup code and linker scripts Signed-off-by: Jerzy Kasenberg <jerzy.kasenberg@codecoup.pl>
This adds memory memory_regions.ld.h for device that have CCM. MCU stack is also placed in CCM memory instead of RAM region. Signed-off-by: Jerzy Kasenberg <jerzy.kasenberg@codecoup.pl>
kasjer
force-pushed
the
kasjer/stm32f4-common-startup
branch
from
0205e0d
to
bcc4e55
Compare
Strip stm32 related files that were already removed from core. Signed-off-by: Jerzy Kasenberg <jerzy.kasenberg@codecoup.pl>
kasjer
force-pushed
the
kasjer/stm32f4-common-startup
branch
from
bcc4e55
to
15b655e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Change STM32F4 to use common startup code and liner script.
Additionally to utilize CCM memory common linker script for cortex-m4
introduces separate memory region for CMU stack.
Stack was always placed at the end of region RAM.
This adds separate memory region for MCU stack.
For some MCUs it is more convenient to place stack in separate memory
space and that was not possible without custom linker script.
__HeapLimit and __HeapBase assignments are simpler to read
__StackTop and __StackLimi as well.
dumplicate .bssnz section is removed
(NOLOAD) attributes added to .bssnz and .bss
.mtb section removed for now
Signed-off-by: Jerzy Kasenberg jerzy.kasenberg@codecoup.pl