This needs TD source update and also dialin update, keep informed system team about this change as this would impact DD demo running.

Please add HDF Pump speed register and add a monitor in the substitution pump controller to compare against the target speed and if the difference is out of range, trigger an alarm.

Substitution pump

Please add the min and max for tmp as well in the outputs.

rejReason = INVALID REQUEST FORMAT

Missing comment.

update comment. next one as well.

blank line after the banner and maintain alphabetical order of include files.

I think these remaining 4 else ifs need to have another && condition that your service is in idle state so that we don't change back into one of these 4 states while a bolus is in progress.

Can we restore the real code here yet? Ask Dara.

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.

Done

Done

Use the NEARLY_ZERO constant (instead 0f 0.01F) like the other one.