Updated

Added

Add a comment:
result already FALSE - do nothing more for modes that do not support jump to sub-mode.

Add a comment here:
Do nothing for modes that do not support jump to sub-mode.

Updated

Updated, thanks

'genPermeateState' in the outputs. please update across the files as needed.

Validates the requested sub mode change is acceptable or not?

That's a question for me, will that be required?

Fault has sub modes(3), Standby has 1 sub mode(I am adding code for this as this might be required)
Init don't have any and service is not defined yet

Updated

Updated

Changed and left them as static, as we don't need these outside this unit

You are right, (missed that while duplicating the functions)

if ( TRUE == result )

Since both states require no special handling, remove breaks and group with default - all modes.

Remove comment.

Add comment that all of these states require no special handling.

I think these functions (validate and signal) should be "test" functions and moved down to Dialin test functions section below - for all modes.

Add comment that no action is required for these modes - defaulting to invalid request.

I think this switch should be on requested mode - we haven't changed modes yet.

We have 1 param (and probably should have 2 params - requested mode and submode).

currentMode is an input.