Skip to content

Topology: Fix DAI_ADD function call in sof-cml-rt1011-rt5682 topology - #2151

Merged
lgirdwood merged 1 commit into
thesofproject:masterfrom
wwittbrx:cml-rt1011-rt5682_topology_fix
Nov 28, 2019
Merged

lgirdwood merged 1 commit into
thesofproject:masterfrom
wwittbrx:cml-rt1011-rt5682_topology_fix

Conversation

@wwittbrx

Copy link
Copy Markdown
Contributor

This patch fixes DAI_ADD function call for speaker pipeline.
The function call had incorrect number of agruments passed which was causing issues with playback.

@wwittbrx

Copy link
Copy Markdown
Contributor Author

Should fix #2107

This patch fixes DAI_ADD function call for speaker pipeline.
The function call had incorrect number of agruments passed
which was causing issues with playback.

Signed-off-by: WojciechX Wittbrodt <wojciechx.wittbrodt@intel.com>
@wwittbrx
wwittbrx force-pushed the cml-rt1011-rt5682_topology_fix branch from 4db75d1 to 8b6faf0 Compare November 28, 2019 09:50
@sathyap-chrome

Copy link
Copy Markdown
Contributor

Thanks for the fix.

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

Good catch, I've grepped and there are no others.

@lgirdwood
lgirdwood merged commit 9ca0a14 into thesofproject:master Nov 28, 2019
@sathyap-chrome

Copy link
Copy Markdown
Contributor

Hi, This change wont impact the issue we are seeing with 008.
As 008 topology doesnt have TIME_DOMAIN changes.

Can we look at why the issue is on 008.

@keyonjie

Copy link
Copy Markdown
Contributor

@sathyap-chrome We need this fix for no sound on 008 issue, @aiChaoSONG is fixing the no sound issue with an extra kernel patch, good news is that it is done, PR will be sent out soon.

@sathyap-chrome

Copy link
Copy Markdown
Contributor

great thanks

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.

5 participants