You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/04/06 18:24:39 UTC
[GitHub] [incubator-nuttx] antmerlino opened a new pull request #3457: stm32h7: Actually use the AXI SRAM as the main heap as the documentation describes.
antmerlino opened a new pull request #3457:
URL: https://github.com/apache/incubator-nuttx/pull/3457
## Summary
stm32h7: Actually use the AXI SRAM as the main heap as the documentation describes.
The comments at the top of the file say this:
```
* - AXI SRAM is a 512kb memory area. This will be automatically registered
* with the system heap in up_allocate_heap, all the other memory
* regions will be registered in arm_addregion().
* So, CONFIG_MM_REGIONS must be at least 1 to use AXI SRAM.
*
* - Internal SRAM is available in all members of the STM32 family.
* This is always registered with system heap.
* There are two contiguous regions of internal SRAM:
* SRAM1+SRAM2+SRAM3 and SRAM4 at a separate address.
* So, add 2 more to CONFIG_MM_REGIONS.
*
* - Tightly Coupled Memory (TCM RAM), we can use Data TCM (DTCM) for system
* heap. Note that DTCM has a number of limitations, for example DMA
* transfers to/from DTCM are limited.
* Define CONFIG_STM32H7_DTCMEXCLUDE to exclude the DTCM from heap.
* +1 to CONFIG_MM_REGIONS if you want to use DTCM.
*
* - External SDRAM can be connected to the FMC peripherial. Initialization
* of FMC is done as arm_addregion() will invoke stm32_fmc_init().
* Please read the comment in stm32_fmc.c how to initialize FMC
* correctly.
*
```
but the implementation was using SRAM123 instead. Furthermore, arm_addregion then re-adds SRAM123 again.
## Impact
The location of all heap memories for H7's will change. SRAM123 will no longer be used as the kernel heap and will not be added twice.
## Testing
Single board testing. Would be good to have other's test this with their configurations.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #3457: stm32h7: Actually use the AXI SRAM as the main heap as the documentation describes in PROTECTED mode.
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #3457:
URL: https://github.com/apache/incubator-nuttx/pull/3457
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org