Index: firmware/App/Services/AlarmMgmt.c
===================================================================
diff -u -rebbb1f85550a1f9b8f946655f7b2b63f76fbf67d -rbb1f2d023593cbd1428c05e361745623a60ffb94
--- firmware/App/Services/AlarmMgmt.c	(.../AlarmMgmt.c)	(revision ebbb1f85550a1f9b8f946655f7b2b63f76fbf67d)
+++ firmware/App/Services/AlarmMgmt.c	(.../AlarmMgmt.c)	(revision bb1f2d023593cbd1428c05e361745623a60ffb94)
@@ -14,6 +14,8 @@
 * @date (original) 04-Feb-2020
 *
 ***************************************************************************/
+
+#define __ALARM_MGMT_C__
 
 #include "AlarmMgmt.h"
 #include "OperationModes.h"
@@ -28,12 +30,15 @@
  */
 
 // ********** private definitions **********
+
+// *** This declaration will cause a compiler error if ALARM_TABLE does not have same # of alarms as the Alarm_List enumeration.
+U08 alarmTableSizeAssertion[ ( ( sizeof( ALARM_TABLE ) / sizeof( ALARM_T ) ) == NUM_OF_ALARM_IDS ? 1 : -1 ) ];
 
-const ALARM_DATA_T blankAlarmData = { ALARM_DATA_TYPE_NONE, 0 };        ///< A blank alarm data record for alarms that do not include alarm data when triggered.
+const ALARM_DATA_T BLANK_ALARM_DATA = { ALARM_DATA_TYPE_NONE, 0 };      ///< A blank alarm data record for alarms that do not include alarm data when triggered.
 
 // ********** private data **********
 
-static OVERRIDE_U32_T alarmIsActive[ NUM_OF_ALARM_IDS ];                ///< Array of current state of each alarm
+static BOOL alarmIsActive[ NUM_OF_ALARM_IDS ];                          ///< Array of current state of each alarm
 static BOOL alarmConditionIsActive[ NUM_OF_ALARM_IDS ];                 ///< Array of flag indicates if an alarm condition is active
 
 // ********** private function prototypes **********
@@ -54,11 +59,7 @@
     // initialize alarm states and start time stamps
     for ( alrm = ALARM_ID_NO_ALARM; alrm < NUM_OF_ALARM_IDS; alrm++ )
     {
-        alarmIsActive[ alrm ].data = FALSE;
-        alarmIsActive[ alrm ].ovData = FALSE;
-        alarmIsActive[ alrm ].ovInitData = TRUE;
-        alarmIsActive[ alrm ].override = OVERRIDE_RESET;
-
+        alarmIsActive[ alrm ] = FALSE;
         alarmConditionIsActive[ alrm ] = FALSE;
     }
 }
@@ -89,11 +90,17 @@
     if ( ( alarm > ALARM_ID_NO_ALARM ) && ( alarm < NUM_OF_ALARM_IDS ) )
     {
         // no need to do anything if alarm is already active
-        if ( FALSE == isAlarmActive( alarm ) )
+        if ( FALSE == alarmIsActive[ alarm ] )
         {
             // activate alarm
-            alarmIsActive[ alarm ].data = TRUE;
-            alarmConditionIsActive[ alarm ] = TRUE;
+            alarmIsActive[ alarm ] = TRUE;
+            alarmConditionIsActive[ alarm ] = TRUE;
+
+            // If alarm is a DG fault, request transition to fault mode
+            if ( TRUE == ALARM_TABLE[ alarm ].alarmIsDGFault )
+            {
+                requestNewOperationMode( DG_MODE_FAUL );
+            }
         }
     }
     else
@@ -112,25 +119,8 @@
  * @return none
  *************************************************************************/
 void activateAlarmNoData( ALARM_ID_T alarm )
-{
-    // broadcast alarm and data if alarm not already active
-    if ( FALSE == alarmIsActive[ alarm ].data )
-    {
-        broadcastAlarmTriggered( alarm, blankAlarmData, blankAlarmData );
-#ifdef DEBUG_ENABLED
-#ifdef ALARMS_DEBUG
-        {
-            // TODO - temporary debug code - remove later
-            char debugStr[ 256 ];
-            sprintf( debugStr, "ALARM trig:%5d \n", alarm );
-            sendDebugData( (U08*)debugStr, strlen(debugStr) );
-            sendDebugDataToUI( (U08*)debugStr );
-        }
-#endif
-#endif
-    }
-
-    activateAlarm( alarm );
+{
+    activateAlarm2Data( alarm, BLANK_ALARM_DATA, BLANK_ALARM_DATA );
 }
 
 /*********************************************************************//**
@@ -145,25 +135,8 @@
  * @return none
  *************************************************************************/
 void activateAlarm1Data( ALARM_ID_T alarm, ALARM_DATA_T alarmData )
-{
-    // broadcast alarm and data if alarm not already active
-    if ( FALSE == alarmIsActive[ alarm ].data )
-    {
-        broadcastAlarmTriggered( alarm, alarmData, blankAlarmData );
-#ifdef DEBUG_ENABLED
-#ifdef ALARMS_DEBUG
-        {
-            // TODO - temporary debug code - remove later
-            char debugStr[ 256 ];
-            sprintf( debugStr, "ALARM trig:%5d %8X \n", alarm, alarmData.data.uInt.data );
-            sendDebugData( (U08*)debugStr, strlen(debugStr) );
-            sendDebugDataToUI( (U08*)debugStr );
-        }
-#endif
-#endif
-    }
-
-    activateAlarm( alarm );
+{
+    activateAlarm2Data( alarm, alarmData, BLANK_ALARM_DATA );
 }
 
 /*********************************************************************//**
@@ -181,22 +154,10 @@
 void activateAlarm2Data( ALARM_ID_T alarm, ALARM_DATA_T alarmData1, ALARM_DATA_T alarmData2 )
 {
     // broadcast alarm and data if alarm not already active
-    if ( FALSE == alarmIsActive[ alarm ].data )
+    if ( FALSE == alarmIsActive[ alarm ] )
     {
         broadcastAlarmTriggered( alarm, alarmData1, alarmData2 );
-#ifdef DEBUG_ENABLED
-#ifdef ALARMS_DEBUG
-        {
-            // TODO - temporary debug code - remove later
-            char debugStr[ 256 ];
-            sprintf( debugStr, "ALARM trig:%5d %8X %8X \n", alarm, alarmData1.data.uInt.data, alarmData2.data.uInt.data );
-            sendDebugData( (U08*)debugStr, strlen(debugStr) );
-            sendDebugDataToUI( (U08*)debugStr );
-        }
-#endif
-#endif
     }
-
     activateAlarm( alarm );
 }
 
@@ -215,22 +176,11 @@
     if ( ( alarm > ALARM_ID_NO_ALARM ) && ( alarm < NUM_OF_ALARM_IDS ) )
     {
         // clear alarm and broadcast alarm clear if not already cleared
-        if ( TRUE == alarmIsActive[ alarm ].data )
+        if ( TRUE == alarmIsActive[ alarm ] )
         {
             broadcastAlarmCleared( alarm );
-            alarmIsActive[ alarm ].data = FALSE;
+            alarmIsActive[ alarm ] = FALSE;
             clearAlarmCondition( alarm );
-#ifdef DEBUG_ENABLED
-#ifdef ALARMS_DEBUG
-            {
-                // TODO - temporary debug code - remove later
-                char debugStr[ 256 ];
-                sprintf( debugStr, "ALARM cleared:%5d \n", alarm );
-                sendDebugData( (U08*)debugStr, strlen(debugStr) );
-                sendDebugDataToUI( (U08*)debugStr );
-            }
-#endif
-#endif
         }
     }
     else
@@ -276,24 +226,7 @@
  *************************************************************************/
 BOOL isAlarmActive( ALARM_ID_T alarm )
 {
-    BOOL result = TRUE;
-    if ( alarm < NUM_OF_ALARM_IDS )
-    {
-        if ( OVERRIDE_KEY == alarmIsActive[ alarm ].override )
-        {
-            result = (BOOL)alarmIsActive[ alarm ].ovData;
-        }
-        else
-        {
-            result = (BOOL)alarmIsActive[ alarm ].data;
-        }
-    }
-    else
-    {
-        activateAlarmNoData( ALARM_ID_DG_SOFTWARE_FAULT );
-    }
-
-    return result;
+    return alarmIsActive[ alarm ];
 }
 
 /*********************************************************************//**