fix: esp_app_desc_t.reserv3 should be [0;3] (#372)

* fix: esp_app_desc_t.reserv3 should be [0;3] see definition https://docs.espressif.com/projects/esp-idf/en/v5.4/esp32/api-reference/system/misc_system_api.html#_CPPv4N14esp_app_desc_t7reserv3E * update changelog
esp-rs · Feb 13, 2025 · 8aa9d07 · 8aa9d07
1 parent e662627
commit 8aa9d07
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [Unreleased]
 
 ### Fixed
+- Fix the `esp_app_desc!` macro so the reserv3 field is correct for ESP IDF v5.4
 - Fix the `esp_app_desc!` macro so that it properly embeds `CONFIG_ESP_EFUSE_BLOCK_REV_MIN_FULL`/`CONFIG_ESP_EFUSE_BLOCK_REV_MAX_FULL` with recent ESP-IDF versions (5.4+, 5.2.3+, 5.3.2+)
 - Make sure that the `esp_app_desc!` macro fails at build-time if some of the fields cannot fit inside the structure
 - Re-enable "esp_lcd_panel_rgb.h" after >= v5.3 for targets with SOC_LCD_RGB_SUPPORTED

diff --git a/src/app_desc.rs b/src/app_desc.rs
@@ -102,7 +102,7 @@ macro_rules! esp_app_desc {
                     esp_idf_version = "5.2",
                     esp_idf_version = "5.3"
                 )))]
-                reserv3: [0; 4],
+                reserv3: [0; 3],
                 #[cfg(not(any(
                     esp_idf_version_major = "4",
                     esp_idf_version = "5.0",