Not a float.

Not a float.

Not a float anymore. Use u32Override function and no need to cast the pointer.

Should we publish all registers read?

Remove this comment since we have a comment to right.

Remove comment.

REG_ADDRESSES

DEVICE_ADDRESSES

Set them individually instead of with memset.

Add blank line before a /// comment.

device and register is fine. Or deviceSel and registerSel.

· /firmware/App/Controllers/SyringePump.c	(-4, +4)	| History	| Source	| Diff
· /firmware/App/Controllers/SyringePump.h	(-4, +4)	| History	| Source	| Diff
· /firmware/App/Modes/ModeFault.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModeService.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModeStandby.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModeTreatment.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/AlarmMgmtSWFaults.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/FpgaTD.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/FpgaTD.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/Messaging.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Tasks/TaskPriority.c	(-2, +2)	| History	| Source	| Diff

]]> Thu, 21 May 2026 15:06:50 -0700 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-3505-1 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-3505-1 Vendor - TEL - Jashwant Gantyada and Sean Nash finished reviewing LEAHI-TD-FIRMWARE-LDT-3505-1 LEAHI-TD-FIRMWARE-LDT-3505_Heparin - TD - 04: DEV - Feature Implementation ]]> Wed, 20 May 2026 16:22:57 -0700 Sean Nash https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-3505-1/#c29262 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-3505-1/#c29262 Sean Nash replied to Vendor - TEL - Arpita Srivastava in LEAHI-TD-FIRMWARE-LDT-3505-1

I thought this was a TEST_CONFIG thing. We aren't doing s/w config status, so maybe we should convert it to a TEST_CONFIG.

Let's hard code for time being with a TODO comment. I don't want alarm to be a problem.

Use OVERRIDE_RESET for .override instead of 0.

align indentation, same for line 338 as well.

Yes.
Otherwise, we will have to hard code the calibration record for the time being.

The function getSoftwareConfigStatus() is also part of NVM module.
Moreover, we are planning to remove this function when NVM will be implemented.
So, we will have to perform a cleanup and remove all similar calls that have been currently commented out in other files as well like AirTrap.c, BloodFlow.c, etc.

Is this the right time to zero this volume?

update total volume too?

maybe use target rate for both.

Use #define for array size.

Make the 6 a #define

only 1 return statement

remove comma

return FALSE on else

signal functions should return BOOL.

Move up with rest