[GitHub] [incubator-nuttx] btashton opened a new pull request #2364: stm32h7: Fix bug where more than CONFIG_MM_REGIONS would be allocated

[GitBox <gi...@apache.org>]

btashton opened a new pull request #2364:
URL: https://github.com/apache/incubator-nuttx/pull/2364


   ## Summary
   The current logic for ` void arm_addregion(void)` for the stm32h7 would check that the number of calls to addregion was less than `CONFIG_MM_REGIONS` but would not consider that the AXI memory was already allocated for the kernel heap.  This would result in DTCM getting allocated when there were no more allocation slots.  This fails `DEBUGASSERT`.
   
   This change makes `arm_addregion` aware of the existing allocation and also increases the memory regions in the configs.
   
   ## Impact
   DEBUGASSERT is no longer fired on the stm32h7 configurations and DTCM can actually be used when enabled.
   
   ## Testing
   Build with debug asserts enabled and memory debug on to verify the regions are allocated.  Also ran the memtest tool.
   


----------------------------------------------------------------
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 #2364: stm32h7: Fix bug where more than CONFIG_MM_REGIONS would be allocated

[GitBox <gi...@apache.org>]

xiaoxiang781216 merged pull request #2364:
URL: https://github.com/apache/incubator-nuttx/pull/2364


   


----------------------------------------------------------------
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