Current MSP handling groups callbacks by their MSP command
and when a response comes back it calls all the pending handlers
with the same response object.
This works fine when all MSP calls are either pure input or pure
output (e.g. the have a non-empty payload in just one direction).
However, we've had some calls that have a payload in both directions
for some time, like MSPV2_SETTING. For those MSP commands, the
response will depend on the request, so calling all handlers on
the first response received produces the wrong results. It's also
problematic on handlers that expect any kind of response, since the
DataView object is reused, but its offset was never reset which
would result in the second handler getting an empty response.
Change this strategy to call just the first pending handlers for
the MSP command received in the response. While this is still
a theoretical race condition (there's no guarantee a sequence of
the same command with different payloads will be replied in the
same order as they were issued to the FC), it's the best we can
do unless we add some kind of nonce/token system to each MSP
request, which would incur a significant overhead.
- Add Pitch and Roll angles below the rates
- Add help messages indicating that these values constrain the
maximum inclination during navigation.
- Add help messages for navigation max bank/climb/dive angles
- Add help for pitch2thr
Affects the mixer and servos tab. This way they show a couple of
warnings ("no servos" and "upgrade the FC firmware" respectively)
rather than hanging the configurator.
Fixes https://github.com/iNavFlight/inav/issues/3462
When a controllable VTX is configured, a new section appears in
the configuration tab which allows the user to set the band, channel
power and "low power on disarm" option.