· /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

]]> Thu, 07 May 2026 09:32:41 -0700 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-1886-2 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-1886-2 Vendor - TEL - Varshini Nagabooshanam finished reviewing LEAHI-TD-FIRMWARE-LDT-1886-2 LEAHI-TD-FIRMWARE-LDT-1886_Blood Set Auto-Load and Auto-Eject - TD ]]> 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-3505-heparin-td-04-dev-feature-implemenation 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

]]> Wed, 06 May 2026 13:58:47 -0700 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-1886-2 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-1886-2 Sean Nash finished reviewing LEAHI-TD-FIRMWARE-LDT-1886-2 LEAHI-TD-FIRMWARE-LDT-1886_Blood Set Auto-Load and Auto-Eject - TD ]]> Wed, 06 May 2026 11:43:07 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-1886-2/#c28965 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-1886-2/#c28965 Sean Nash commented on LEAHI-TD-FIRMWARE-LDT-1886-2

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

@details \b Outputs:
Please make sure to update all of them to Leahi format In this case, there should be a \b before the type of details (Inputs and Outputs here) so they will be bold when doxygen documentation is generated.

@details \b Inputs:

Can we please add more comments in this function to explain the rationale behind the checks and the actions?

add a space between message and )

Change // to /// for doxygen to catch it.

Can we remove the parameter and switch directly over currentPostTreatmentState?

The public function name signalGoToInstallState is not sufficiently descriptive. Would renaming it to requestTubeSetInstall() be appropriate?

I prefer you create a local BOOL result so you can have a simpler return statement.

Should go to your new complete state here.