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

TBS Sixty9 SA2.1 UART Configuration #7985

Merged
merged 2 commits into from May 8, 2022

Conversation

erstec
Copy link
Collaborator

@erstec erstec commented Apr 24, 2022

Solves #7938

TBS Sixty9 VTX SmartAudio 2.1 not working with 8N2 UART configuration (tested on two Sixty9's I have, v6.13 and 6.17).
New CLI parameter vtx_smartaudio_sixtynine, default is OFF added.
After enabling 8N1 choosen for SA.

docs/Settings.md Outdated
@@ -5932,6 +5932,16 @@ Enable workaround for early AKK SAudio-enabled VTX bug.

---

### vtx_smartaudio_sixtynine
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@erstec let's make the name more generic like vtx_smartaudio_stopbits with default 2. What do you think?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@DzikuVx sure! Updated commit forcepushed, together with rebase on actual 'master'

@DzikuVx DzikuVx added this to the 5.0 milestone May 7, 2022
@erstec erstec requested a review from DzikuVx May 7, 2022 18:55
Copy link
Member

@DzikuVx DzikuVx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My bad, I forgot about PG version bump

src/main/io/vtx_control.c Show resolved Hide resolved
@erstec erstec requested a review from DzikuVx May 7, 2022 20:29
@DzikuVx DzikuVx added the Release Notes Add this when a PR needs to be mentioned in the release notes label May 8, 2022
@DzikuVx DzikuVx merged commit fc5c336 into iNavFlight:master May 8, 2022
@MrD-RC
Copy link
Collaborator

MrD-RC commented May 12, 2022

@erstec Could information on setting this up also be added to https://github.com/iNavFlight/inav/blob/master/docs/VTx.md please.

@erstec erstec mentioned this pull request May 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release Notes Add this when a PR needs to be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants