Can these be done without loop (just fix the size to cover both columns of the array)?

This memset should be deleted since handled above in loop now.

Add a blank line after declaration.

implement it in a single structure packing by moving all other pragma pack

done

done

changed

done

done

done

done.

done.

done.

declared in the header and updated them.

Resolved. Thanks

Resolved. Thanks

We can but not necessary - only necessary when using == operator.

Can we write it as ( NU_EMB_MODE_CMD != expChar1 ) ?

Minor: There is a typo with Inputs

For init functions, I'm ok with saying something like "Blood leak driver unit variables initialized".

Remove extra blank line.

Should these 2 #defines be converted to an enum? If we did that, we could add a "NUM_OF_" to end and use the "NUM_OF_" enum for array declarations instead of literal "2".

Some of these memset calls are zeroing a 2-dimensional array, so the size param is too short (only sizing 1st dimension).

Ok as is. We only need to have constant/literal first when using == operator to prevent confusion with = operator.

Actually, for casting, I prefer no spaces around ().

Declare these earlier and just assign them here (i.e. keep declarations together at top of function).