Skip to content

dai-zephyr: add a DAI PRE_COPY trigger - #5963

Merged
lgirdwood merged 1 commit into
thesofproject:mainfrom
abonislawski:dai-zephyr_trigger_pre-copy
Jul 11, 2022
Merged

lgirdwood merged 1 commit into
thesofproject:mainfrom
abonislawski:dai-zephyr_trigger_pre-copy

Conversation

@abonislawski

Copy link
Copy Markdown
Member

This is similar to 8f9b769
but using zephyr native dai method

@lgirdwood lgirdwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why is this needed and how does it differ from dai-legacy.c ?

Comment thread src/audio/dai-zephyr.c Outdated
@abonislawski

Copy link
Copy Markdown
Member Author

@lgirdwood this is PR which introduced such approach to eliminate DMIC task:
#5852

The problem was that it was added in legacy way to both dai-legacy and dai-zephyr so Jaska removed this from dai-zephyr and Im adding it again in zephyr native way.

So the difference from dai-legacy is a call to zephyr dai rather than sof dai.

There are also changes to Zephyr dai so Zephyr DMIC can use this:
https://github.com/abonislawski/zephyr/commits/DmicUpstream

@marcinszkudlinski marcinszkudlinski 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.

go ahead

@lgirdwood lgirdwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Lets clean up a little.

Comment thread src/audio/dai-zephyr.c Outdated
Add a DAI COPY trigger to be called before dma_copy

Signed-off-by: Adrian Bonislawski <adrian.bonislawski@intel.com>
@abonislawski
abonislawski force-pushed the dai-zephyr_trigger_pre-copy branch from da56e33 to 33e7dd0 Compare July 7, 2022 10:23
@abonislawski

Copy link
Copy Markdown
Member Author

@lgirdwood updated with comment and return check with warning
also changed PRE_COPY to just general COPY

@lgirdwood
lgirdwood merged commit ef39318 into thesofproject:main Jul 11, 2022
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