We generally don't add { } to switch cases unless they require local variable declarations.

If we're not including a timestamp, can we just remove this line of code?

Why commented out?

Swap order of == for these 2 if statements - put constant on left side.

Why?

Why is UF volume being added here? If we add it, we need to change DD f/w too. But it is not clear to me why DD f/w needs to know the UF volume.

Use ternary operator to explicitly set to TRUE or FALSE.

Our TODO comments are all CAPs.

I think these if statements should be looking for valve state == VALVE_STATE_IDLE, not <

Fix implementation to return a BOOL (TRUE if SW_COMPATIBILITY_REV == ddVersion.compatibilityRev.

Should return T/F. Change comment to "TRUE if DD f/w is compatible with TD f/w, FALSE if not.".

Should return a BOOL (that is what ModeStandby is expecting).

Fix isDDCompatible() function so that it returns a T/F instead of a compatibility revision.

Align comment.

Why are these added? And why commented out. H13 and H20 are not rotary valves - they are 3 way valves. They do not belong here.

Change INPROGRESS to OPEN (because you're going to remove INPROGRESS).

You can do this sendMessage just once after else.

add a space before and after 0

add a blank line above

add a blank line above

minor spell issue : change responceStatus to responseStatus

add offRequestAwaitingUserConfirmation and offRequestPendingTimer to outputs

should we move the initialization in the init function?

please break this line into two lines

Vendor - TEL - Arpita Srivastava
please add a space before and after in sizeof(U32) in [sizeof(U32)]

please add a space before and after 0 in payload[0]

output is none

please break the line in two or more.

add a space before and after index 'i'

add a space before and after index 'i'