Restore autobaud for m8 gps #9219
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Will setup some hardware on my bench to diagnose further
Collaborator
Author
|
M8 seems to top up 460800. I may add a gps_max_autobaud_speed and set default to 230400, for compatibility with older units. |
Some old devices seem to reset if set to too high of a baud rate. A particular old m8 devices tops up at 115200. INAV 6.0 used to go all the way up to 230400
Collaborator
Author
|
Ok, we should now be able to keep everyone happy. This change keeps support for 460800 and 921600. As a bonus, should allow auto baud to work with @stronnag 's m8 with very old api version to work, by setting gps_auto_baud_max to 115200. |
Collaborator
Author
|
Verified working both with m8q and m10 from iflight. |
rmaia3d
pushed a commit
to rmaia3d/inav
that referenced
this pull request
Sep 9, 2023
…d-more-speeds Restore autobaud for m8 gps
rmaia3d
pushed a commit
to rmaia3d/inav
that referenced
this pull request
Feb 19, 2024
…d-more-speeds Restore autobaud for m8 gps
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #9218
Add gps_auto_baud_max with a default setting of 230400, matching INAV 6.0 behavior.
Newer GPS units may be set to higher baud rates and some old units seem to have bugs causing them to reset and revert 38400 if you attempt to auto baud higher than 115200.