fix: some radio settings were overwritten or not applied

[raphaelcoeffic]

• fix default VCP configuration (was not applied)
• fix internal module configuration (was always overwritten with default module)

[pfeerick]

Fixes both the VCP not being set to CLI issue on supported radios (i.e. TX16S, Zorro) we were talking about. I was going to say it also fixes the internalModule == OFF issue when upgrading, but for me at least, this had basically no effect - since it was correctly set when going from 2.5 -> 2.7rc1. I think that culprit is somewhat intermittent, but this might at least catch one more instance of it not being set.

[TheIsotopes]

@raphaelcoeffic Is it normal that VCP after a change (e.g. OFF) is always set back to CLI?
found on TX16S and Zorro

[raphaelcoeffic]

@raphaelcoeffic Is it normal that VCP after a change (e.g. OFF) is always set back to CLI?
found on TX16S and Zorro

Yes, as designed. The point being here that ELRS relies on CLI for passthrough flashing. So if it is OFF, it will go back to default, which is CLI.

[pfeerick]

We could possibly make it more specific though? IF Internal Module == CRSF, default it back to CLI if off/not set? As it's not needed for MULTI... or is that because you're thinking we would be update MPM firmware through CLI in the future also? It's not like it's really a problem since that is/was the primary use for VCP (or as debug).

[raphaelcoeffic]

I would really like being able to update the MPM via passthrough as well! And it should even be possible in fact.