Skip to content

[DNM] Tools: Topology2: Set hda-generic DMIC pipeline to high priority#9224

Closed
singalsu wants to merge 3 commits intothesofproject:mainfrom
singalsu:tplg2_dmic_p0_other_p1
Closed

[DNM] Tools: Topology2: Set hda-generic DMIC pipeline to high priority#9224
singalsu wants to merge 3 commits intothesofproject:mainfrom
singalsu:tplg2_dmic_p0_other_p1

Conversation

@singalsu
Copy link
Collaborator

No description provided.

singalsu added 3 commits June 12, 2024 13:38
This patch replaces all occurrences of priority 0 with a macro
set priority. There is no effective change since the macro is
defined to zero.

Successive patches will adjust the relative priorities of
pipelines. This allows e.g. to set all pipelines except DMIC
capture to priority 1 while DMIC is priority 0.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
The host pipeline in dmic-generic.conf is missing priority, so
it is added. Also the passthrough alternative dai and host
pipelines are missing the core and priority pipeline attributes
set so they are added.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
The DMIC capture pipeline is set to priority 0 while other
pipelines are set to priority 1. This change should prevent
overrun of DMIC FIFO and data corruption.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
@singalsu singalsu force-pushed the tplg2_dmic_p0_other_p1 branch from 118fe66 to 5ca948c Compare June 12, 2024 14:05
@singalsu singalsu changed the title Tools: Topology2: Set hda-generic DMIC pipeline to high priority [DNM] Tools: Topology2: Set hda-generic DMIC pipeline to high priority Jun 12, 2024
@lgirdwood lgirdwood added this to the v2.11 milestone Jun 25, 2024
@lgirdwood
Copy link
Member

Lets revisit this after v2.10. Seems we need to work out why priority is different.

@singalsu
Copy link
Collaborator Author

@lgirdwood Closing this for now. The issue with ARL-S DMIC was solved other way.

@singalsu singalsu closed this Jul 24, 2024
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.

2 participants