-
Notifications
You must be signed in to change notification settings - Fork 7.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes EspClass::deepSleep(64 bits) #9077
Conversation
This will change time in microseconds from 32 bits to 64 bits as defined in IDF.
👋 Hello SuGlider, we appreciate your contribution to this project! Click to see more instructions ...
Review and merge process you can expect ...
|
@me-no-dev - Please merge this PR to |
* Fixes EspClass::deepSleep(64 bits) This will change time in microseconds from 32 bits to 64 bits as defined in IDF. * updates function declaration
Done! |
Description of Change
esp_deep_sleep(uint64_t time_in_us)
takes a 64 bit argument, butvoid EspClass::deepSleep(uint32_t time_us)
allows only 32 bit.This fixes it, for a sleep time over 2,147 seconds (over 36 minutes).
https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/system/sleep_modes.html#_CPPv414esp_deep_sleep8uint64_t
Tests scenarios
Just CI.
Related links
Fixes #9074