Index: firmware/App/Modes/FPModes/ModePreGenPermeateDefeatured.c
===================================================================
diff -u -r64acabc33ab795591768801f8b443f0da9ee2b98 -ra36dcbc3e2f94d418d8aa54588c836a3adf35e17
--- firmware/App/Modes/FPModes/ModePreGenPermeateDefeatured.c	(.../ModePreGenPermeateDefeatured.c)	(revision 64acabc33ab795591768801f8b443f0da9ee2b98)
+++ firmware/App/Modes/FPModes/ModePreGenPermeateDefeatured.c	(.../ModePreGenPermeateDefeatured.c)	(revision a36dcbc3e2f94d418d8aa54588c836a3adf35e17)
@@ -317,6 +317,24 @@
 
 /*********************************************************************//**
  * @brief
+ * The validatePreGenPDefeaturedChangeReq function validate whether requested
+ * sub mode change is legal or not.
+ * @details \b Inputs: none
+ * @details \b Outputs: none
+ * @return TRUE if request is legal, FALSE if not.
+ *************************************************************************/
+BOOL validatePreGenPDefeaturedChangeReq( U32 newSubMode )
+{
+    BOOL result = FALSE;
+
+    // TODO add validation logic
+    result = TRUE;
+
+    return result;
+}
+
+/*********************************************************************//**
+ * @brief
  * The signalNewPreGenPDefeaturedSubMode function requests transition to a new
  * defeatured PreGen Permeate sub mode.
  * @details \b Alarm: ALARM_ID_FP_SOFTWARE_FAULT if given new sub mode is invalid.
@@ -330,25 +348,21 @@
 
     if ( reqSubMode < NUM_OF_FP_PRE_GENP_DEF_MODE_STATES )
     {
+        preGenPermeateDefState = reqSubMode;
+        // handle the pre-requisite if any
         switch ( reqSubMode )
         {
             case FP_PRE_GENP_DEF_INLET_PRESSURE_CHECK:
-                preGenPermeateDefState = FP_PRE_GENP_DEF_INLET_PRESSURE_CHECK;
-                setModePreGenPDefTransition( reqSubMode );
                 break;
 
             case FP_PRE_GENP_DEF_FLUSH:
-                preGenPermeateDefState = FP_PRE_GENP_DEF_FLUSH;
-                setModePreGenPDefTransition( reqSubMode );
                 break;
 
             case FP_PRE_GENP_DEF_PAUSED:
-                preGenPermeateDefState = FP_PRE_GENP_DEF_PAUSED;
-                setModePreGenPDefTransition( reqSubMode );
-
             default:
                 break;
         }
+        setModePreGenPDefTransition( reqSubMode );
     }
 }