Has integrity unit been added to TD project?

Where are these being triggered?

Is this necessary? If so, why?

Updated.

Yes, we need to

Deleted.

Does OperationModes.c need to include TxParams.h? TxParams is not a mode.

I think we should initialize pre-treatment mode here now that we have a pre-treatment mode.

Should we request pre-treatment mode here instead? How do we get to pre-treatment mode?

Yes, I deleted.

Deleted.

Removed

Deleted.

Remove extra blank line.

MODE_TPAR should be deleted.

It should no longer be called from OperationModes.c because we are deleted the TxParams mode (delete from enum and any switch statements in OperationModes.c).

ModeTxParams.c/.h are still not deleted. I want you to delete them and commit/push that change. Then I will see both files in RED here in this CR.

Thanks for the clarification. This handler already follows the single-parameter MESSAGE_T *message pattern, and the message ID is already defined and registered in Messaging.c. I’ll keep using the existing ID, ensure payload length is validated, and parse the param/value from the payload as expected.

Yes, this is called in OperationModes.c

Sure.

Added.

Added.

I didn't see this in header file.

I don't see these Lower/Upper Range Limit functions declared in the header file.

Need to delete this unit (ModeTxParams.c/.h). Also need to remove the mode from the modes enumeration and from OperationModes.c.