Not a float.

Not a float.

Not a float anymore. Use u32Override function and no need to cast the pointer.

Should we publish all registers read?

Remove this comment since we have a comment to right.

Remove comment.

REG_ADDRESSES

DEVICE_ADDRESSES

Set them individually instead of with memset.

Add blank line before a /// comment.

device and register is fine. Or deviceSel and registerSel.

Which alarm?

bpPublishTimerCtr is the output?

Input is none.
Add @Alarms

Should we list the alarm ids? @Alarms?

requestAbortBPReading is the output?

requestPedsBPReading is the output?

requestAdultBPReading is the output?

pendingVitalsRequest is the output?

We should initialize in the init function?

bpMeasurementReady is an output?

nextState is the return value.
Should we add requestAdultBPMeasurement, bpMeasurementReady and bpDriverError to outputs?

Where are we updating the bpResults?

We should initialize this in init functions.

please add a comment

pressure is a param?

command is a param?

Move to driver.

Recommend command be a U08 here. Driver should have the enum and pass the enum to this function. FPGA should just take a byte.

Move publish call from exec to the check state handle function.