-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
Drivers: Serial: Refactored [tx/rx]-invert as generic uart properties #72947
Drivers: Serial: Refactored [tx/rx]-invert as generic uart properties #72947
Conversation
Hello @cx-anuj-pathak, and thank you very much for your first pull request to the Zephyr project! |
a1b3757
to
a3fa092
Compare
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.
LGTM, minor stuff.
a3fa092
to
9b6e98b
Compare
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'd suggest to split the change into several commit:
- Add the new symbols in the header, define new bindings
- Get the drivers to support the new bindings
Sure I'll do that. |
4e3f9b8
to
8591faa
Compare
FYI - CI Failures seem to be related to GH/CI issues. Let's re-run them after they're settled. |
8591faa
to
42704c3
Compare
@henrikbrixandersen @dcpleung @erwango @ubieda Hi guys, I have planned simplify this PR, keeping in mind that runtime tx/rx inversion capability is not needed as of now.
Let me know if you have any objections. otherwise I'll proceed with suggested changes. |
Not sure a separate header is needed for this - can't we just keep these two booleans in the config structure of the drivers supporting it? Otherwise sounds good to me. |
Without extended config structure in a separate header file each driver file will have duplicates. EDIT: agreed to merge only yaml changes. |
Refactored vendor-specific duplicates of uart tx/rx invert properties into a common dts file uart-controller-pin-inversion.yaml Signed-off-by: Anuj Pathak <anuj@croxel.com>
42704c3
to
0c7e6c7
Compare
@henrikbrixandersen @erwango @dcpleung Can you please review the latest changes, I'm good with merging only yaml changes |
Hi @cx-anuj-pathak! To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge. Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁 |
Initially TX/RX-inverted capability is only supported in STM32 driver, But recently NXP and ESP driver support is also added. With these changes adding support in additional vendors will be easy