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

simplicity_sdk: Import dmadrv and patch dma channel allocator #85

Merged

Conversation

Martinhoff-maker
Copy link
Collaborator

The goal of this PR is to import the DMA driver (dmadrv) from Simplicity SDK and patch it to maintain synchronization of allocated channel between zephyr DMA Driver et simplicity_SDK DMA driver. DMADRV is only supposed to be called by zephyr DMA driver.

@Martinhoff-maker Martinhoff-maker marked this pull request as ready for review February 7, 2025 13:52
@Martinhoff-maker Martinhoff-maker force-pushed the dmadrv_import_and_patch branch 2 times, most recently from 4424ebc to 86fc494 Compare February 7, 2025 14:32
@jhedberg jhedberg requested a review from asmellby February 7, 2025 14:42
Prepare for import of DMADRV HAL functions.

Signed-off-by: Martin Hoff <martin.hoff@silabs.com>
Origin: Simplicity SDK
License: Zlib
URL: https://github.com/SiliconLabs/simplicity_sdk
Commit: 8627f8482564dc6d94b56512740a39d6f409a0eb
Version: 2024.12.0
Purpose: Import dmadrv in order to simplify the use of other hal driver.
         It has be done escpecially for future sl_* driver.

Signed-off-by: Martin Hoff <martin.hoff@silabs.com>
@jhedberg
Copy link
Member

jhedberg commented Feb 7, 2025

@Martinhoff-maker do you have a corresponding main tree PR that references this? That's needed so that we can get some CI results before merging.

@Martinhoff-maker
Copy link
Collaborator Author

Martinhoff-maker commented Feb 7, 2025

@jhedberg No but I'm doing it right now where I update CmakeLists for sisdk module with these new files.

Patch needed to remain synchronize between zephyr DMA channels allocator
and dmadrv channel allocator.

Signed-off-by: Martin Hoff <martin.hoff@silabs.com>
@Martinhoff-maker
Copy link
Collaborator Author

Martinhoff-maker commented Feb 7, 2025

The upstream PR : zephyrproject-rtos/zephyr#85402
@asmellby I run some test manually like all the DMA test and the USART async test and everything is ok.

@jerome-pouiller jerome-pouiller changed the title Simplicity_sdk: Import dmadrv and patch dma channel allocator simplicity_sdk: Import dmadrv and patch dma channel allocator Feb 10, 2025
@jhedberg jhedberg merged commit 1e60d97 into zephyrproject-rtos:main Feb 11, 2025
@jhedberg
Copy link
Member

@Martinhoff-maker feel free to update the main tree west.yml reference

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants