Skip to content

Xmc interrupt uart fix#430

Merged
dineshgit411 merged 6 commits intomasterfrom
xmc-interrupt-uart-fix
Feb 27, 2026
Merged

Xmc interrupt uart fix#430
dineshgit411 merged 6 commits intomasterfrom
xmc-interrupt-uart-fix

Conversation

@dineshgit411
Copy link
Copy Markdown
Contributor

@dineshgit411 dineshgit411 commented Feb 23, 2026

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---

CONTRIBUTING.md also tells you what to expect in the PR process.

Description
https://jirard.intra.infineon.com/browse/DESMAKERS-4820
Code execution stopped after attachInterrupt in XMC1400 boards. Due to use same channel name in uart and interrupt

Signed-off-by: MDin <Dinesh.M-EE@infineon.com>
Signed-off-by: MDin <Dinesh.M-EE@infineon.com>
Signed-off-by: MDin <Dinesh.M-EE@infineon.com>
Signed-off-by: MDin <Dinesh.M-EE@infineon.com>
Signed-off-by: MDin <Dinesh.M-EE@infineon.com>
@dineshgit411 dineshgit411 marked this pull request as ready for review February 24, 2026 12:59
@jaenrig-ifx
Copy link
Copy Markdown
Member

All good if you have checked it works in the specific boards?
We still need to the diagram update?
And the HIL back to life?

@dineshgit411
Copy link
Copy Markdown
Contributor Author

All good if you have checked it works in the specific boards? We still need to the diagram update? And the HIL back to life?

yes. i tested with specific boards, and diagram i will update. HIL need to check

@ederjc
Copy link
Copy Markdown
Member

ederjc commented Feb 26, 2026

Please convert back to "Ready for Review" when HiL is passing and the diagrams are updated

@ederjc ederjc marked this pull request as draft February 26, 2026 17:29
Signed-off-by: MDin <Dinesh.M-EE@infineon.com>
@dineshgit411 dineshgit411 marked this pull request as ready for review February 27, 2026 11:06
@LinjingZhang
Copy link
Copy Markdown
Contributor

LinjingZhang commented Feb 27, 2026

By creating this pull request you agree to the terms in CONTRIBUTING.md. https://github.com/Infineon/.github/blob/master/CONTRIBUTING.md --- DO NOT DELETE ANYTHING ABOVE THIS LINE ---

CONTRIBUTING.md also tells you what to expect in the PR process.

Description Provide the information we need to review your PR. What problem does the pull request solve? "Bug fix" is not a good description.

Related Issue If you opened an issue before creating the PR, point to it here.

Context What do we need to know about your development environment, tools, target, and so on. Screenshots are always helpful if there is a UI element to this PR.

It would be most helpful if you could provide some additional information/Jira link in the description😄
I assume that certain boards/pins were not tested in the HiL, which is also not necessary. However, it would be beneficial next time to provide manual test results for those specific pins (with changes made in test_config) to ensure they are functioning correctly.

Copy link
Copy Markdown
Contributor

@LinjingZhang LinjingZhang left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Copy Markdown
Member

@jaenrig-ifx jaenrig-ifx left a comment

Choose a reason for hiding this comment

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

All good! Not sure if the diagram has enough resolution. Please check in the readthedocs generated html. If that is good enough, all fine 👍

@ederjc
Copy link
Copy Markdown
Member

ederjc commented Feb 27, 2026

All good! Not sure if the diagram has enough resolution. Please check in the readthedocs generated html. If that is good enough, all fine 👍

Resolution looks fine for me, the included image anyway has limited resolution.

@dineshgit411 dineshgit411 merged commit 3e35ec4 into master Feb 27, 2026
273 of 322 checks passed
@dineshgit411 dineshgit411 deleted the xmc-interrupt-uart-fix branch February 27, 2026 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants