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:
us...@infra.apache.org


Reply via email to