· /firmware/App/Controllers/AirPump.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/StateTxPaused.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtSWFaults.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtTD.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/FluidBolus.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/FluidBolus.h	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetAutoEject.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetAutoEject.h	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetInstall.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetInstall.h	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Services/SystemCommTD.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/TxParams.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/TDCommon.h	(-2, +2)	| History	| Source	| Diff

]]> Thu, 02 Jul 2026 15:06:54 -0700 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4013-2 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4013-2 Vendor - TEL - Jashwant Gantyada and Sean Nash finished reviewing LEAHI-TD-FIRMWARE-LDT-4013-2 LEAHI-TD-FIRMWARE-LDT-4013_Fluid Bolus - FW Implementation - 2/4: ]]> Thu, 02 Jul 2026 14:59:07 -0700 Vendor - TEL - Praneeth Bunne https://devapps.diality.us/changelog/td-firmware?cs=88816d9971ae12f9c6d66b4fee68774caa3ba8d3 https://devapps.diality.us/changelog/td-firmware?cs=88816d9971ae12f9c6d66b4fee68774caa3ba8d3 Vendor - TEL - Praneeth Bunne committed 88816d9 on branch LDT-4013-fluid-bolus-1, LDT-4028-pre-treatment---fw-implementation and 5 more in td-firmware LDT-4013-1: Address CR comments

· /firmware/App/Modes/ModeStandby.c	(-5, +5)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.h	(-1, +1)	| History	| Source	| Diff

]]> Thu, 02 Jul 2026 14:42:11 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4013-2/#c30350 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4013-2/#c30350 Sean Nash commented on LEAHI-TD-FIRMWARE-LDT-4013-2

Align =

Should modality be an enum type?

Align =

Should this be an enum type?

· /firmware/App/Modes/ModeStandby.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/FluidBolus.c	(-1, +1)	| History	| Source	| Diff
· /firmware/App/Services/TxParams.c	(-8, +8)	| History	| Source	| Diff

]]> Wed, 01 Jul 2026 13:36:09 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4013-2/#c30318 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4013-2/#c30318 Sean Nash commented on LEAHI-TD-FIRMWARE-LDT-4013-2

What is the 3rd modality?

Return type should be an enum (TREATMENT_TYPE_T).

Return type should be an enum (FLUID_BOLUS_MEDIUM_T).

Move after Modes so they are alphabetized.

· /firmware/App/Modes/ModeStandby.c	(-3, +66)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.h	(-0, +11)	| History	| Source	| Diff
· /firmware/App/Modes/StateTxPaused.c	(-5, +2)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/FluidBolus.c	(-25, +5)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/FluidBolus.h	(-1, +0)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetInstall.c	(-0, +35)	| History	| Source	| Diff
· /firmware/App/Services/StateServices/TubeSetInstall.h	(-1, +2)	| History	| Source	| Diff
· /firmware/App/Services/TxParams.c	(-9, +18)	| History	| Source	| Diff

]]> Mon, 15 Jun 2026 11:36:28 -0700 Vendor - TEL - Arpita Srivastava https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4013-2/#c29916 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4013-2/#c29916 Vendor - TEL - Arpita Srivastava replied to Sean Nash in LEAHI-TD-FIRMWARE-LDT-4013-2

Thank you for the clarification!

break into two lines

Add alarm ALARM_ID_TD_EMPTY_SALINE_BAG

Add newAlarmIndicateStop to output

Add newAlarmIndicateStop to input and output.

Arpita, we don't want to use doxygen style comments for function prototypes since we cover functions with the function headers. so this is the proper type of comments.

pauseBolusResumeState is input

add alarm

Replace // to ///< for line 56 to 60

Use // comments for function prototypes.

Either no comment or // comment for function prototypes since doxygen captures function details from function headers.

These were fine (and preferred) as // because doxygen gets function details from function headers in the .c file.

Should we be looking at tubing set type instead of modality here?

Can we not determine this now? Why TODO?