Add to inputs : heparinSetupResumeRequested

Add @alarm

Add to inputs : interruptedPreTxHeparinSetupState

Does default mean invalid state (s/w fault) or does it mean it's a state that we don't need to do anything for? If latter, add a comment here saying that.

Use if else or ternary operator to set BOOL to TRUE or FALSE explicitly.

Add a TODO comment to adjust this when new syringe pump assembly is ready.

· /firmware/App/Controllers/AirPump.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Controllers/AirPump.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/Messaging.c	(-1, +1)	| History	| Source	| Diff

]]> Fri, 07 Aug 2026 17:07:37 -0700 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-5214-1 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-5214-1 Vinayakam Mani and Vendor - TEL - Arpita Srivastava finished reviewing LEAHI-TD-FIRMWARE-LDT-5214-1 LEAHI-TD-FIRMWARE-LDT-5214_[FW+Dialin] New override for Air Pump Measured Speed parameter ]]> Fri, 07 Aug 2026 14:01:05 -0700 Vendor - TEL - Santhoshkumar Reddy Alasandagutti https://devapps.diality.us/changelog/td-firmware?cs=0fd5f41d073fa1b56d423f9fc3697fa2e88181dd https://devapps.diality.us/changelog/td-firmware?cs=0fd5f41d073fa1b56d423f9fc3697fa2e88181dd Vendor - TEL - Santhoshkumar Reddy Alasandagutti committed 0fd5f41 on branch LDT-2240-disposable-tube-set-self-test, LDT-3992-disposable-tube-set-auto-priming and 11 more in td-firmware space allignment2

· /firmware/App/Controllers/AirPump.c

(-2, +2)

| History

| Source

| Diff

]]> Fri, 07 Aug 2026 13:56:00 -0700 Vendor - TEL - Santhoshkumar Reddy Alasandagutti https://devapps.diality.us/changelog/td-firmware?cs=c60f37d20491d7dccb0c4597ea03b6bf241fe985 https://devapps.diality.us/changelog/td-firmware?cs=c60f37d20491d7dccb0c4597ea03b6bf241fe985 Vendor - TEL - Santhoshkumar Reddy Alasandagutti committed c60f37d on branch LDT-2240-disposable-tube-set-self-test, LDT-3992-disposable-tube-set-auto-priming and 11 more in td-firmware space allignment

· /firmware/App/Controllers/AirPump.c

(-2, +2)

| History

| Source

| Diff

]]> Fri, 07 Aug 2026 13:52:21 -0700 Vendor - TEL - Santhoshkumar Reddy Alasandagutti https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-5214-1/#c31120 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-5214-1/#c31120 Vendor - TEL - Santhoshkumar Reddy Alasandagutti replied to Vendor - TEL - Arpita Srivastava in LEAHI-TD-FIRMWARE-LDT-5214-1

Corrected in latest commit

· /firmware/App/Controllers/AirPump.c

(-1, +3)

| History

| Source

| Diff

]]> Wed, 05 Aug 2026 12:44:35 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c31028 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1/#c31028 Sean Nash commented on LEAHI-TD-FIRMWARE-LDT-4029-1

What if we're in paused state (because your alarm has stop property set)? Should we return TRUE here if in paused state and interrupted state was the await syringe load confirmation state?

Can we please break this line as to match our standard of 100 columns?

SET_ALARM_WITH_2_U32_DATA( ALARM_ID_TD_AIR_PUMP_STALL,
currentAirPumpPowerLevel,
getU32OverrideValue( &currentAirPumpRPM ) );

Set "Stops" property back to TRUE like the other reserved UI alarms.

Updated

break this line

add an empty else block after the last else if block in these functions
handlePreTxHeparinPreloadState
handlePreTxHeparinAwaitSyringeLoadConfirmationState
handlePreTxHeparinSeekState
handlePreTxHeparinPrimeState
handlePreTxHeparinBolusState

break this line

Add initialization for heparinSetupResumeRequested and syringeLoadConfirmed

Align all the function comments

Delete this row.

· /firmware/App/Controllers/AirPump.c

(-2, +2)

| History

| Source

| Diff

]]> Mon, 27 Jul 2026 17:08:53 -0700 Vinayakam Mani https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-5214-1/#c30756 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-5214-1/#c30756 Vinayakam Mani commented on LEAHI-TD-FIRMWARE-LDT-5214-1

the function call seems returning U16. Type cast to U32.

Should this be the "Load Syringe" alarm that you just added?

I think ( setTxDur - preStop ) should be changed to ( preStop ).
Also, I think preStop should be renamed to hepDurMin.