Fix UART mapping in dev-mcxa platform to use LPUART2 instead of LPUART3#17
Merged
Merged
Conversation
kurtjd
approved these changes
May 14, 2026
Contributor
There was a problem hiding this comment.
Btw the ec-test-app repo is outdated, development for it has moved to: https://github.com/OpenDevicePartnership/odp-platform-common/tree/main/ec/test-tui
There was a problem hiding this comment.
Pull request overview
This PR corrects the dev-mcxa board UART routing so the ODP UART service uses the MCU-LINK VCOM-connected LPUART2 instead of LPUART3.
Changes:
- Rebinds the LPUART interrupt handler from LPUART3 to LPUART2.
- Updates UART initialization to use LPUART2 with the corresponding P2_2/P2_3 pins.
felipebalbi
approved these changes
May 14, 2026
Contributor
Author
Thanks, tested there too and it works flawlessly! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
According to the FRDM-MCXA266 user manual, the MCU-LINK VCOM port is connected to LPUART2 instead of LPUART3. Forgot to check that in when #12 was merged in, so this PR updates it. Tested to be working with ec-test-app.