From 2a2a36b267acf9ddf108718544fcbba3833e90dc Mon Sep 17 00:00:00 2001
From: Ollo <Ollo@host>
Date: Thu, 6 Apr 2023 17:56:17 +0200
Subject: [PATCH] sleep at least several minutes, so the solar panel can charge
 the battery

---
 esp32/include/ControllerConfiguration.h |  1 +
 esp32/src/main.cpp                      | 18 ++++++++++++++++--
 2 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/esp32/include/ControllerConfiguration.h b/esp32/include/ControllerConfiguration.h
index f6e300f..6c68bda 100644
--- a/esp32/include/ControllerConfiguration.h
+++ b/esp32/include/ControllerConfiguration.h
@@ -108,6 +108,7 @@
 #define VOLT_MAX_BATT               4.2f
 #define VOLT_MIN_BATT               3.0f    /**< Minimum battery voltage for normal operation */
 #define LOWVOLT_SLEEP_FACTOR        3       /**< Factor for nightsleep delay, if the battery drops below minimum (@see VOLT_MIN_BATT) */
+#define LOWVOLT_SLEEP_MINIMUM       1800    /**< At low voltage sleep at least for 30 minutes */
 
 #define MAX_CONFIG_SETTING_ITEMS 100 /**< Parameter, that can be configured in Homie */
 #define MAX_JSON_CONFIG_FILE_SIZE_CUSTOM 2500
diff --git a/esp32/src/main.cpp b/esp32/src/main.cpp
index db64b10..5a37f30 100644
--- a/esp32/src/main.cpp
+++ b/esp32/src/main.cpp
@@ -192,8 +192,17 @@ void espDeepSleep(bool afterPump = false)
   {
     if (mBatteryVoltage < VOLT_MIN_BATT) 
     {
-      log(LOG_LEVEL_INFO, String(String(mBatteryVoltage) + "V! Almost empty -> deepSleepNight times " + String(LOWVOLT_SLEEP_FACTOR)), LOG_SLEEP_LOWVOLTAGE);
-      secondsToSleep = (deepSleepNightTime.get() * LOWVOLT_SLEEP_FACTOR);
+      log(LOG_LEVEL_INFO, String(String(mBatteryVoltage) + "V! Almost empty -> deepSleepNight " + String(LOWVOLT_SLEEP_FACTOR)) + " times", LOG_SLEEP_LOWVOLTAGE);
+      /* use the largest sleeping duration */
+      if (deepSleepNightTime.get() > deepSleepTime.get()) {
+        secondsToSleep = (deepSleepNightTime.get() * LOWVOLT_SLEEP_FACTOR);
+      } else {
+        secondsToSleep = (deepSleepTime.get() * LOWVOLT_SLEEP_FACTOR);
+      }
+      /* sleep at least several minutes, so the solar panel can charge the battery */
+      if (secondsToSleep < LOWVOLT_SLEEP_MINIMUM) {
+        secondsToSleep = LOWVOLT_SLEEP_MINIMUM;
+      }
     }
     else if (mSolarVoltage < SOLAR_CHARGE_MIN_VOLTAGE)
     {
@@ -208,6 +217,10 @@ void espDeepSleep(bool afterPump = false)
   }
 
   finsihedCycleSucessfully();
+  /* sleep always at least one second */
+  if (secondsToSleep < 0) {
+    secondsToSleep = 1;
+  }
   esp_sleep_enable_timer_wakeup((secondsToSleep * 1000U * 1000U));
   if (aliveWasRead())
   {
@@ -363,6 +376,7 @@ void readPowerSwitchedSensors()
     tankSensor.setVcselPulsePeriod(VL53L0X::VcselPeriodFinalRange, 14);
     tankSensor.setMeasurementTimingBudget(200000);
     for (int readCnt = 0; readCnt < WATERSENSOR_CYCLE; readCnt++)
+    
     {
       if (!tankSensor.timeoutOccurred())
       {