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

AP_GPS: Add driver option to allow GPS autobaud bypass and instead use configured SERIALx_BAUD #25488

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Ryanf55
Copy link
Collaborator

@Ryanf55 Ryanf55 commented Nov 8, 2023

This is a draft of allowing users to manually configure the baud rates. See #25449 for more context, which illustrated the need for a generic way to handle manual baud rates rather than driver-specific bits or params.

The goal of this work is to allow using any of the manual baud rates from SERIAL3_BAUD and bypass autodetection. With this feature, the correct driver is immediately instantiated, and more immune behavior to GPS power cycles triggering autobaud sequences.

Because the only way that SBF and SBF2 are differentiated is in their detection sequence, this probably shouldn't be used on those devices. Or, run the detect sequence for SBF/SBF2, but don't switch baud if the detect fails.

* Adds ability to stop autobaud on all receivers

Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
@Ryanf55 Ryanf55 changed the title AP_GPS: Add driver option to allow GPS autobaud bypass AP_GPS: Add driver option to allow GPS autobaud bypass and instead use configured SERIALx_BAUD Nov 8, 2023
Signed-off-by: Ryan Friedman <ryanfriedman5410+github@gmail.com>
@Ryanf55 Ryanf55 force-pushed the gps-bypass-autobaud-use-serial-manager-baud branch from 9f58821 to 80fed87 Compare November 8, 2023 06:50
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.

1 participant