-
Notifications
You must be signed in to change notification settings - Fork 1.7k
extend msp_override_mask to 32bits / channels #10891
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
I'm going through and marking a bunch of PRs as testing needed, then I'm going to encourage some folks to start testing. |
|
Tested successfully with |
|
Tested OK with 32bit mask: Both 16bit (ch12) and 32bit(ch20) are overwritten No channels are overwritten. |
|
Will run some tests later today as well. But I would say we can add this to a 8.1 release since it does not change existing behavior, even if existing config value is re-used. |
|
It changes the size of the parameter group, requiring a full erase |
* extend msp_override_mask to 32bits / channels * update other override 16bit variables
* extend msp_override_mask to 32bits / channels * update other override 16bit variables
After the number of channels was raised to 34 in INAV 8, it seems appropriate to allow a 32bit / 32 channel MSP override mask.