-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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 rc_type_crsf #2162
add rc_type_crsf #2162
Conversation
5e73213
to
feb3888
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @benjinne As per #2163 (comment) I think this is a good change in that it reflects the way that flight stacks are using the message currently - we can't break that.
@julianoes FYI, I'm going to merge. For all the same radio types as previously the same command params would be set for the same radio. @benjinne Also has a QGC implementation, so if there are issues, that will catch them.
Based on recommendation to add for CRSF binding
PX4/PX4-Autopilot#23294 (comment)