please add blank space between ( (int)postState ), TRUE );

please add blank space between ( (int)postState ), TRUE );

Alarm id should be 196?
Description looks same as 194.

Change PRES_OCCL to PRESSURE

Maybe move OperationModes.c?

Use generic confirmation functions/messages.

Move this function to OperationModes.c.

From here forward, we can just call sendMessage() with appropriate params.

Use generic helper function (sendMessage) in MessageSupport.c.

Use generic confirmation functions/messages for off button.

Remove change.

· /firmware/App/Controllers/SyringePump.c	(-4, +4)	| History	| Source	| Diff
· /firmware/App/Controllers/SyringePump.h	(-4, +4)	| History	| Source	| Diff
· /firmware/App/Modes/ModeFault.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModeService.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModeTreatment.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtSWFaults.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/FpgaTD.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/FpgaTD.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/Messaging.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Tasks/TaskPriority.c	(-2, +2)	| History	| Source	| Diff

]]> Thu, 21 May 2026 15:06:50 -0700 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-3505-1 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-3505-1 Vendor - TEL - Jashwant Gantyada and Sean Nash finished reviewing LEAHI-TD-FIRMWARE-LDT-3505-1 LEAHI-TD-FIRMWARE-LDT-3505_Heparin - TD - 04: DEV - Feature Implementation ]]> Wed, 20 May 2026 16:22:57 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-3505-1/#c29262 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-3505-1/#c29262 Sean Nash replied to Vendor - TEL - Arpita Srivastava in LEAHI-TD-FIRMWARE-LDT-3505-1

I thought this was a TEST_CONFIG thing. We aren't doing s/w config status, so maybe we should convert it to a TEST_CONFIG.

Let's hard code for time being with a TODO comment. I don't want alarm to be a problem.

Use OVERRIDE_RESET for .override instead of 0.

align indentation, same for line 338 as well.

Yes.
Otherwise, we will have to hard code the calibration record for the time being.

The function getSoftwareConfigStatus() is also part of NVM module.
Moreover, we are planning to remove this function when NVM will be implemented.
So, we will have to perform a cleanup and remove all similar calls that have been currently commented out in other files as well like AirTrap.c, BloodFlow.c, etc.

Align comment.

Keep 2 blank lines above banner.

Should this line be un-commented?

So we are going to fail this test until calibration record is ready?

In Leahi, we want to do all initialization in the init function instead of here at declaration (see Bubble.c for example).

· /firmware/App/Controllers/SyringePump.c	(-150, +238)	| History	| Source	| Diff
· /firmware/App/Controllers/SyringePump.h	(-20, +18)	| History	| Source	| Diff
· /firmware/App/Modes/ModeFault.c	(-1, +0)	| History	| Source	| Diff
· /firmware/App/Modes/ModeService.c	(-1, +0)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.c	(-1, +0)	| History	| Source	| Diff
· /firmware/App/Services/FpgaTD.c	(-1, +0)	| History	| Source	| Diff
· /firmware/App/Services/Messaging.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Tasks/TaskPriority.c	(-3, +3)	| History	| Source	| Diff

]]> Thu, 07 May 2026 11:35:49 -0700 Vendor - TEL - Arpita Srivastava https://devapps.diality.us/changelog/td-firmware?cs=16d6398acc1c6cf604d9dc993b27c94d895132e9 https://devapps.diality.us/changelog/td-firmware?cs=16d6398acc1c6cf604d9dc993b27c94d895132e9 Vendor - TEL - Arpita Srivastava committed 16d6398 on branch LDT-1886-blood-set-auto-load-and-auto-ej, LDT-2240-disposable-tube-set-self-test and 20 more in td-firmware LDT-3505: Removed syringe detection override functions

· /firmware/App/Controllers/SyringePump.c	(-43, +0)	| History	| Source	| Diff
· /firmware/App/Controllers/SyringePump.h	(-2, +0)	| History	| Source	| Diff
· /firmware/App/Services/FpgaTD.c	(-6, +6)	| History	| Source	| Diff
· /firmware/App/Services/Messaging.c	(-1, +0)	| History	| Source	| Diff
· /firmware/source/sys_main.c	(-0, +2)	| History	| Source	| Diff

]]> Wed, 06 May 2026 17:06:09 -0700 Vendor - TEL - Arpita Srivastava https://devapps.diality.us/changelog/td-firmware?cs=de1c5ec969a5529cbdb7e66a4168fc11b9b3e58a https://devapps.diality.us/changelog/td-firmware?cs=de1c5ec969a5529cbdb7e66a4168fc11b9b3e58a Vendor - TEL - Arpita Srivastava committed de1c5ec on branch LDT-1886-blood-set-auto-load-and-auto-ej, LDT-2240-disposable-tube-set-self-test and 20 more in td-firmware LDT-3505: Removed Syringe Detection Checks

· /firmware/App/Controllers/SyringePump.c	(-106, +4)	| History	| Source	| Diff
· /firmware/App/Controllers/SyringePump.h	(-1, +0)	| History	| Source	| Diff

]]>