[PATCH v2 2/2] bsps/stm32h7: Move SDRAM1 to correct memory range

[Karel Gardas]

Hello Kinsley,

you are indeed right. I've not fixed this bit since it also requires 
fixes in linker scripts and additional sdram region for sdram2 remap.

E.g. Original Sebastian's code is using SDRAM_1 as a remap of SDRAM_2. 
Linker script(s) then are using SDRAM_1 as an executable region (remap 
of SDRAM_2). If you fix SDRAM_1 to be real SDRAM_1, then we will need 
SDRAM_2_REMAP defined and fix also all SDRAM_1 occurances in linker 
scripts and replace those with SDRAM_2_REMAP.

Or that at least how I understand it...

Thanks,
Karel

On 7/12/23 18:00, Kinsey Moore wrote:
> According to the documentation in STM reference manuals RM0399 and
> RM0433, the standard memory space for SDRAM bank 1 is 0xc0000000 to
> 0xcfffffff.
> ---
>   spec/build/bsps/arm/stm32h7/linkcmdsmemory.yml | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/spec/build/bsps/arm/stm32h7/linkcmdsmemory.yml b/spec/build/bsps/arm/stm32h7/linkcmdsmemory.yml
> index 88dd4e8c91..2b3aacce5d 100644
> --- a/spec/build/bsps/arm/stm32h7/linkcmdsmemory.yml
> +++ b/spec/build/bsps/arm/stm32h7/linkcmdsmemory.yml
> @@ -14,9 +14,9 @@ content: |
>       SRAM_BACKUP : ORIGIN = 0x38800000, LENGTH = ${STM32H7_MEMORY_SRAM_BACKUP_SIZE:#010x}
>       PERIPHERAL  : ORIGIN = 0x40000000, LENGTH = ${STM32H7_MEMORY_PERIPHERAL_SIZE:#010x}
>       NOR         : ORIGIN = 0x60000000, LENGTH = ${STM32H7_MEMORY_NOR_SIZE:#010x}
> -    SDRAM_1     : ORIGIN = 0x70000000, LENGTH = ${STM32H7_MEMORY_SDRAM_1_SIZE:#010x}
>       NAND        : ORIGIN = 0x80000000, LENGTH = ${STM32H7_MEMORY_NAND_SIZE:#010x}
>       QUADSPI     : ORIGIN = 0x90000000, LENGTH = ${STM32H7_MEMORY_QUADSPI_SIZE:#010x}
> +    SDRAM_1     : ORIGIN = 0xc0000000, LENGTH = ${STM32H7_MEMORY_SDRAM_1_SIZE:#010x}
>       SDRAM_2     : ORIGIN = 0xd0000000, LENGTH = ${STM32H7_MEMORY_SDRAM_2_SIZE:#010x}
>     }
>