BMS: add bench-mode macro to disable temperature enforcement

FEB_BMS_DISABLE_TEMP_CHECKS (FEB_Const.h, default 0) compiles out all
cell-temperature enforcement for bench bring-up of ADBMS modules with
unpopulated thermistors: the over/under-temp fault latch in
validate_temps(), the charging temp soft-stop/hard-fault block in
FEB_CAN_Charging_Status(), and both balance thermal gates (which all-NaN
readings would otherwise hold closed). Readings and diagnostics stay
visible; suppressed violations still print a one-shot warning. While
enabled, boot logs a prominent warning, the console banner flags bench
mode, and BMS|config reports the override state. Never enable on a real
pack.

Author: ZacharyBer

BMS/Core/User/Inc/FEB_Const.h

@@ -143,6 +143,19 @@
 #define FEB_IVT_PACK_VOLTAGE_CHANNEL 2
 #endif
 
+// ********************************** Temperature Enforcement Override (BENCH ONLY)
+
+// Set to 1 to disable ALL cell-temperature enforcement: the over/under-temp
+// fault latch (validate_temps), the charging temp soft-stop/hard-fault block
+// (FEB_CAN_Charging_Status), and the balance thermal gates. For bench bring-up
+// of ADBMS modules with unpopulated thermistors: floating inputs can latch
+// spurious temp faults, and all-NaN readings block balancing entirely.
+// Readings and diagnostics remain visible; only enforcement is suppressed.
+// Logs a warning at boot while enabled. NEVER enable with a real pack.
+#ifndef FEB_BMS_DISABLE_TEMP_CHECKS
+#define FEB_BMS_DISABLE_TEMP_CHECKS 0
+#endif
+
 // ********************************** Accumulator Structure **********************
 
 typedef struct

BMS/Core/User/Src/FEB_ADBMS6830B.c

@@ -478,11 +478,19 @@ static void validate_temps()
           FEB_ACC.banks[bank].temp_violations[sensor]++;
           if (FEB_ACC.banks[bank].temp_violations[sensor] >= FEB_TEMP_ERROR_THRESH)
           {
+#if FEB_BMS_DISABLE_TEMP_CHECKS
+            /* Bench mode: report the violation but do not latch the fault.
+             * Fires once per sensor thanks to the saturating counter above. */
+            printf("[ADBMS] WARNING: temp violation IGNORED (FEB_BMS_DISABLE_TEMP_CHECKS) - "
+                   "Bank %d Sensor %d: %.1fC (limits: %.1f-%.1fC)\r\n",
+                   bank, sensor, temp / 10.0f, tMin / 10.0f, tMax / 10.0f);
+#else
             printf("[ADBMS] FAULT: Cell temperature out of range - Bank %d Sensor %d: %.1fC (limits: %.1f-%.1fC)\r\n",
                    bank, sensor, temp / 10.0f, tMin / 10.0f, tMax / 10.0f);
             FEB_ADBMS_Update_Error_Type(ERROR_TYPE_TEMP_VIOLATION);
             /* Latch for the SM task; evaluate_faults() routes per state group. */
             adbms_fault_flags |= ADBMS_FAULT_FLAG_TEMP;
+#endif
           }
         }
       }
@@ -799,6 +807,7 @@ void FEB_Cell_Balance_Start()
 // Caller must hold ADBMSMutexHandle.
 void FEB_Cell_Balance_Process()
 {
+#if !FEB_BMS_DISABLE_TEMP_CHECKS
   // Thermal safety gate. NaN fails the comparison → stops balancing when
   // telemetry is unavailable. Direct field read; caller already holds the mutex.
   const float gate_max_temp_dC = FEB_ACC.pack_max_temp * 10.0f;
@@ -807,6 +816,7 @@ void FEB_Cell_Balance_Process()
     LOG_W(TAG_BALANCE, "Temp limit: pack max=%.1fC, skipping balance cycle", gate_max_temp_dC / 10.0f);
     return;
   }
+#endif
 
   determineMinV();
 
@@ -876,6 +886,7 @@ void FEB_Cell_Balance_Process()
 
 bool FEB_Cell_Balancing_Status(void)
 {
+#if !FEB_BMS_DISABLE_TEMP_CHECKS
   // The per-cell loop used to index temp_sensor_readings by cell index, which
   // only covered FEB_NUM_CELLS_PER_BANK (14) of the FEB_NUM_TEMP_SENSORS (42)
   // sensors. Use the pack-wide max computed by compute_pack_temp_stats() so
@@ -888,6 +899,7 @@ bool FEB_Cell_Balancing_Status(void)
     LOG_W(TAG_BALANCE, "Temp limit: pack max=%.1fC, stopping", max_temp_dC / 10.0f);
     return false;
   }
+#endif
 
   float min_v = FLT_MAX;
   float max_v = -FLT_MAX;

BMS/Core/User/Src/FEB_CAN_Charger.c

@@ -137,6 +137,7 @@ int8_t FEB_CAN_Charging_Status(void)
     return 1;
   }
 
+#if !FEB_BMS_DISABLE_TEMP_CHECKS
   /* Highest temperature: soft limit -> stop charging, hard limit -> fault.
    * NaN (no temp scan yet) fails both comparisons -> treated as OK; cell-data
    * presence is already gated by pack_v above and ADBMS staleness is a fault. */
@@ -149,6 +150,7 @@ int8_t FEB_CAN_Charging_Status(void)
     }
     return 1;
   }
+#endif
 
   return 0;
 }

BMS/Core/User/Src/FEB_Commands.c

@@ -602,6 +602,11 @@ static void subcmd_config(int argc, char *argv[])
   FEB_Console_Printf("Min Cell V:       %.3f V\r\n", FEB_CELL_MIN_VOLTAGE_MV / 1000.0f);
   FEB_Console_Printf("Max Cell Temp:    %.1f C\r\n", FEB_CELL_MAX_TEMP_DC / 10.0f);
   FEB_Console_Printf("Min Cell Temp:    %.1f C\r\n", FEB_CELL_MIN_TEMP_DC / 10.0f);
+#if FEB_BMS_DISABLE_TEMP_CHECKS
+  FEB_Console_Printf("Temp Checks:      DISABLED (BENCH MODE)\r\n");
+#else
+  FEB_Console_Printf("Temp Checks:      enabled\r\n");
+#endif
 }
 
 /* ============================================================================

BMS/Core/User/Src/FEB_Main.c

@@ -17,6 +17,7 @@
 #include "FEB_CAN_PingPong.h"
 #include "FEB_CAN_Charger.h"
 #include "FEB_SM.h"
+#include "FEB_Const.h"
 #include "cmsis_os2.h"
 #include "FreeRTOS.h"
 #include "task.h"
@@ -104,10 +105,19 @@ void FEB_Init(void)
   /* Log after all subsystems initialized */
   LOG_I(TAG_MAIN, "BMS initialization complete");
 
+#if FEB_BMS_DISABLE_TEMP_CHECKS
+  LOG_W(TAG_MAIN, "TEMPERATURE ENFORCEMENT DISABLED (FEB_BMS_DISABLE_TEMP_CHECKS=1)");
+  LOG_W(TAG_MAIN, "Bench mode: temp faults, charging temp limits, and balance");
+  LOG_W(TAG_MAIN, "thermal gates are BYPASSED. Do NOT run a real pack.");
+#endif
+
   /* Startup banner */
   FEB_Console_Printf("\r\n");
   FEB_Console_Printf("========================================\r\n");
   FEB_Console_Printf("        BMS Console Ready\r\n");
+#if FEB_BMS_DISABLE_TEMP_CHECKS
+  FEB_Console_Printf("  !! TEMP CHECKS DISABLED (BENCH) !!\r\n");
+#endif
   FEB_Console_Printf("========================================\r\n");
   FEB_Console_Printf("Use | as delimiter: BMS|status\r\n");
   FEB_Console_Printf("Type 'help' for available commands\r\n");

BMS/VERSION

@@ -1 +1 @@
-1.7.8
+1.7.9

VERSION

@@ -1 +1 @@
-1.7.23
+1.7.24