Merge pull request #173 from opentrv/TRV1.5-rc5-lowbattery

Added error report for low battery
opentrv · Nov 14, 2017 · b890692 · b890692
2 parents cf38e65 + 2c606fb
commit b890692
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/content/OTRadioLink/utility/OTV0P2BASE_ErrorReport.h b/content/OTRadioLink/utility/OTV0P2BASE_ErrorReport.h
@@ -96,6 +96,9 @@ class ErrorReport final : public OTV0P2BASE::Actuator<int8_t>
 
             // Unspecified error.
             ERR_UNSPECIFIED = 1,
+
+            // Supply voltage is low (as defined by SupplyVoltageCentiVolts::BATTERY_LOW_cV).
+            ERR_BATTERY_LOW = 10,
             };
 
     private:

diff --git a/content/OTRadioLink/utility/OTV0P2BASE_PowerManagement.cpp b/content/OTRadioLink/utility/OTV0P2BASE_PowerManagement.cpp
@@ -38,6 +38,7 @@ Author(s) / Copyright (s): Damon Hart-Davis 2013--2016
 #include "OTV0P2BASE_ADC.h"
 #include "OTV0P2BASE_Entropy.h"
 #include "OTV0P2BASE_Sleep.h"
+#include "OTV0P2BASE_ErrorReport.h"
 
 namespace OTV0P2BASE
 {
@@ -315,6 +316,10 @@ uint16_t SupplyVoltageCentiVolts::read()
   isLow = isVeryLow || (result <= BATTERY_LOW_cV);
   static_assert(BATTERY_LOW_cV > BATTERY_VERY_LOW_cV, "thresholds should be such that 'not low' entails 'not very low'");
   static_assert(MAINS_MIN_cV > BATTERY_LOW_cV, "thresholds should be such that 'on mains' entails 'not low'");
+
+  // Set an error if battery is low.
+  ErrorReporter.set(ErrorReport::ERR_BATTERY_LOW);
+
 #if 0 && defined(DEBUG)
   DEBUG_SERIAL_PRINT_FLASHSTRING("Battery cV: ");
   DEBUG_SERIAL_PRINT(result);