Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(Other): Add DMA channel number conversion for Zephyr #1307

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

ttmut
Copy link
Contributor

@ttmut ttmut commented Dec 19, 2024

Description

Add a DMA channel conversion function to convert chanel numbers between Zephyr and MSDK. For example, if an SoC has two DMA instances named DMA0 and DMA1, with 4 channels each, DMA1 channel numbers should be offset by 4 before passing to MSDK.

Zephyr DMA0 MSDK DMA0
0 0
1 1
2 2
3 3
Zephyr DMA1 MSDK DMA1
0 4
1 5
2 6
3 7

MAX32657 DMA is an exception to this.

Add a DMA channel conversion function to convert chanel numbers between
Zephyr and MSDK. For example, if an SoC has two DMA instances named DMA0
and DMA1, with 4 channels each, DMA1 channel numbers should be offset by
4 before passing to MSDK.

Signed-off-by: Tahsin Mutlugun <[email protected]>
@ttmut ttmut requested a review from sihyung-maxim December 19, 2024 13:09
@github-actions github-actions bot added the Zephyr MSDK Zephyr related change. label Dec 19, 2024
@ttmut ttmut requested review from hfakkiz and lorne-maxim December 19, 2024 13:10
@sihyung-maxim
Copy link
Contributor

Should I wait until Furkan reviews this PR before merging?

@ttmut
Copy link
Contributor Author

ttmut commented Dec 19, 2024

Should I wait until Furkan reviews this PR before merging?

I guess it can be merged now.

@sihyung-maxim sihyung-maxim merged commit 32d53f8 into main Dec 19, 2024
12 checks passed
@sihyung-maxim sihyung-maxim deleted the feat/dma_chan_wrapper branch December 19, 2024 20:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Zephyr MSDK Zephyr related change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants