Skip to content

Commit

Permalink
kconfig: Set BOOTLOADER_SRAM_SIZE default to 0
Browse files Browse the repository at this point in the history
The current default value for `BOOTLOADER_SRAM_SIZE` is set to an
arbitrary-seeming value of `16`, which feels like a random magic
number.

Given that adding an offset in SRAM for a bootloader should always be
a conscious choice for a specific platform, this PR sets the default
to `0`. An appropriate value can be set at the SoC or board level,
overriding this default.

Signed-off-by: Kevin Townsend <kevin.townsend@linaro.org>
  • Loading branch information
microbuilder authored and fabiobaltieri committed Jul 17, 2023
1 parent 0e35bbb commit f68cee9
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
2 changes: 1 addition & 1 deletion Kconfig.zephyr
Original file line number Diff line number Diff line change
Expand Up @@ -827,7 +827,7 @@ config IS_BOOTLOADER

config BOOTLOADER_SRAM_SIZE
int "SRAM reserved for bootloader"
default 16
default 0
depends on !XIP || IS_BOOTLOADER
depends on ARM || XTENSA
help
Expand Down
4 changes: 4 additions & 0 deletions doc/releases/release-notes-3.5.rst
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,10 @@ API Changes
Changes in this release
=======================

* Set :kconfig:option:`CONFIG_BOOTLOADER_SRAM_SIZE` default value to ``0`` (was
``16``). Bootloaders that use a part of the SRAM should set this value to an
appropriate size. :github:`60371`

Removed APIs in this release
============================

Expand Down

0 comments on commit f68cee9

Please sign in to comment.