Skip to content

topology: align pipeline parameters with hardware configuration - #2886

Merged
lgirdwood merged 1 commit into
thesofproject:masterfrom
slawblauciak:ehl_dmic_fix
Jun 4, 2020
Merged

lgirdwood merged 1 commit into
thesofproject:masterfrom
slawblauciak:ehl_dmic_fix

Conversation

@slawblauciak

Copy link
Copy Markdown
Collaborator

The stream parameters set in the pipeline were mismatched with
the DMIC hardware configuration, which resulted in failures.

Signed-off-by: Slawomir Blauciak slawomir.blauciak@linux.intel.com

@slawblauciak

Copy link
Copy Markdown
Collaborator Author

Internal Intel CI tests failing are irrelevant, since this is a topology change.
Something else must've broken them.

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

Can you include "fix" in the commit subject. This way it stands out as a fix for stable branches.

@slawblauciak

Copy link
Copy Markdown
Collaborator Author

Commit message changed.

The stream parameters set in the pipeline were mismatched with
the DMIC hardware configuration, which resulted in failures.

Signed-off-by: Slawomir Blauciak <slawomir.blauciak@linux.intel.com>
@libinyang

Copy link
Copy Markdown
Contributor

@slawblauciak Maybe you can also change the PCM id of PIPELINE_PCM_ADD() to align with PCM_PLAYBACK_ADD()

@lgirdwood
lgirdwood merged commit 9b2b031 into thesofproject:master Jun 4, 2020
@lgirdwood

Copy link
Copy Markdown
Member

@slawblauciak Maybe you can also change the PCM id of PIPELINE_PCM_ADD() to align with PCM_PLAYBACK_ADD()

@slawblauciak @libinyang if this is a new bug then we can do a new PR.

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