Index: firmware/App/Controllers/RinsePump.c =================================================================== diff -u -r8348015aa77e1ee2c4c20e9f8e2c45b59bcf3f17 -r0253afcc2b02def4d57a4a21a20a9b3a6fe65f22 --- firmware/App/Controllers/RinsePump.c (.../RinsePump.c) (revision 8348015aa77e1ee2c4c20e9f8e2c45b59bcf3f17) +++ firmware/App/Controllers/RinsePump.c (.../RinsePump.c) (revision 0253afcc2b02def4d57a4a21a20a9b3a6fe65f22) @@ -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 @@ -277,8 +277,13 @@ if ( TRUE == getTestConfigStatus( TEST_CONFIG_DD_FP_ENABLE_BETA_2_0_HW ) ) { - // Set PWM count zero to stop the logical rinse pump - setRinsePumpPwmCount( D79_RINSE_PUMP, RINSE_PUMP_OFF_COUNT ); + 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 { @@ -302,13 +307,13 @@ if ( TRUE == getTestConfigStatus( TEST_CONFIG_DD_FP_ENABLE_BETA_2_0_HW ) ) { - 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 ); } else { @@ -321,20 +326,24 @@ /*********************************************************************//** * @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: @@ -359,7 +368,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 ) ); Index: firmware/App/DDCommon.h =================================================================== diff -u -r4efb1705bf036fff357b41566028823a01d5d6d0 -r0253afcc2b02def4d57a4a21a20a9b3a6fe65f22 --- firmware/App/DDCommon.h (.../DDCommon.h) (revision 4efb1705bf036fff357b41566028823a01d5d6d0) +++ firmware/App/DDCommon.h (.../DDCommon.h) (revision 0253afcc2b02def4d57a4a21a20a9b3a6fe65f22) @@ -39,8 +39,8 @@ //Uncomment below if barometric pressure sensor reading available //#define __BARO_PRES_SENSOR__ 1 -//Uncomment below if required hardware available ( baro and cond sensor) for testing -//#define ENABLE_ALARM_1 +//Uncomment below to disable the communication to dialin for blood leak embedded mode +//#define __BLOOD_LEAK_EMB_MODE_RESPONSE_DISABLED__ 1 //Uncomment below once characterization/study completed //#define ENABLE_ALARM_2 @@ -53,10 +53,20 @@ //Uncomment below to disable revised heater model #define __REVISED_HEATER_MODEL__ 1 + // comment below to disable bicarb multi level sensor for time based chamber F fill #define CONDUCTIVE_LEVEL_SENSOR_ENABLED 1 +//Uncomment below to disable d30 heater alarm(as all units are not equipped with d30 temp sensor) +//#define __D30_TEMPERATURE_DISABLED__ +//Uncomment below to disable the DD too-many-bad-message-CRCs alarm. +//#define __BAD_MSG_CRC_ALARM_DISABLED__ 1 + +//Uncomment below to disable the conductivity sensor alarms +//#define __CONDUCTIVITY_SENSOR_ALARMS_DISABLED__ + + #include #include #endif