Remove this blank line.

Put () around each of the 2 conditions being ANDed.

Remove blank line.

Remove blank line.

Add a blank line after declarations.

Remove blank line.

Should we remove this else if or is it going to be restored later?

Remove extra blank lines.

Shouldn't we be assigning dryBicartState instead of dryBicartSubstate in all of these cases?

Why do we need this?

Remove extra blank line.

Remove extra blank line.

Not clear what GENMASK is doing. What do h and l refer to. This doesn't look like a simple bit mask.

Is this for 64 bit unsigned integer? If not, this is redundant with BIT_BY_POS above.
What does nr stand for?

Let's remove this commented line.

Add blank line before return.

Add blank line after declarations.

Why U32? Why not RECOVER_STATE_T?

Lower case x

Is blood flow in need of a PID controller too?

I don't need to edit this file. I will check with Dara to exclude this file from code review.

only maximum 30 float data is allowed. If that is the case then I cannot broadcast the acid or bicarb control signals.
need to split it or need two message id.

Its a problem . Vinay has to decide it

per formatting for the test support function header. it is format used across many files

fixed

fixed