What alarm is this? It doesn't look like you've added a new alarm. This is one of the reserved (unused) alarms that UI s/w might use.

Do we really need these setup functions for each state? They all look very simple - just 1 or 2 lines of code.

I believe the SyringePump controller will trigger this alarm at end of seek operation.

Syringe pump should already be stopped if preload is done. Do we need to call this here?

This is called from preload state. Not needed here I think.

Don't put comments on function prototypes. Doxygen gets details of functions from the function header.

Why (void)?

Where is this being called? Should be called from pre-treatment init I think.

Confirmation signal sounds like a message handler from UI. Message handlers must accept a message as parameter and return BOOL. They must also be added to the message handler table in Messaging.c.

Why are we signaling syringe load confirmed here?

Do these states need to be public?

Move up one row to alphabetize.

· /firmware/App/Modes/ModePreTreat.c	(-5, +55)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.h	(-1, +2)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxHeparinSetup.c	(-0, +539)	| History	| Source
· /firmware/App/Modes/StatePreTxHeparinSetup.h	(-0, +62)	| History	| Source
· /firmware/App/Services/AlarmMgmtSWFaults.h	(-0, +1)	| History	| Source	| Diff

]]> Thu, 02 Jul 2026 13:47:35 -0700 Dara Navaei https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4029-1 Dara Navaei started review LEAHI-TD-FIRMWARE-LDT-4029-1 ]]>