configs: stm32mp1: Remove misleading CONFIG_SPL_BSS_START_ADDR
authorAlexandru Gagniuc <mr.nuke.me@gmail.com>
Mon, 22 Mar 2021 13:20:00 +0000 (08:20 -0500)
committerPatrice Chotard <patrice.chotard@foss.st.com>
Fri, 9 Apr 2021 09:59:13 +0000 (11:59 +0200)
CONFIG_SPL_BSS_START_ADDR is only used on a few mach- linker scripts.
stm32mp1 uses the generic script under arch/arm/cpu/u-boot-spl.lds,
which does not make use of this definition.

The SPL BSS starts in SRAM, right after .text, .rodata, .data, and
.u_boot_list. A very short version of the STM32MP1 memory map is:
  * SYSRAM: 2ffc0000 - 30000000     <- all of SPL is here
  * DRAM:   c0000000+

0xC0200000 is a DRAM address, and has nothing to do with SPL. It is
just very misleading to have it next to CONFIG_SPL_BSS_MAX_SIZE, or to
have it at all.

Signed-off-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
Reviewed-by: Patrice Chotard <patrice.chotard@foss.st.com>
Reviewed-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
include/configs/stm32mp1.h

index 7fdb3ff..56a70cb 100644 (file)
@@ -50,7 +50,6 @@
 /* SPL support */
 #ifdef CONFIG_SPL
 /* SPL use DDR */
-#define CONFIG_SPL_BSS_START_ADDR      0xC0200000
 #define CONFIG_SPL_BSS_MAX_SIZE                0x00100000
 #define CONFIG_SYS_SPL_MALLOC_START    0xC0300000
 #define CONFIG_SYS_SPL_MALLOC_SIZE     0x01D00000