Need to set state to preload state I think. Otherwise you will go to interrupted state.

Why did you remote call to init function here? Is it initialized somewhere else?

Use ternary operation to assign TRUE or FALSE explicitly.

This is at end of enum list above. So it should be at end of table here.

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.

I would keep blank line between library includes and regular includes.

We can't have 2 alarms with ID 196. The enum only has the empty saline bag alarm. Looks like alarm 197 is duplicate of this 196 alarm you've added.
I think you should just remove this new alarm 196 that you've added here.

Fix merge conflict stuff here.

Fix merge conflict stuff here.

· /firmware/App/Modes/ModePreTreat.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxRecirculate.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/StatePreTxRecirculate.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtSWFaults.h	(-2, +2)	| History	| Source	| Diff

]]> Fri, 12 Jun 2026 23:07:22 -0700 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4027-1 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4027-1 Sean Nash and Vendor - TEL - Arpita Srivastava finished reviewing LEAHI-TD-FIRMWARE-LDT-4027-1 LEAHI-TD-FIRMWARE-LDT-4027_Pre-Treatment - FW Implementation - 2/6: ]]> Fri, 12 Jun 2026 17:40:28 -0700 Vinayakam Mani https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-2240-1/#c29896 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-2240-1/#c29896 Vinayakam Mani commented on LEAHI-TD-FIRMWARE-LDT-2240-1

Bubble status, Low and high level level sensors.

Input : Blood pump door status?

To be restored?

Are these being initialized in the init function?

Need to resolve the merge issues?

· /firmware/App/Modes/ModePreTreat.h

(-1, +1)

| History

| Source

| Diff

]]>