We tend to initialize the local variables as a standard practice.

These don't need to be initialized to zero.

Done

Done

Done

Done

Done

Done.

No. It is permanent, I would like to know the value for the homing.

The else in this trinary looks wrong - doesn't round up to multiple of 8.

This if should have an else where we zero .valveOpsStartTime (in case it briefly got set but then failed this if before timeout).

Declarations should be at top of scope.

Is this temporary?

Done

Not sure how Doxygen will handle this comment between the /// and the const. I think we should remove it.

Done

move static function after public function

let public function on top

Let all public functions come on top for all files

Should we comment this out before merging?

Done

Done

Align =

Should be 2 blank lines above/below test banner.

Done

Should we initialize to inactive (and use #define)?

Took note that the FPGA interface is at the driver level.

Done