-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
SITL: implemented built-in serial receivers support in SITL, implemented FC proxy mode, updated SITL docs #9365
Conversation
implemented FC Proxy for SITL update SITL.md
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NAK: This will not (does not) work in Linux, nor will it work in MacOS.
Working example in https://github.com/stronnag/bbl2kml/tree/master/tools/ser2tcp/eg_c or I can make a PR to your repo (let me know if you'd prefer that option). |
Is it intentional that both the serial device node (
|
This is what I actually was hoping for. I am not Linux/MasOs user and will not be able to test it. Please make a PR. |
OK, I'll try and get something done this weekend. |
Yes, why not? User can use TCP Port or COM port without changing configuration, both will work. |
…ort_setting [SITL] Update SITL serial proxy posix port setting
…it-serial-receiver
Merged pull request RomanLut#12 to branch RomanLut:submit-serial-receiver. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm content. Nice to get rid of the external serial bridge for physical RX.
@DzikuVx apropos testing, in addition to @RomanLut 's Windows testing: Test StatusLinux (Arch, aarch64, riscv64, x86_64; Alpine ia32)CRSF (ELRS)via
IBUSvia
FC Proxy (Sbus TX)
Correct movement of sticks etc. seen in Configurator. FreeBSD (x64_64)IBUSvia
CRSF (ELRS)via
Correct movement of sticks etc. seen in Configurator. MacOS (x64_86 kvm/qemu)IBUSvia
Correct movement of sticks etc. seen in Configurator. CRSF not tested as environment does support pl2303/400000 baud (however,
|
Just wanted to say thanks for writing:
It saved me from a big headache. |
Retested on Linux ... time to merge. |
…ted FC proxy mode, updated SITL docs (iNavFlight#9365) * implemented built-in serial receivers support in SITL implemented FC Proxy for SITL update SITL.md * Update SITL.md * Update SITL.md * Update SITL.md * Update SITL.md * [SITL] update POSIX serial functions to support non-standard baud rates * disable xon/hof on serial port (SITL serial_proxy) --------- Co-authored-by: Jonathan Hudson <[email protected]>
Requires Configurator changes iNavFlight/inav-configurator#1849
Tested under
Windows
Linux (Arch, aarch64, riscv64, x86_64; Alpine ia32)
FreeBSD (x64_64)
MacOS (x64_86 kvm/qemu)
Please see https://github.com/RomanLut/inav/blob/submit-serial-receiver/docs/SITL/SITL.md for changes.