Skip to content

Conversation

remdzi
Copy link
Contributor

@remdzi remdzi commented Oct 9, 2025

The DMA configurations for TX and RX were mixed-up letting the DMA RX channel not fully configured.

This fix correctly configures the DMA RX channel with DMA_ADDR_ADJ_NO_CHANGE.

Related to issue #97289

Fixes: #97289

Copy link

github-actions bot commented Oct 9, 2025

Hello @remdzi, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

@remdzi remdzi force-pushed the remdzi/fix-driver-dma-uart-mcux-flexcomm branch from bcb1fc9 to 0370791 Compare October 9, 2025 15:34
@zephyrbot zephyrbot added platform: NXP NXP area: UART Universal Asynchronous Receiver-Transmitter labels Oct 9, 2025
@remdzi remdzi force-pushed the remdzi/fix-driver-dma-uart-mcux-flexcomm branch from c34da05 to 0370791 Compare October 9, 2025 17:28
@zephyrbot zephyrbot added the size: XS A PR changing only a single line of code label Oct 9, 2025
@mjchen0
Copy link
Contributor

mjchen0 commented Oct 9, 2025

I concur this was a typo bug in my CL and the fix looks correct.

@DerekSnell DerekSnell linked an issue Oct 10, 2025 that may be closed by this pull request
1 task
@dleach02 dleach02 added the Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc. label Oct 13, 2025
@mmahadevan108
Copy link
Contributor

@remdzi , thanks for your contribution. Please fix the commit message, link to guidelines:
https://docs.zephyrproject.org/latest/contribute/guidelines.html#commit-message-guidelines

@remdzi remdzi force-pushed the remdzi/fix-driver-dma-uart-mcux-flexcomm branch from 0370791 to a768105 Compare October 14, 2025 08:56
The DMA configurations for TX and RX  were mixed-up
letting the DMA RX channel not fully configured.

This fix correctly configures the DMA RX channel
with DMA_ADDR_ADJ_NO_CHANGE.

Signed-off-by: Rémy Dziemiaszko <[email protected]>
@remdzi remdzi force-pushed the remdzi/fix-driver-dma-uart-mcux-flexcomm branch from a768105 to ab03a90 Compare October 14, 2025 09:14
Copy link

Please retry analysis of this Pull-Request directly on SonarQube Cloud

@cfriedt cfriedt merged commit c34e224 into zephyrproject-rtos:main Oct 14, 2025
26 checks passed
Copy link

Hi @remdzi!
Congratulations on getting your very first Zephyr pull request merged 🎉🥳. This is a fantastic achievement, and we're thrilled to have you as part of our community!

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! 🪁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: UART Universal Asynchronous Receiver-Transmitter platform: NXP NXP size: XS A PR changing only a single line of code Trivial Changes that can be reviewed by anyone, i.e. doc changes, minor build system tweaks, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

drivers: uart_mcux_flexcomm: fix dma rx config

7 participants