Function headers that we copy from Denali will not have some of the new formatting changes - make sure to update all of them to Leahi format (see from other units for example). In this case, there should be a \b before the type of details (Inputs and Outputs here) so they will be bold when documentation is generated.
Add an else block and keep the state to idle. In the beginning of the function assign the state to BP_DRIVER_CLEAR_CMD_STATE and remove line 194, 202 and 208.
This unit does not compile w/o errors - always build and check for errors before committing/pushing changes. At least one issue is that we need to include Valve3Way.h and I suspect other includes are missing as well (check bottom of compiler warnings list).
This change clamps tValue between [min, max]. For example, given value=2, min=5, max=10, step=1 and increment is called, without this change value will be incremented to 3 (still outside [min, max]), with this change value will be incremented to 5, pulling it into the valid range.
