itsscb/esp-idf-hal
1
0
Fork 0
mirror of https://github.com/esp-rs/esp-idf-hal.git synced 2025-09-27 12:21:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix esp_reset_reason_t constant names (#483)

Browse Source 
The esp_reset_reason_t ESP_RST_PG and ESP_RST_CPU constant names don't
seem to exist. Instead, the actual names (added in esp-idf 5.2.2) are
ESP_RST_PWR_GLITCH and ESP_RST_CPU_LOCKUP.

The incorrect names caused the second-to-last match arm to be
interpreted as a binding name (that is, a catch-all), causing incorrect
behavior and build warnings when using esp-idf 5.2.2 and up.
This commit is contained in:
Eli Lipsitz 2024-09-26 03:19:21 -04:00 committed by GitHub
parent d3247c29c8
commit 3dbf3fe332
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/reset.rs
View File

@ -124,7 +124,7 @@ impl From<esp_reset_reason_t> for ResetReason {
esp_idf_version_full = "5.2.0",
esp_idf_version_full = "5.2.1",
)))]
esp_reset_reason_t_ESP_RST_PG => Self::PowerGlitch,
esp_reset_reason_t_ESP_RST_PWR_GLITCH => Self::PowerGlitch,
#[cfg(not(any(
esp_idf_version_major = "4",
all(esp_idf_version_major = "5", esp_idf_version_minor = "0"),
@ -132,7 +132,7 @@ impl From<esp_reset_reason_t> for ResetReason {
esp_idf_version_full = "5.2.0",
esp_idf_version_full = "5.2.1",
)))]
esp_reset_reason_t_ESP_RST_CPU => Self::CPULockup,
esp_reset_reason_t_ESP_RST_CPU_LOCKUP => Self::CPULockup,
_ => unreachable!(),
}
}