-
Notifications
You must be signed in to change notification settings - Fork 16.8k
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
Implement RX bind for CRSF #26375
Implement RX bind for CRSF #26375
Conversation
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.
There should be no caller to this When HAL_CRSF_TELEM_ENABLED is false, so remove it completely.
You could also remove it from the header file if you felt like it; if you do so you cet a compile-time rather than link-time failure which makes it easier to find and fix build problems.
Tested and working with TBS RX |
Preferred my solution which was far less hassle and avoid all sorts of compilation problems |
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.
Just saying my comment has been addressed....
CRSF supports a bind command, this uses it to support mavlink bind for CRSF