-
Notifications
You must be signed in to change notification settings - Fork 205
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
[BACKPORT] UART ioctl's #54
Conversation
… to invert and swap RX/TX on STM32F7 UARTs. I added the TIOCGINVERT as well to reserve the IOCTL number, but did not implement it. This is the same as for TIOCGSINGLEWIRE.
…equest #930) serial: add TIOCSINVERT and TIOCSSWAP ioctl's for stm32{l4,h7} and imxrt * stm32l4 serial: add TIOCSINVERT and TIOCSSWAP ioctl's * stm32l4 serial: ensure TIOCSSINGLEWIRE is atomic and UE disabled * stm32h7 serial: add TIOCSINVERT and TIOCSSWAP ioctl's * imxrt serial: add TIOCSINVERT ioctl Approved-by: Gregory Nutt <gnutt@nuttx.org>
Great, this looks much cleaner. Given the timing of an imminent PX4/Firmware NuttX minor bump (7.28 -> 7.29 PX4/PX4-Autopilot#12344) how about merging into If testing hits any issues that drag it out beyond this week we can reconsider. |
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.
Looks great - @bkueng - thank you for getting it in.
Yes that's fine with me. We can also apply it to both branches, since it's not doing anything w/o corresponding FW changes, and then pull into whichever comes first. |
Yes both is fine, I just didn't want it to get lost in the submodule change. |
This brings the
TIOCSINVERT
andTIOCSSWAP
UART ioctl's from NuttX master.It allows us to handle the RC port in a cleaner way with less board-specific code and configurable RC port.