Index: firmware/App/Controllers/Valves.c
===================================================================
diff -u -rb50acaa8dda74292fb41110440eaaa40f2959fb7 -r89c8709e3b27648926fbb20f25c9a67cbeb99adc
--- firmware/App/Controllers/Valves.c	(.../Valves.c)	(revision b50acaa8dda74292fb41110440eaaa40f2959fb7)
+++ firmware/App/Controllers/Valves.c	(.../Valves.c)	(revision 89c8709e3b27648926fbb20f25c9a67cbeb99adc)
@@ -1,14 +1,14 @@
 /**************************************************************************
 *
-* Copyright (c) 2024-2025 Diality Inc. - All Rights Reserved.
+* Copyright (c) 2024-2026 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    Valves.c
 *
-* @author  (last)      “Raghu
-* @date    (last)      16-Oct-2025
+* @author  (last)      “rkallala”
+* @date    (last)      28-Jan-2026
 *
 * @author  (original)  Vinayakam Mani
 * @date    (original)  26-Aug-2024
@@ -35,6 +35,10 @@
 #define ENERGIZED                                       1                                                           ///< 1 for energized valve.
 #define ALL_VALVES_DEENERGIZED                          0x0000                                                      ///< 0 in U16 bit field for all valves.
 
+#define VALVE_PWM_PERIOD                                50                                                          ///< Valve PWM Period in 0.1 us res ( 20kHz )
+#define VALVE_PWM_LOW                                   ( VALVE_PWM_PERIOD * 0.2 )                                  ///< Valve low PWM percentage
+#define VALVE_PWM_PULL_IN                               100                                                         ///< Valve 'strike' period at 100% PWM in ms
+
 #define MAX_VALVE_STATE_MISMATCH_TIMER_COUNT            (100 / TASK_PRIORITY_INTERVAL )                             ///< Maximum time commanded valves state can fail to match read back valve states in a row.
 #define VALVES_STATE_PUB_INTERVAL                       ( MS_PER_SECOND / TASK_PRIORITY_INTERVAL )                  ///< Interval ( ms / task time) at which valves states are published on CAN bus.
 #define DATA_PUBLISH_COUNTER_START_COUNT                50                                                          ///< Data publish counter start count.
@@ -71,6 +75,7 @@
 static BOOL checkValveStateName( DD_VALVES_T valveID, VALVE_STATE_NAMES_T valveStateName );
 static U32 convertValveStateNameToValveState( VALVE_STATE_NAMES_T valveStateName );
 static U32 getValveState( U32 valveID );
+static void setFPGAPWMConfigurations( void );
 
 /*********************************************************************//**
  * @brief
@@ -118,16 +123,77 @@
     valvesStatesPublicationTimerCounter     = DATA_PUBLISH_COUNTER_START_COUNT;  // reset valves states publication timer
     fpValveStatesPublicationTimerCounter    = DATA_PUBLISH_COUNTER_START_COUNT;  // reset valves states publication timer
     readCommandedValveStates();
+
+    // TODO set FPGA PWM after finalizing the decision on valve strike and hold behavior
+    // initially set valve PWM configuration and enable
+//    setFPGAPWMConfigurations();
+
     // initially set all valves to de-energized state via FPGA
     setFPGADDValveStates( commandedValvesStates );
     setFPGABCValveStates( commandedBCValveStates );
     setFPGASpareValveStates( commandedSpareValveStates );
-    setFPGAIOValveStates( commandedIOValveStates );
     setFPGAFPValveStates( commandedFPValveStates );
+    setFPGAIOValveStates( commandedIOValveStates );
 }
 
 /*********************************************************************//**
  * @brief
+ * The setFPGAPWMConfigurations function sets the FPGA for Valve PWM control.
+ * @details \b Inputs: none
+ * @details \b Outputs: Valves PWM timing set and enabled.
+ * @return none
+ *************************************************************************/
+static void setFPGAPWMConfigurations( void )
+{
+    // Balancing Chamber valves are not PWM configured.
+    U32 i;
+    U16 valveDDBits = 0;
+    U08 valveSPBits = 0;
+    U08 valveIOBits = 0;
+    U08 valveFPBits = 0;
+
+    // set timing values
+    setFPGAValveDDPWMPullIn( VALVE_PWM_PULL_IN );
+    setFPGAValveDDPWMLow( VALVE_PWM_LOW );
+    setFPGAValveDDPWMPeriod( VALVE_PWM_PERIOD );
+
+    setFPGAValveSPPWMPullIn( VALVE_PWM_PULL_IN );
+    setFPGAValveSPPWMLow( VALVE_PWM_LOW );
+    setFPGAValveSPPWMPeriod( VALVE_PWM_PERIOD );
+
+    setFPGAValveIOFPPWMPullIn( VALVE_PWM_PULL_IN );
+    setFPGAValveIOFPPWMLow( VALVE_PWM_LOW );
+    setFPGAValveIOFPPWMPeriod( VALVE_PWM_PERIOD );
+
+    for ( i = FIRST_HYD_VALVE; i <= LAST_HYD_VALVE; i++ )
+    {
+        valveDDBits |= 0x01 << i;
+    }
+
+    for ( i = FIRST_SP_VALVE; i <= LAST_SP_VALVE; i++ )
+    {
+        valveSPBits |= 0x01 << ( i - FIRST_SP_VALVE );
+    }
+
+    for ( i = FIRST_IO_VALVE; i <= LAST_IO_VALVE; i++ )
+    {
+        valveIOBits |= 0x01 << ( i - FIRST_IO_VALVE );
+    }
+
+    for ( i = FIRST_FP_VALVE; i <= LAST_FP_VALVE; i++ )
+    {
+        valveFPBits |= 0x01 << ( i - FIRST_FP_VALVE );
+    }
+
+    // set enable
+    setFPGAValveDDPWMEnable( valveDDBits );
+    setFPGAValveSPPWMEnable( valveSPBits );
+    setFPGAValveIOPWMEnable( valveIOBits );
+    setFPGAValveFPPWMEnable( valveFPBits );
+}
+
+/*********************************************************************//**
+ * @brief
  * The execValves function executes the valves driver.
  * @details \b Inputs: valvesStates, valveStateMismatchCounter
  * pendingValveStateChangeCountDowns, commandedValvesStates,
@@ -141,11 +207,11 @@
 void execValves( void )
 {
     U32 i;
-    U16 readValvesStates    = getFPGADDValveStates();
-    U08 readValveBCStates   = getFPGAValveBCStates();
-    U08 readValveSpStates   = getFPGAValveSpStates();
-    U08 readValveIOStates   = getFPGAIOValveStates();
-    U08 readValveFPStates   = getFPGAFPValveStates();
+    U16 readValvesStates  = getFPGADDValveStates();
+    U08 readValveBCStates = getFPGAValveBCStates();
+    U08 readValveSpStates = getFPGAValveSpStates();
+    U08 readValveIOStates = getFPGAIOValveStates();
+    U08 readValveFPStates = getFPGAFPValveStates();
 
     // Verify read back FPGA valve states match last commanded valve states
     if ( ( readValvesStates != commandedValvesStates ) ||
@@ -184,8 +250,8 @@
     setFPGADDValveStates( commandedValvesStates );
     setFPGABCValveStates( commandedBCValveStates );
     setFPGASpareValveStates( commandedSpareValveStates );
-    setFPGAIOValveStates( commandedIOValveStates );
     setFPGAFPValveStates( commandedFPValveStates );
+    setFPGAIOValveStates( commandedIOValveStates );
 
     // Publish valve states on interval
     publishValvesStates();
@@ -385,8 +451,8 @@
                  ( D40_VALV == valveID ) || ( D47_VALV == valveID ) || ( D3_VALV == valveID )  || ( M4_VALV == valveID )  || ( D23_VALV == valveID )  ||
                  ( D19_VALV == valveID ) || ( D25_VALV == valveID ) || ( D21_VALV == valveID ) || ( D24_VALV == valveID ) || ( D20_VALV == valveID )  ||
                  ( D26_VALV == valveID ) || ( D22_VALV == valveID ) || ( D80_VALV == valveID ) || ( D81_VALV == valveID ) || ( D79_PMP_VALV == valveID )   ||
-                 ( D85_VALV == valveID ) || ( P11_VALV == valveID ) || ( P33_VALV == valveID ) || ( P34_VALV == valveID ) || ( P37_VALV == valveID ) ||
-                 ( P6_VALV == valveID )  || ( M12_VALV == valveID ) || ( P39_VALV == valveID ) )
+                 ( P11_VALV == valveID ) || ( P33_VALV == valveID ) || ( P34_VALV == valveID ) || ( P37_VALV == valveID ) || ( P6_VALV == valveID ) ||
+                 ( M12_VALV == valveID ) || ( P39_VALV == valveID ) || ( D83_VALV == valveID ) || ( D91_VALV == valveID ) || ( D100_VALV == valveID ) || ( D85_VALV == valveID )  )
             {
                 result = TRUE;
             }