Index: firmware/App/Modes/OperationModes.c
===================================================================
diff -u -r6818dafc0839d6e8585f5e6f35013f539ad5a0ed -r94a190522ce398399c7b93c59f788d7666ec0060
--- firmware/App/Modes/OperationModes.c	(.../OperationModes.c)	(revision 6818dafc0839d6e8585f5e6f35013f539ad5a0ed)
+++ firmware/App/Modes/OperationModes.c	(.../OperationModes.c)	(revision 94a190522ce398399c7b93c59f788d7666ec0060)
@@ -1,30 +1,31 @@
 /**************************************************************************
 *
-* Copyright (c) 2019-2020 Diality Inc. - All Rights Reserved.
+* Copyright (c) 2019-2021 Diality Inc. - All Rights Reserved.
 *
 * THIS CODE MAY NOT BE COPIED OR REPRODUCED IN ANY FORM, IN PART OR IN
 * WHOLE, WITHOUT THE EXPLICIT PERMISSION OF THE COPYRIGHT OWNER.
 *
-* @file OperationModes.c
+* @file    OperationModes.c
 *
-* @author (last) Quang Nguyen
-* @date (last) 24-Aug-2020
+* @author  (last)      Dara Navaei
+* @date    (last)      06-Nov-2021
 *
-* @author (original) Dara Navaei
-* @date (original) 05-Nov-2019
+* @author  (original)  Dara Navaei
+* @date    (original)  05-Nov-2019
 *
 ***************************************************************************/
 
 #include "gio.h"
 
+#include "MessageSupport.h"
 #include "ModeChemicalDisinfect.h"
 #include "ModeDrain.h"
 #include "ModeFault.h"
 #include "ModeFill.h"
 #include "ModeFlush.h"
+#include "ModeGenIdle.h"
 #include "ModeHeatDisinfect.h"
 #include "ModeInitPOST.h"
-#include "ModeRecirculate.h"
 #include "ModeService.h"
 #include "ModeSolo.h"
 #include "ModeStandby.h"
@@ -50,19 +51,20 @@
 /// DG operation mode data publish interval.
 static OVERRIDE_U32_T dgOpModePublishInterval = { BROADCAST_DG_OP_MODE_INTERVAL, BROADCAST_DG_OP_MODE_INTERVAL, 0, 0 };
 static U32 broadcastModeIntervalCtr = 11;                                           ///< Interval counter used to determine when to broadcast operation mode.  Initialize to 11 to stagger broadcast.
+static U32 priorSubMode = 0;                                                        ///< The prior submode state.
 
 /// This matrix determines legal transitions from one mode to another.
 static const DG_OP_MODE_T MODE_TRANSITION_TABLE[ NUM_OF_DG_MODES - 1 ][ NUM_OF_DG_MODES - 1 ] =
 {
-// from     to->           FAULT        SERVICE         INIT         STANBY       STBY-SOLO      RE-CIRC       FILL          DRAIN         FLUSH         HEAT DIS     CHEM DIS
+// from     to->           FAULT        SERVICE         INIT         STANBY       STBY-SOLO      GEN-IDLE       FILL          DRAIN         FLUSH         HEAT DIS     CHEM DIS
         /* FAUL     */{ DG_MODE_FAUL, DG_MODE_SERV, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG },
         /* SERV     */{ DG_MODE_FAUL, DG_MODE_SERV, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG },
         /* INIT     */{ DG_MODE_FAUL, DG_MODE_NLEG, DG_MODE_INIT, DG_MODE_STAN, DG_MODE_SOLO, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG },
-        /* STAN     */{ DG_MODE_FAUL, DG_MODE_SERV, DG_MODE_NLEG, DG_MODE_STAN, DG_MODE_SOLO, DG_MODE_CIRC, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_FLUS, DG_MODE_HEAT, DG_MODE_CHEM },
+        /* STAN     */{ DG_MODE_FAUL, DG_MODE_SERV, DG_MODE_NLEG, DG_MODE_STAN, DG_MODE_SOLO, DG_MODE_GENE, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_FLUS, DG_MODE_HEAT, DG_MODE_CHEM },
         /* SOLO     */{ DG_MODE_FAUL, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_STAN, DG_MODE_SOLO, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_FLUS, DG_MODE_HEAT, DG_MODE_CHEM /*DG_MODE_NLEG*/ }, // TODO for testing only
-        /* CIRC     */{ DG_MODE_FAUL, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_STAN, DG_MODE_NLEG, DG_MODE_CIRC, DG_MODE_FILL, DG_MODE_DRAI, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG },
-        /* FILL     */{ DG_MODE_FAUL, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_STAN, DG_MODE_NLEG, DG_MODE_CIRC, DG_MODE_FILL, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG },
-        /* DRAI     */{ DG_MODE_FAUL, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_STAN, DG_MODE_NLEG, DG_MODE_CIRC, DG_MODE_NLEG, DG_MODE_DRAI, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG },
+        /* GENE     */{ DG_MODE_FAUL, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_STAN, DG_MODE_NLEG, DG_MODE_GENE, DG_MODE_FILL, DG_MODE_DRAI, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG },
+        /* FILL     */{ DG_MODE_FAUL, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_STAN, DG_MODE_NLEG, DG_MODE_GENE, DG_MODE_FILL, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG },
+        /* DRAI     */{ DG_MODE_FAUL, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_STAN, DG_MODE_NLEG, DG_MODE_GENE, DG_MODE_NLEG, DG_MODE_DRAI, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG },
         /* FLUS     */{ DG_MODE_FAUL, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_STAN, DG_MODE_SOLO, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_FLUS, DG_MODE_NLEG, DG_MODE_NLEG },
         /* HEAT     */{ DG_MODE_FAUL, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_STAN, DG_MODE_SOLO, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_HEAT, DG_MODE_NLEG },
         /* CHEM     */{ DG_MODE_FAUL, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_STAN, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_NLEG, DG_MODE_CHEM }
@@ -96,6 +98,7 @@
     // start in init mode
     currentMode = DG_MODE_INIT;
     currentSubMode = 0;
+    priorSubMode = 0;
     transitionToNewOperationMode( DG_MODE_INIT );
 
     // call initializers for the individual modes
@@ -104,7 +107,7 @@
     initInitAndPOSTMode();
     initStandbyMode();
     initSoloMode();
-    initRecirculateMode();
+    initGenIdleMode();
     initFillMode();
     initDrainMode();
     initFlushMode();
@@ -123,6 +126,8 @@
 {
     DG_OP_MODE_T newMode;
 
+    priorSubMode = currentSubMode;
+
     // any new mode requests?
     newMode = arbitrateModeRequest();   // will return current mode if no pending requests
     newMode = MODE_TRANSITION_TABLE[ currentMode ][ newMode ];
@@ -166,8 +171,8 @@
             currentSubMode = execSoloMode();
             break;
 
-        case DG_MODE_CIRC:
-            currentSubMode = execRecirculateMode();
+        case DG_MODE_GENE:
+            currentSubMode = execGenIdleMode();
             break;
 
         case DG_MODE_FILL:
@@ -197,6 +202,14 @@
             break;
     }
 
+    // Send sub-mode change event when appropriate
+    if ( priorSubMode != currentSubMode )
+    {
+        SEND_EVENT_WITH_2_U32_DATA( DG_EVENT_SUB_MODE_CHANGE, priorSubMode, currentSubMode )
+    }
+
+    priorSubMode = currentSubMode;
+
     // publish op mode on interval
     broadcastOperationMode();
 }
@@ -296,46 +309,52 @@
  *************************************************************************/
 static void transitionToNewOperationMode( DG_OP_MODE_T newMode )
 {
+    SEND_EVENT_WITH_2_U32_DATA( DG_EVENT_OP_MODE_CHANGE, lastMode, newMode )
+
     // setup for new operating mode
     switch ( newMode )
     {
         case DG_MODE_FAUL:
-            transitionToFaultMode();
+            currentSubMode = transitionToFaultMode();
             break;
         case DG_MODE_SERV:
-            transitionToServiceMode();
+            currentSubMode = transitionToServiceMode();
             break;
         case DG_MODE_INIT:
-            transitionToInitAndPOSTMode();
+            currentSubMode = transitionToInitAndPOSTMode();
             break;
         case DG_MODE_STAN:
-            transitionToStandbyMode();
+            currentSubMode = transitionToStandbyMode();
             break;
         case DG_MODE_SOLO:
-            transitionToSoloMode();
+            currentSubMode = transitionToSoloMode();
             break;
-        case DG_MODE_CIRC:
-            transitionToRecirculateMode();
+        case DG_MODE_GENE:
+            currentSubMode = transitionToGenIdleMode();
             break;
         case DG_MODE_FILL:
-            transitionToFillMode();
+            currentSubMode = transitionToFillMode();
             break;
         case DG_MODE_DRAI:
-            transitionToDrainMode();
+            currentSubMode = transitionToDrainMode();
             break;
         case DG_MODE_FLUS:
-            transitionToFlushMode();
+            currentSubMode = transitionToFlushMode();
             break;
         case DG_MODE_HEAT:
-            transitionToHeatDisinfectMode();
+            currentSubMode = transitionToHeatDisinfectMode();
             break;
         case DG_MODE_CHEM:
-            transitionToChemicalDisinfectMode();
+            currentSubMode = transitionToChemicalDisinfectMode();
             break;
         default:
             SET_ALARM_WITH_2_U32_DATA( ALARM_ID_DG_SOFTWARE_FAULT, SW_FAULT_ID_OP_MODES_INVALID_MODE_TO_TRANSITION_TO, (U32)newMode )
             break;
     }
+
+    SEND_EVENT_WITH_2_U32_DATA( DG_EVENT_SUB_MODE_CHANGE, priorSubMode, currentSubMode )
+
+    priorSubMode = currentSubMode;
 }
 
 /*********************************************************************//**
@@ -350,8 +369,14 @@
 {
     if ( ++broadcastModeIntervalCtr >= getDGOpModePublishInterval() )
     {
+        OP_MODES_DATA_T data;
+
+        data.currentMode    = (U32)currentMode;
+        data.currentSubMode = currentSubMode;
+
+        broadcastData( MSG_ID_DG_OP_MODE, COMM_BUFFER_OUT_CAN_DG_BROADCAST, (U08*)&data, sizeof( OP_MODES_DATA_T ) );
+
         broadcastModeIntervalCtr = 0;
-        broadcastDGOperationMode( (U32)currentMode, currentSubMode );
     }
 }