Skip to content

esp_reset_reason_set_hint() replacement by NVS flag (ESP32-S2)#128

Closed
T94T wants to merge 8 commits into
adafruit:masterfrom
T94T:master
Closed

esp_reset_reason_set_hint() replacement by NVS flag (ESP32-S2)#128
T94T wants to merge 8 commits into
adafruit:masterfrom
T94T:master

Conversation

@T94T
Copy link
Copy Markdown

@T94T T94T commented Jul 7, 2021

Since esp_reset_reason_set_hint() is not working/supported anymore (see: espressif/esp-idf#7230) I decided to rewrite the corresponding code using NVS storage instead.

Please note that this pull request does not work at the moment. There is still a compilation error:

tinyuf2/lib/esp-idf/components/nvs_flash/src/nvs_platform.hpp:31:10: fatal error: freertos/FreeRTOS.h: No such file or directory
 #include "freertos/FreeRTOS.h"
          ^~~~~~~~~~~~~~~~~~~~~

I hope someone can fix this.

@T94T
Copy link
Copy Markdown
Author

T94T commented Jul 8, 2021

will prepare the pull request again.

@T94T T94T closed this Jul 8, 2021
@hathach
Copy link
Copy Markdown
Member

hathach commented Jul 9, 2021

@T94T I dived a bit into this, there is no need for using NVS, the reason hint is still accessible within stage2 bootloader. IDF just slightly changes the reset reason for esp_restart(). I will make an PR to correct this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants