Mirrors/esp-idf
1
0
Fork 0
mirror of https://github.com/espressif/esp-idf.git synced 2026-08-14 12:59:29 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix(esp_psram): fix unused-but-set-variable warning with GCC 16

Browse Source 
GCC 16 flags the volatile accumulator in the cache writeback loops even
though it is volatile, as it is only used to update itself. Drop the
accumulator and rely on the volatile-qualified psram pointer to keep
the cache-line reads from being optimized out.
This commit is contained in:
Mahavir Jain
2026-07-02 07:55:14 +05:30
parent bd567cf046
commit e16593ebc0
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
components/esp_psram/esp32/esp_psram_extram_cache.c
View File

@@ -24,7 +24,6 @@
void IRAM_ATTR esp_psram_extram_writeback_cache(void)
{
int x;
volatile int i = 0;
volatile uint8_t *psram = (volatile uint8_t*)SOC_EXTRAM_DATA_LOW;
int cache_was_disabled = 0;
@@ -51,7 +50,7 @@ void IRAM_ATTR esp_psram_extram_writeback_cache(void)
the cache by just reading 64K worth of cache lines.
*/
for (x = 0; x < 1024 * 64; x += 32) {
i += psram[x];
(void)psram[x];
}
#else
/*
@@ -61,8 +60,8 @@ void IRAM_ATTR esp_psram_extram_writeback_cache(void)
we ever support external RAM chips of 2M or smaller, this may need adjusting.
*/
for (x = 0; x < 1024 * 64; x += 32) {
i += psram[x];
i += psram[x + (1024 * 1024 * 2)];
(void)psram[x];
(void)psram[x + (1024 * 1024 * 2)];
}
#endif