bugfix(gmf_io): Fixed file io cache memory caps

Author: ThunderDai

elements/gmf_io/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## v0.7.3
+
+### Bug Fixes
+
+- Fixed cache caps issues for esp32 in `io_file`
+
 ## v0.7.2
 
 ### Features

elements/gmf_io/esp_gmf_io_file.c

@@ -111,6 +111,7 @@ static esp_gmf_err_t _file_open(esp_gmf_io_handle_t io)
         ESP_GMF_MEM_VERIFY(TAG, file_io->cache, { fclose(file_io->file); return ESP_GMF_ERR_MEMORY_LACK;},
                             "file stream cache", file_io->cache_size);
         setvbuf(file_io->file, (char *)file_io->cache, _IOFBF, file_io->cache_size);
+        ESP_LOGD(TAG, "File_io cache: %p, size: %d, caps: 0x%x", file_io->cache, file_io->cache_size, file_io->cache_caps);
     }
 
     file_io->is_open = true;
@@ -287,10 +288,9 @@ esp_gmf_err_t esp_gmf_io_file_init(file_io_cfg_t *config, esp_gmf_io_handle_t *i
     if (cfg->cache_size <= IO_FILE_DEFAULT_CACHE_ALIGN) {
         file_io->cache_size = 0;
     } else {
-        file_io->cache_caps = (cfg->cache_caps == 0) ? (MALLOC_CAP_INTERNAL | MALLOC_CAP_CACHE_ALIGNED) : cfg->cache_caps;
+        file_io->cache_caps = (cfg->cache_caps == 0) ? (MALLOC_CAP_DMA) : cfg->cache_caps;
         file_io->cache_size = IO_FILE_ALIGN_UP(cfg->cache_size, IO_FILE_DEFAULT_CACHE_ALIGN);
     }
-    ESP_LOGD(TAG, "Initialization, %s-%p, cache_size: %d", OBJ_GET_TAG(obj), file_io, file_io->cache_size);
     return ESP_GMF_ERR_OK;
 _file_fail:
     esp_gmf_obj_delete(obj);

elements/gmf_io/idf_component.yml

@@ -1,4 +1,4 @@
-version: "0.7.2"
+version: "0.7.3"
 description: Espressif GMF IO Module
 url: https://github.com/espressif/esp-gmf/tree/main/elements/gmf_io
 documentation: "https://github.com/espressif/esp-gmf/blob/main/elements/gmf_io/README.md"

elements/gmf_io/include/esp_gmf_io_file.h

@@ -21,9 +21,12 @@ typedef struct {
     const char  *name;        /*!< Name for this instance */
     int          cache_size;  /*!< Cache size for file IO operations in bytes. If size <= 512, it will be set to 0.
                                    Note: Larger cache size will improve read and write performance but consume more memory */
-    int          cache_caps;  /*!< Cache memory capabilities, if zero then it will be set to (MALLOC_CAP_INTERNAL | MALLOC_CAP_CACHE_ALIGNED).
-                                   Note: If chips have SOC_SDMMC_PSRAM_DMA_CAPABLE capability(such as ESP32P4),
-                                         then you can set (MALLOC_CAP_SPIRAM | MALLOC_CAP_CACHE_ALIGNED) to save SRAM */
+    int          cache_caps;  /*!< Cache memory capabilities, if zero then it will be set to MALLOC_CAP_DMA.
+                                   Note:
+                                        1. If chips have SOC_SDMMC_PSRAM_DMA_CAPABLE capability(such as ESP32P4),
+                                            then you can set (MALLOC_CAP_SPIRAM | MALLOC_CAP_DMA) to save SRAM
+                                        2. For ESP32, should use (MALLOC_CAP_INTERNAL | MALLOC_CAP_8BIT) or MALLOC_CAP_DMA caps to malloc cache
+                                        3. For ESP32Sxx and ESP32Cxx, can also use MALLOC_CAP_INTERNAL caps to malloc cache */
 } file_io_cfg_t;
 
 #define FILE_IO_CFG_DEFAULT() {     \