Lines of code count not available
Done.
No. It is permanent, I would like to know the value for the homing.
I have to make sure the valve is correct before using it.
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.
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?
Align =
Should be 2 blank lines above/below test banner.
Should we initialize to inactive (and use #define)?
Took note that the FPGA interface is at the driver level.
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.