· /firmware/App/Controllers/BloodFlow.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Controllers/Ejector.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Controllers/Ejector.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModeInitPOST.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModePostTreat.c	(-3, +3)	| History	| Source	| Diff
· /firmware/App/Modes/ModePostTreat.h	(-3, +3)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModePreTreat.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/OperationModes.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtSWFaults.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/Messaging.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetAutoEject.c	(-3, +3)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetAutoEject.h	(-3, +3)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetInstall.c	(-3, +3)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetInstall.h	(-3, +3)	| History	| Source	| Diff

]]> Wed, 20 May 2026 15:48:42 -0700 Vendor - TEL - Arpita Srivastava https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-1886-2/#c29259 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-1886-2/#c29259 Vendor - TEL - Arpita Srivastava commented on LEAHI-TD-FIRMWARE-LDT-1886-2

We can rename priorState to previousInstallState.
Also, it should be declared as a static variable to be able to retain previousState value when the control goes out of the function.

There is still merge conflict?
Please resolve merge conflict and build the code in your local before pushing.

This merge conflict is still unresolved.

leak in the comment section.

Where is alarm 191..203?

please add bpLeftHomeTimerCounter to inputs and outputs.

please add bpLeftHomeTimerCounter to inputs and outputs.

· /firmware/App/Controllers/BloodFlow.h	(-0, +3)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetAutoEject.c	(-12, +19)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetInstall.c	(-14, +21)	| History	| Source	| Diff

]]> Tue, 12 May 2026 13:57:24 -0700 Vendor - TEL - Arpita Srivastava https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-1886-2/#c29090 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-1886-2/#c29090 Vendor - TEL - Arpita Srivastava commented on LEAHI-TD-FIRMWARE-LDT-1886-2

We only add File level or global variables in Inputs and Outputs.
Please remove functions from the input section of this and other functions.

Treatment mode will also need one of these (later). Should we add PreTx to the function name to differentiate?

Add a detail for Message Sent for the broadcast data message.

create local Bool

Add a detail for Message Sent for the broadcast data message.

There should be a \b before the type of details
update it for all

please add a comment.

How are we entering the complete state? If we are planning to implement this function later, can we please add a TODO?

There should be a \b before the type of details I/p and O/p please update for all

currentAutoEjectState should be input?

currentAutoEjectState should be input?

add autoEjectTimerCounter to outputs

As per MISRA 15.7, All if, else if constructs shall be terminated with an else statement.

As per MISRA 15.7, All if, else if constructs shall be terminated with an else statement.

change to ///< Tubing set install state

Put it back

There should be a \b before the type of details (Inputs and Outputs here) for doxygen update for all