done updated below in the function as well.

done.

Intentional?

update the header and field comments in later commits

If we don't think we need this, let's just delete it. If we think it's coming back, let's keep it and just set the max count to something really high (e.g. 1,000,000).

done.

Update I/O section here to include rotor revs accumulated and previous.

handled accordingly.

removed and added.

Converting U16 to U32. Presumably, U16 will wrap back to zero when it gets to 65535. We don't want our U32 to wrap, so we need to do something to handle this conversion properly.

I think this whole function should be moved to the PeristalticPump.c driver unit. We don't want controller/monitor level units directly accessing fpga registers - the drivers should be doing that.

documentation error- is it transition start time ?

Done

Then put an #if 0 around this if and add a TODO comment to implement the test configuration.

Done

Done

None?

None?

I don't see this input used in function.

Done

Done

Done

The checks are back in a generic function.