Support edma3 for s32z270dc2 board #78368
Merged
Conversation
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
haduongquang
requested review from
dleach02,
manuargue and
mmahadevan108
as code owners
zephyrbot
added
area: DMA
zephyrbot
requested review from
Dat-NguyenDuy,
MarkWangChinese,
PetervdPerk-NXP,
bperseghetti,
danieldegrasse,
dbaluta,
decsny and
teburd
|
The following west manifest projects have been modified in this Pull Request:
Note: This message is automatically posted and updated by the Manifest GitHub Action. |
zephyrbot
added
manifest
manifest-hal_nxp
DNM
Dat-NguyenDuy
requested changes
Sep 13, 2024
tests/drivers/dma/chan_blen_transfer/boards/s32z2xxdc2_s32z270_rtu1.overlay
Outdated
Show resolved
Hide resolved
tests/drivers/dma/chan_blen_transfer/boards/s32z2xxdc2_s32z270_rtu1.overlay
Outdated
Show resolved
Hide resolved
haduongquang
force-pushed
the
support-edma3-for-s32z-2.0.0
branch
7 times, most recently
from
6d05b95 to
8efc8f3
Compare
Contributor
Author
|
Added commit to Split |
Contributor
Author
|
@teburd @manuargue @Dat-NguyenDuy please help me take a look into this PR. All comments addressed. Thanks |
manuargue
previously approved these changes
Sep 28, 2024
Dat-NguyenDuy
previously approved these changes
Sep 30, 2024
Contributor
Author
|
Hi @teburd, Could you please help me take a look into this PR? |
haduongquang
dismissed stale reviews from Dat-NguyenDuy and manuargue
via
a47d71e
haduongquang
force-pushed
the
support-edma3-for-s32z-2.0.0
branch
from
262db2c to
a47d71e
Compare
Contributor
Author
|
I rebased and fixed the conflict. |
manuargue
approved these changes
Oct 9, 2024
Dat-NguyenDuy
previously approved these changes
Oct 9, 2024
dleach02
previously approved these changes
Oct 15, 2024
Member
|
@teburd I've add @manuargue (NXP S32 platform maintainer) as a co-assignee so we can move forward with this PR. |
Member
|
@haduongquang HAL pr merged, please rebase and update manifest to 4597b16cfedf5553cb155151e65eb994d5d0ef25 |
Support multiple instances for dma. Memset struct data and tcdpool on initialization is not necessary. Therefore remove it. Signed-off-by: Ha Duong Quang <[email protected]>
haduongquang
dismissed stale reviews from dleach02 and Dat-NguyenDuy
via
9158449
haduongquang
force-pushed
the
support-edma3-for-s32z-2.0.0
branch
from
a47d71e to
9158449
Compare
Contributor
Author
I rebased and updated the manifest |
Enable support EDMA for S32Z270. Add eDMA3 instance 0, 1, 4 and 5 for S32Z270 devices. Signed-off-by: Ha Duong Quang <[email protected]>
Enable dma boards/test for S32Z270. The non-cacheable memory used for DMA tests (chan_blen_transfer and loop_transfer) is split from the system SRAM. Signed-off-by: Ha Duong Quang <[email protected]>
haduongquang
force-pushed
the
support-edma3-for-s32z-2.0.0
branch
from
9158449 to
aeb0801
Compare
manuargue
approved these changes
Oct 18, 2024
Member
manuargue
left a comment
manuargue
left a comment
There was a problem hiding this comment.
dismiss #78368 (review), my mistake, first we need to merge this pr
Dat-NguyenDuy
approved these changes
Oct 18, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 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.
This PR introduces NXP S32 EDMA3 driver for SoC NXP S32Z27 and enables its usage for board s32z270dc2
Tests:
tests\drivers\dma\chan_blen_transfer:
SUITE PASS - 100.00% [dma_m2m]: pass = 4, fail = 0, skip = 0, total = 4 duration = 8.063 secondstests\drivers\dma\chan_link_transfer:
SUITE PASS - 100.00% [dma_m2m_link]: pass = 3, fail = 0, skip = 0, total = 3 duration = 6.047 secondstests\drivers\dma\loop_transfer:
SUITE PASS - 100.00% [dma_m2m_loop]: pass = 3, fail = 0, skip = 0, total = 3 duration = 1.103 secondstests\drivers\dma\scatter_gather:
SUITE PASS - 100.00% [dma_m2m_sg]: pass = 1, fail = 0, skip = 0, total = 1 duration = 0.076 seconds