Update header

Align indent.

Add Doxygen comments.

Align indent

0.0F

p40RPMCnts -> typecast to F32.

mention as 30.0F, its being used in float calculations.

p40RPMCnts variable defined as U16, but assigning U32 value.

My understanding is that Standby should stop all pumps and heaters, including D108. Although the D108 driver implementation is not available yet, I thought keeping this call would preserve the intended Standby behavior and allow it to work once driver support is added. Could you please let me know if there is any concern with leaving it in place for now?

Michael confirmed that FP will eventually support two firmware-controlled sampling modes: Filter Sample and RO Permeate Sample. Since the flow paths are not defined yet, we will keep a generic placeholder for the sampling mode/state and update the implementation and requirements once the feature is better defined.

Add TODO or revert back changes.

Delete empty line

Add the comment.

Align the indent throughout.

Add comment.

Space before and after paranthesis.

If debug is complete, do you want to revert back to actual values? if you still need it for some more time, please add TODO comment.

Update the comment.

Aling the indentation of value field.

Align the indentation.

you need to bring in the new set of valves here as well.

I would recommend commenting the stopping D108 heater here, as there is no implementation yet at driver level. Add a TODO comment for now.

We don't have a specific DD permeate sample mode implementation yet. We need to align with Michael on this. Looks FP may need a mode to handle permeate request (if FP was not in gen permeate mode by TD request already).

Align the indent. Replace RO to FP.