Set up watchdog to restart device if no telemetry is published

kivancsikert · Dec 15, 2024 · cfcebc9 · cfcebc9
1 parent 74390ae
commit cfcebc9
Show file tree

Hide file tree

Showing 4 changed files with 17 additions and 2 deletions.
diff --git a/main/devices/Device.hpp b/main/devices/Device.hpp
@@ -503,6 +503,10 @@ class Device {
 
         Task::loop("telemetry", 8192, [this](Task& task) {
             publishTelemetry();
+
+            // Signal that we are still alive
+            // kernel.watchdog.restart();
+
             // TODO Configure these telemetry intervals
             // Publishing interval
             const auto interval = 1min;

diff --git a/main/kernel/Kernel.hpp b/main/kernel/Kernel.hpp
@@ -17,6 +17,7 @@
 #include <kernel/I2CManager.hpp>
 #include <kernel/PowerManager.hpp>
 #include <kernel/StateManager.hpp>
+#include <kernel/Watchdog.hpp>
 #include <kernel/drivers/LedDriver.hpp>
 #include <kernel/drivers/MdnsDriver.hpp>
 #include <kernel/drivers/RtcDriver.hpp>
@@ -303,6 +304,13 @@ class Kernel {
     TDeviceConfiguration& deviceConfig;
 
 public:
+    Watchdog watchdog { "watchdog", 10s, true, [](WatchdogState state) {
+                           if (state == WatchdogState::TimedOut) {
+                               LOGE("Watchdog timed out");
+                               esp_system_abort("Watchdog timed out");
+                           }
+                       } };
+
     PowerManager powerManager { deviceConfig.sleepWhenIdle.get() };
 
 private:

diff --git a/main/kernel/Watchdog.hpp b/main/kernel/Watchdog.hpp
@@ -18,10 +18,13 @@ typedef std::function<void(WatchdogState)> WatchdogCallback;
 
 class Watchdog {
 public:
-    Watchdog(const String& name, const ticks timeout, WatchdogCallback callback)
+    Watchdog(const String& name, const ticks timeout, bool startImmediately, WatchdogCallback callback)
         : name(name)
         , timeout(timeout)
         , callback(callback) {
+        if (startImmediately) {
+            restart();
+        }
     }
 
     void restart() {

diff --git a/main/peripherals/chicken_door/ChickenDoor.hpp b/main/peripherals/chicken_door/ChickenDoor.hpp
@@ -112,7 +112,7 @@ class ChickenDoorComponent
               SwitchMode::PullUp,
               [this](const Switch&) { updateState(); },
               [this](const Switch&, milliseconds) { updateState(); }))
-        , watchdog(name + ":watchdog", movementTimeout, [this](WatchdogState state) {
+        , watchdog(name + ":watchdog", movementTimeout, false, [this](WatchdogState state) {
             handleWatchdogEvent(state);
         })
         , publishTelemetry(publishTelemetry)