From cf6d42cc516db669a7831b94ef0d4108d3020852 Mon Sep 17 00:00:00 2001
From: Claudio Micheli <claudio@auterion.com>
Date: Thu, 11 Mar 2021 02:17:33 +0100
Subject: [PATCH] Add battery faults flags for wrong firmware and battery
 models (#1577)

* common.xml: add new battery fault flags

Signed-off-by: Claudio Micheli <claudio@auterion.com>

* Change to BATTERY_FAULT_INCOMPATIBLE_CELLS_CONFIGURATION

Co-authored-by: Hamish Willee <hamishwillee@gmail.com>
---
 message_definitions/v1.0/common.xml | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/message_definitions/v1.0/common.xml b/message_definitions/v1.0/common.xml
index 58ddb04334..5df8d08a65 100644
--- a/message_definitions/v1.0/common.xml
+++ b/message_definitions/v1.0/common.xml
@@ -2932,6 +2932,12 @@
       <entry value="64" name="MAV_BATTERY_FAULT_INCOMPATIBLE_VOLTAGE">
         <description>Vehicle voltage is not compatible with this battery (batteries on same power rail should have similar voltage).</description>
       </entry>
+      <entry value="128" name="MAV_BATTERY_FAULT_INCOMPATIBLE_FIRMWARE">
+        <description>Battery firmware is not compatible with current autopilot firmware.</description>
+      </entry>
+      <entry value="256" name="BATTERY_FAULT_INCOMPATIBLE_CELLS_CONFIGURATION">
+        <description>Battery is not compatible due to cell configuration (e.g. 5s1p when vehicle requires 6s).</description>
+      </entry>
     </enum>
     <enum name="MAV_GENERATOR_STATUS_FLAG" bitmask="true">
       <description>Flags to report status/failure cases for a power generator (used in GENERATOR_STATUS). Note that FAULTS are conditions that cause the generator to fail. Warnings are conditions that require attention before the next use (they indicate the system is not operating properly).</description>