Skip to content

zephyr: test: userspace: intel_hda_dma: convert to use IPC service - #11387

Merged
kv2019i merged 1 commit into
thesofproject:mainfrom
kv2019i:202611-test-dma-convert
Nov 3, 2026
Merged

kv2019i merged 1 commit into
thesofproject:mainfrom
kv2019i:202611-test-dma-convert

Conversation

@kv2019i

@kv2019i kv2019i commented Nov 2, 2026

Copy link
Copy Markdown
Collaborator

Convert the test case to use Zephyr IPC service interface.

Convert the test case to use Zephyr IPC service interface.

Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🔵 Needs a closer look

The IPC backend and cavstool harness are outside the snapshot, so endpoint contracts, acknowledgement handling, and the revised DMA drain condition require validation on Intel ADSP hardware.

No open issues 🎉

What changed in this PR

Converts the userspace Intel HDA DMA test to use Zephyr’s IPC service for communication with the cavstool host.

Changes:

  • Replaces direct IPC handlers with endpoint registration, messaging, and cleanup.
  • Adds explicit waits for host acknowledgments and validation results.
  • Updates DMA completion polling to wait until no data remains pending.
File Description
zephyr/​test/​userspace/​test_intel_hda_dma.c Migrates host communication to the IPC service and updates transfer completion and validation synchronization.

Review effort: Balanced

@intel-sofci

Copy link
Copy Markdown

PR 11387: test results

Run date: 2026-11-02 12:00 UTC

Tested commit: b0df52d38250f41564dd9449a778939bce4c456d

mtl pass rate lnl pass rate ptl pass rate wcl pass rate nvl pass rate

@kv2019i
kv2019i merged commit 9dad1cf into thesofproject:main Nov 3, 2026
45 checks passed
@kv2019i

kv2019i commented Nov 3, 2026

Copy link
Copy Markdown
Collaborator Author

FYI @tmleman this old IPC API user now converted

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.

4 participants