Index: firmware/App/Controllers/RinsePump.c =================================================================== diff -u -rf37bfe2eaf910104d6edfcf46a9771871fbfbc69 -r3a6adcf5e8b360efe3b38bbe55fc8f22f1c69ab8 --- firmware/App/Controllers/RinsePump.c (.../RinsePump.c) (revision f37bfe2eaf910104d6edfcf46a9771871fbfbc69) +++ firmware/App/Controllers/RinsePump.c (.../RinsePump.c) (revision 3a6adcf5e8b360efe3b38bbe55fc8f22f1c69ab8) @@ -7,8 +7,8 @@ * * @file RinsePump.c * -* @author (last) Vinayakam Mani -* @date (last) 14-Apr-2026 +* @author (last) Raghu Kallala +* @date (last) 19-May-2026 * * @author (original) Vinayakam Mani * @date (original) 02-Oct-2025 @@ -66,7 +66,7 @@ static RINSE_PUMP_STATE_T handleRinsePumpOnState ( void ); static void monitorRinsePumpSpeed( void ); static void publishRinsePumpData( void ); -static void setRinsePumpPwmCount( RINSE_PUMP_ID_T pumpId, U32 pwmCount ); +static void setRinsePumpPwm( RINSE_PUMP_ID_T pumpId, F32 pwmPercent ); /*********************************************************************//** * @brief @@ -276,8 +276,13 @@ if ( TRUE == getTestConfigStatus( TEST_CONFIG_DD_FP_ENABLE_BETA_1_9_HW ) ) { - // Current Beta 1.9 system uses on/off bit - setValveState( D88_79_VALV, VALVE_STATE_CLOSED ); + F32 pwmPercent; + + // Set PWM to zero to stop the logical rinse pump + rinsePumpPwmPercentage.data = 0.0F; + // PWM should be zero. Ignore the override value for rinsePumpPwmPercentage if any. + pwmPercent = rinsePumpPwmPercentage.data; + setRinsePumpPwm( D79_RINSE_PUMP, pwmPercent ); } else { @@ -301,39 +306,38 @@ if ( TRUE == getTestConfigStatus( TEST_CONFIG_DD_FP_ENABLE_BETA_1_9_HW ) ) { - // Current Beat 1.9 system uses on/off bit - setValveState( D88_79_VALV, VALVE_STATE_OPEN ); - } - else - { - F32 pwmPercent = getF32OverrideValue( &rinsePumpPwmPercentage ); // TODO should this conversion happen in setPWMCount function? - U32 pwmInCount = (U32)( ( pwmPercent / RINSE_PUMP_MAX_PWM_PERCENT ) * RINSE_PUMP_PWM_IN_COUNT_MAX ); + F32 pwmPercent; - rinsePumpTargetSpeedRPM = (U32)( ( (F32)pwmInCount / RINSE_PUMP_PWM_IN_COUNT_MAX ) * (F32)RINSE_PUMP_MAX_RPM_ESTIMATE ); + rinsePumpPwmPercentage.data = RINSE_PUMP_DEFAULT_PWM_PERCENT; + pwmPercent = getF32OverrideValue( &rinsePumpPwmPercentage ); // Turn on logical rinse pump with given PWM value - setRinsePumpPwmCount( D79_RINSE_PUMP, (U32)pwmInCount ); + setRinsePumpPwm( D79_RINSE_PUMP, pwmPercent ); } return state; } /*********************************************************************//** * @brief - * The setRinsePumpPwmCount function maps a logical rinse pump ID to the - * underlying FPGA control and applies the requested PWM count. - * @details \b Inputs: pumpId, pwmCount + * The setRinsePumpPwm function maps a logical rinse pump ID to the + * underlying FPGA control and applies the requested PWM. + * @details \b Inputs: pumpId, pwmPercent * @details \b Outputs: FPGA rinse pump control register * @param pumpId Logical rinse pump identifier * @param pwmCount PWM magnitude (0..RINSE_PUMP_PWM_IN_COUNT_MAX) * @return none *************************************************************************/ -static void setRinsePumpPwmCount( RINSE_PUMP_ID_T pumpId, U32 pwmCount ) +static void setRinsePumpPwm( RINSE_PUMP_ID_T pumpId, F32 pwmPercent ) { + U08 pwmCount = (U08)( ( pwmPercent / RINSE_PUMP_MAX_PWM_PERCENT ) * RINSE_PUMP_PWM_IN_COUNT_MAX ); + + rinsePumpTargetSpeedRPM = (U32)( ( (F32)pwmCount / RINSE_PUMP_PWM_IN_COUNT_MAX ) * (F32)RINSE_PUMP_MAX_RPM_ESTIMATE ); + switch ( pumpId ) { case D79_RINSE_PUMP: - setFPGAD79RinsePumpPWMControl( (U08)pwmCount ); + setFPGAD79RinsePumpPWMControl( pwmCount ); break; default: @@ -358,7 +362,7 @@ RINSE_PUMP_PAYLOAD_T data; data.d79State = getRinsePumpState(); - data.d79PumpPWM = getF32OverrideValue( &rinsePumpPwmPercentage ); + data.d79PumpPWM = ( getRinsePumpState() == RINSE_PUMP_STATE_OFF ? 0.0F : getF32OverrideValue( &rinsePumpPwmPercentage ) ); data.d79PumpRPM = rinsePumpMeasuredSpeed; broadcastData( MSG_ID_DD_RINSE_PUMP_DATA, COMM_BUFFER_OUT_CAN_DD_BROADCAST, (U08*)&data, sizeof( RINSE_PUMP_PAYLOAD_T ) );