Skip to content

AP_SerialManager: allow for permanent passthrough #30212

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tridge
Copy link
Contributor

@tridge tridge commented Jun 1, 2025

it is sometimes useful to setup a serial passthrough as permanent, this allows for that by setting SERIAL_PASSTIMO to -1

Copy link
Contributor

@andyp1per andyp1per left a comment

Choose a reason for hiding this comment

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

Although I think this is useful there is some danger - I have sometimes made my flight controllers inaccessible and the timeout has saved my bacon.

@tridge
Copy link
Contributor Author

tridge commented Jun 1, 2025

I have sometimes made my flight controllers inaccessible and the timeout has saved my bacon.

so don't enable the option if you don't want it permanent!
you can always install a different vehicle fw to reset
for me this was extremely useful in developing the driver for this PR:
#30213

@Hwurzburg Hwurzburg added the WikiNeeded needs wiki update label Jun 1, 2025
Copy link
Member

@IamPete1 IamPete1 left a comment

Choose a reason for hiding this comment

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

LGTM, can see why it might be useful in some cases.

it is sometimes useful to setup a serial passthrough as permanent,
this allows for that by setting SERIAL_PASSTIMO to -1
@tridge tridge force-pushed the pr-serial-passthru-permanent branch from 27017f3 to 80f11f6 Compare June 2, 2025 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants