Skip to content

soc: stm32: move stack size definitions to the correct place #92825

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.

Sign up for GitHub

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

Merged
merged 4 commits into from
Oct 17, 2025
Merged

soc: stm32: move stack size definitions to the correct place #92825

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
d3ca399
soc: st: stm32: add common kconfig symbols for kernel stack size
djiatsaf-st Oct 15, 2025
653818e
soc: st: stm32f0x: remove kernel stack size definitions
djiatsaf-st Jul 8, 2025
ab868fa
soc: st: stm32g0x: remove kernel stack size definitions
djiatsaf-st Jul 8, 2025
b003155
soc: st: stm32l0x: remove kernel stack size definitions
djiatsaf-st Jul 8, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 15 additions & 0 deletions soc/st/stm32/Kconfig.defconfig
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,21 @@ config CLOCK_CONTROL
config CORTEX_M_SYSTICK
default n if STM32_LPTIM_TIMER

# Reduce kernel stack sizes to fit MCUs with 8 KiB RAM or less
config MAIN_STACK_SIZE
default 320 if SRAM_SIZE <= 2
default 512 if SRAM_SIZE <= 4
default 640 if SRAM_SIZE <= 8

config IDLE_STACK_SIZE
default 100 if SRAM_SIZE <= 2
default 150 if SRAM_SIZE <= 4
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Despite this change is synced with the value previosly used in applicable SoCs, I find it strange the stack size is not a multiple of a 32bit word size.
This comment is outside the scope of this P-R, not blocking.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think Zephyr aligns the size internally.

default 200 if SRAM_SIZE <= 8

config ISR_STACK_SIZE
default 256 if SRAM_SIZE <= 2
default 512 if SRAM_SIZE <= 8

DT_STM32_RCC_PATH := $(dt_nodelabel_path,rcc)
DT_STM32_RCC_CLOCK_FREQ := $(dt_node_int_prop_int,$(DT_STM32_RCC_PATH),clock-frequency)

Expand Down