app: boards: imx95: switch to using nxp,edma driver#10139
Merged
kv2019i merged 1 commit intothesofproject:mainfrom Jul 28, 2025
Merged
app: boards: imx95: switch to using nxp,edma driver#10139kv2019i merged 1 commit intothesofproject:mainfrom
kv2019i merged 1 commit intothesofproject:mainfrom
Conversation
Zephyr commit 774370a1d712 ("boards: nxp: imx95_evk_mimx9596_m7: add uart
dma support") replaces the configuration of the "edma2" node with that
mandated by the programming model associated with the "nxp,mcux-edma"
compatible. This change was done because the EDMA driver associated with
said compatible is compatible with multiple peripheral drivers as opposed
to the previous "nxp,edma" driver.
However, SOF currently requires using the previous EDMA driver (i.e. the
one associated with the "nxp,edma" driver). Therefore, switch back to the
previous programming model.
Furthermore, the DMA channels used by the SAI IP were also changed. Since
these were not tested, switch back to the previous DMA channels.
Signed-off-by: Laurentiu Mihalcea <laurentiu.mihalcea@nxp.com>
dbaluta
approved these changes
Jul 28, 2025
dbaluta
approved these changes
Jul 28, 2025
Collaborator
|
Thanks! @LaurentiuM1234. I forgot to update the DMA channels used by SAI on imx-stable-v2.14. I will backport your patch once merged. |
kv2019i
approved these changes
Jul 28, 2025
Collaborator
kv2019i
left a comment
kv2019i
left a comment
There was a problem hiding this comment.
@LaurentiuM1234 can this be merged first, and then the Zephyr west update?
Contributor
Author
should be fine. I was expecting the build tool to complain when applying the overlay but I guess not? |
LaurentiuM1234
requested review from
lbetlej,
lgirdwood,
mmaka1 and
plbossart
as code owners
Collaborator
|
@lrudyX @lgirdwood ignored the Intel quickbuild fail, this PR (NXP overlay change only) cannot possibly influence Intel builds. |
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
3 participants
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.
Zephyr commit 774370a1d712 ("boards: nxp: imx95_evk_mimx9596_m7: add uart dma support") replaces the configuration of the "edma2" node with that mandated by the programming model associated with the "nxp,mcux-edma" compatible. This change was done because the EDMA driver associated with said compatible is compatible with multiple peripheral drivers as opposed to the previous "nxp,edma" driver.
However, SOF currently requires using the previous EDMA driver (i.e. the one associated with the "nxp,edma" driver). Therefore, switch back to the previous programming model.
Furthermore, the DMA channels used by the SAI IP were also changed. Since these were not tested, switch back to the previous DMA channels.