-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Add additional MSP-based sensors #6090
Conversation
@tridge - FYI to keep things aligned with AP_Periph |
I think we should add a uint8_t instance number to all sensors, so we can support dual GPS, mag, baro etc in the future. |
69e3a1c
to
d5108f3
Compare
Added |
Tested baro, compass and GPS - all working. Compass default alignment is now hardcoded in the driver so it's CW0 rotation for the user. |
Configurator support PR iNavFlight/inav-configurator#1062 |
@digitalentity I think we should add an MSP airspeed driver too. |
…lement MSP-based BARO and MAG
… Disable baro median filtering by default; Skip autodetection of MSP-based sensors (only manual selection is allowed)
eab0ecd
to
c91b91a
Compare
@digitalentity please read here, also my comments at the end:: #7576 |
Required to support AP_Periph-based sensors (i.e. a new CAN-GPS from @MATEKSYS)