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

STM32F3 common startup #3240

Merged
merged 6 commits into from
Jun 10, 2024
Merged

Conversation

kasjer
Copy link
Contributor

@kasjer kasjer commented Jun 5, 2024

This switches STM32F3 boards to use common startup code.
Additionally:

  • reset reason reporting fixed
  • nucleo-f303re download script was incorrect now all use common script
  • .cmd and local .ld and startup files removed

kasjer added 3 commits June 5, 2024 22:32
POR was not correctly reported.

Signed-off-by: Jerzy Kasenberg <[email protected]>
Signed-off-by: Jerzy Kasenberg <[email protected]>
STM32F3 now uses common startup code for Cortex-M4 and
autogenerated linker script.

Flash cache initialization moved to hal_system_init.c to
make system_stm32f3xx.c more like original ST file
so future updates will be easier.

Unused ld scripts remove

NVIC_NUM_VECTOR corrected (probably value was from other MCU)

Signed-off-by: Jerzy Kasenberg <[email protected]>
@kasjer kasjer force-pushed the kasjer/stm32f3-common-startup branch from 8019d3e to 80f84fa Compare June 5, 2024 21:20
kasjer added 2 commits June 6, 2024 00:03
This removes local startup code and linker scripts

Signed-off-by: Jerzy Kasenberg <[email protected]>
@kasjer kasjer force-pushed the kasjer/stm32f3-common-startup branch from 80f84fa to 0af6177 Compare June 5, 2024 22:03
STM32F303K8 board don't have space for bootloader in bsp.

Signed-off-by: Jerzy Kasenberg <[email protected]>
@kasjer kasjer added enhancement STM STM32 related CI labels Jun 6, 2024
@kasjer kasjer merged commit 7939d1e into apache:master Jun 10, 2024
17 checks passed
@kasjer kasjer deleted the kasjer/stm32f3-common-startup branch June 10, 2024 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants