sergiotarxz/nuttx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

esp32s3_textheap.c: fix range checks

Browse Source 
While there constants are commented as "First and last words of the
D/IRAM region", the _HIGH addresses are actually not in the corresponding
region.
This commit is contained in:
YAMAMOTO Takashi 2024-05-15 09:30:13 +09:00 committed by Alan Carvalho de Assis
parent 10c8c39426
commit 5df4925672
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
arch/xtensa/src/esp32s3/esp32s3_textheap.c
View File

@ -93,7 +93,7 @@ void *up_textheap_memalign(size_t align, size_t size)
*/ */
uintptr_t addr = (uintptr_t)ret; uintptr_t addr = (uintptr_t)ret;
if (SOC_DIRAM_DRAM_LOW <= addr && addr <= SOC_DIRAM_DRAM_HIGH) if (SOC_DIRAM_DRAM_LOW <= addr && addr < SOC_DIRAM_DRAM_HIGH)
{ {
addr = MAP_DRAM_TO_IRAM(addr); addr = MAP_DRAM_TO_IRAM(addr);
} }
@ -135,7 +135,7 @@ void up_textheap_free(void *p)
#endif #endif
{ {
uintptr_t addr = (uintptr_t)p; uintptr_t addr = (uintptr_t)p;
if (SOC_DIRAM_IRAM_LOW <= addr && addr <= SOC_DIRAM_IRAM_HIGH) if (SOC_DIRAM_IRAM_LOW <= addr && addr < SOC_DIRAM_IRAM_HIGH)
{ {
addr = MAP_IRAM_TO_DRAM(addr); addr = MAP_IRAM_TO_DRAM(addr);
} }
@ -181,7 +181,7 @@ bool up_textheap_heapmember(void *p)
#endif #endif
uintptr_t addr = (uintptr_t)p; uintptr_t addr = (uintptr_t)p;
if (SOC_DIRAM_IRAM_LOW <= addr && addr <= SOC_DIRAM_IRAM_HIGH) if (SOC_DIRAM_IRAM_LOW <= addr && addr < SOC_DIRAM_IRAM_HIGH)
{ {
addr = MAP_IRAM_TO_DRAM(addr); addr = MAP_IRAM_TO_DRAM(addr);
} }