Skip to content

Conversation

ZhaoxiangJin
Copy link
Contributor

@ZhaoxiangJin ZhaoxiangJin commented Oct 11, 2025

The EDMA1 controller on MCXN236 SoCs has 8 channels not 16.
see:https://github.com/zephyrproject-rtos/hal_nxp/blob/master/mcux/mcux-sdk-ng/devices/MCX/MCXN/MCXN236/MCXN236_COMMON.h#L162
85-92 are resolved

@ZhaoxiangJin
Copy link
Contributor Author

add @NeilChen93 to check

@ZhaoxiangJin ZhaoxiangJin changed the title dts: arm: nxp: correct mcxn236 edma1 channels correct mcxn236 edma instance1 interrupt numbers Oct 11, 2025
dleach02
dleach02 previously approved these changes Oct 13, 2025
@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
Copy link
Member

@decsny decsny left a comment

Choose a reason for hiding this comment

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

if the reason is because there is only 8 dma channels, shouldn't you also change dma-channels from 16 to 8

1. MCXN23x edma1 has 8 channels, not 16.
2. MCXN23x edma0 and edma1 have 94 requests, not 120.
3. MCXNx4x edma0 and edma1 have 117 requests, not 120.

Signed-off-by: Zhaoxiang Jin <[email protected]>
@ZhaoxiangJin
Copy link
Contributor Author

if the reason is because there is only 8 dma channels, shouldn't you also change dma-channels from 16 to 8

yes, should change to 8

Copy link

@ZhaoxiangJin
Copy link
Contributor Author

update:
1.correct MCXN23x edma1 controller dma-channels to 8.
2.MCXN23x edma0 and edma1 have 94 requests, not 120. see https://github.com/zephyrproject-rtos/hal_nxp/blob/master/mcux/mcux-sdk-ng/devices/MCX/MCXN/periph1/PERI_DMA.h
3.MCXNx4x edma0 and edma1 have 117 requests, not 120. see https://github.com/zephyrproject-rtos/hal_nxp/blob/master/mcux/mcux-sdk-ng/devices/MCX/MCXN/periph/PERI_DMA.h#L123

@ZhaoxiangJin ZhaoxiangJin changed the title correct mcxn236 edma instance1 interrupt numbers correct mcxn23x and mcxnx4x edma properties information Oct 14, 2025
@jhedberg jhedberg merged commit 757ffe5 into zephyrproject-rtos:main Oct 15, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: Boards/SoCs platform: NXP MCU platform: NXP NXP 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.

7 participants