eric/Crosspoint
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Additional checks

Browse Source
This commit is contained in:
jpirnay
2026-04-06 09:56:27 +02:00
parent 213ebfc016
commit 633cc04f41
3 changed files with 30 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/hal/HalPowerManager.cpp
+10 -7
View File
@@ -61,9 +61,11 @@ void HalPowerManager::setPowerSaving(bool enabled) {
}
void HalPowerManager::startDeepSleep(HalGPIO& gpio, bool keepClockAlive) const {
LOG_DBG("PWR", "startDeepSleep: waiting for power button release (isPressed=%d, rawPin=%d, keepClock=%d)",
gpio.isPressed(HalGPIO::BTN_POWER), digitalRead(InputManager::POWER_BUTTON_PIN) == LOW, keepClockAlive);
gpio.waitForStablePowerRelease();
LOG_DBG("PWR", "startDeepSleep: isPressed=%d, rawPin=%d, keepClock=%d", gpio.isPressed(HalGPIO::BTN_POWER),
digitalRead(InputManager::POWER_BUTTON_PIN) == LOW, keepClockAlive);
// Perform all hardware preparation immediately (while the button may still be held)
// so the user gets instant visual feedback (display already off). Only block for
// button release at the very end, right before entering sleep.
// GPIO13 is connected to the battery latch MOSFET.
// When keepClockAlive is false (default): GPIO13 goes LOW, the MCU is
// completely powered off during sleep (including the LP timer / RTC memory).
@@ -82,16 +84,17 @@ void HalPowerManager::startDeepSleep(HalGPIO& gpio, bool keepClockAlive) const {
gpio_deep_sleep_hold_en();
gpio_hold_en(GPIO_SPIWP);
pinMode(InputManager::POWER_BUTTON_PIN, INPUT_PULLUP);
// Now wait for the power button to be fully released before arming the wakeup
// trigger and entering sleep — prevents immediate re-wake from a held button.
gpio.waitForStablePowerRelease();
// Arm the wakeup trigger *after* the button is released
// Note: when keepClockAlive is false, this is only useful for waking up on USB power. On battery, the MCU will be
// completely powered off, so the power button is hard-wired to briefly provide power to the MCU, waking it up
// regardless of the wakeup source configuration.
// When keepClockAlive is true, this is the actual wakeup mechanism since the MCU stays powered.
esp_deep_sleep_enable_gpio_wakeup(1ULL << InputManager::POWER_BUTTON_PIN, ESP_GPIO_WAKEUP_GPIO_LOW);
// Final check: is the raw pin still LOW (button still physically pressed)?
if (digitalRead(InputManager::POWER_BUTTON_PIN) == LOW) {
LOG_DBG("PWR", "startDeepSleep: WARNING raw pin still LOW after release wait — may wake immediately!");
}
LOG_DBG("PWR", "startDeepSleep: entering deep sleep now");
// Enter Deep Sleep
esp_deep_sleep_start();