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

[Kinsey Moore]

Ok, it sounds like that's quite a bit more involved than just updating 
this mapping so I'll drop this patch for the time being.


Kinsey

On 7/12/2023 11:12 AM, Karel Gardas wrote:
>
> 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}
>>     }
>