Skip to content
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

ELRS improvements #22266

Merged
merged 6 commits into from
Dec 22, 2022
Merged

ELRS improvements #22266

merged 6 commits into from
Dec 22, 2022

Conversation

andyp1per
Copy link
Collaborator

@andyp1per andyp1per commented Nov 22, 2022

Shave off a few of the rough edges when using ELRS including radio failsafes that occur at higher frame rates

It seems the main change that people need to make for ELRS is to set the UART RX baudrate to 416666 when upgrading

Adds a bit to RC_OPTIONS to allow CRSF to bootstrap at 420000

Prompted by ExpressLRS/ExpressLRS#1773

@timtuxworth
Copy link
Contributor

Hi @andyp1per, this looks pretty awesome. Do you think you can make the loop rate change for Plane too - it might be helpful particularly for QuadPlanes.

@rmackay9
Copy link
Contributor

@andyp1per,

If possible we should always have a description for each PR. Even if what the PR is doing is somewhat obvious to some devs, it's almost never obvious to everyone.

@andyp1per andyp1per force-pushed the pr-crsf-elrs branch 2 times, most recently from 39315cb to 92dfe1e Compare December 22, 2022 08:20
Move get_link_rate() and get_protocol_string() to CRSF protocol
allow ELRS at 420kbaud to be configured
allow CRSF to bootstrap at ELRS desired baudrate
Encode actual protocol being used
cleanup is_elrs() and version numbers
@IamPete1 IamPete1 merged commit 96060ef into ArduPilot:master Dec 22, 2022
@andyp1per andyp1per deleted the pr-crsf-elrs branch December 22, 2022 19:41
@andyp1per andyp1per mentioned this pull request Jan 7, 2023
@tridge
Copy link
Contributor

tridge commented Jan 7, 2023

@andyp1per if we want this for 4.3.3, can you do a backport PR against Copter-4.3 or Plane-4.3? It doesn't apply cleanly and I can't test it

@andyp1per
Copy link
Collaborator Author

@tridge #22606

@andyp1per andyp1per mentioned this pull request Jan 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants