Skip to content

iut: Add test case to test SEDI dma between SRAM and IMR#70

Merged
kwd-doodling merged 2 commits intomainfrom
dwang4_dma_imr
Sep 19, 2025
Merged

iut: Add test case to test SEDI dma between SRAM and IMR#70
kwd-doodling merged 2 commits intomainfrom
dwang4_dma_imr

Conversation

@kwd-doodling
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new test case to verify SEDI DMA functionality for data transfer between SRAM and IMR (Intel Memory Region) memory areas.

  • Implements DMA transfer functions for copying data between SRAM and IMR regions
  • Adds comprehensive test cases including stress testing with variable data lengths
  • Integrates the new test into the CMake build system with proper conditional compilation

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
test_sedi_dma.c New test implementation for SEDI DMA functionality between SRAM and IMR
CMakeLists.txt Adds conditional compilation of the new DMA test based on CONFIG_DMA_SEDI

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread zephyr/iut_test/test_sedi/test_sedi_dma.c Outdated
Comment thread zephyr/iut_test/test_sedi/test_sedi_dma.c
Comment thread zephyr/iut_test/test_sedi/test_sedi_dma.c
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread zephyr/iut_test/test_sedi/test_sedi_dma.c Outdated
Comment thread zephyr/iut_test/test_sedi/test_sedi_dma.c Outdated
Comment thread zephyr/iut_test/test_sedi/test_sedi_dma.c
Comment thread zephyr/iut_test/test_sedi/test_sedi_dma.c
Comment thread zephyr/iut_test/test_sedi/test_sedi_dma.c Outdated
@kwd-doodling kwd-doodling force-pushed the dwang4_dma_imr branch 2 times, most recently from 7fa54c2 to efee7af Compare September 19, 2025 06:49
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread zephyr/iut_test/test_sedi/test_sedi_dma.c
@kwd-doodling kwd-doodling marked this pull request as ready for review September 19, 2025 06:53
usage: iut run sedi_dma_imr [mode] [len]
Add a stress test with increased DMA length.

Signed-off-by: Dong Wang <dong.d.wang@intel.com>
They're temperarily disabled in Zephyr dts files for ISH. Get them enabled
in IUT to make sure their build is not broken by new patches.

Signed-off-by: Dong Wang <dong.d.wang@intel.com>
@kwd-doodling kwd-doodling merged commit 82a33b2 into main Sep 19, 2025
5 checks passed
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