Skip to content

drivers: Intel: dmic: make DMIC prm and active fifo count shared - #4195

Merged
lgirdwood merged 1 commit into
thesofproject:mainfrom
ranj063:fix/dmic_shared
May 16, 2021
Merged

lgirdwood merged 1 commit into
thesofproject:mainfrom
ranj063:fix/dmic_shared

Conversation

@ranj063

@ranj063 ranj063 commented May 14, 2021

Copy link
Copy Markdown
Collaborator

When the DMIC pipeline is moved to a core other than the
primary code, the DMIC params and dmic_active_fifos are
not valid. Modify them to be shared data and handle
cache management when accessing/modifying them.

Signed-off-by: Ranjani Sridharan ranjani.sridharan@linux.intel.com

@ranj063
ranj063 requested a review from singalsu as a code owner May 14, 2021 21:28
@ranj063
ranj063 requested review from keyonjie, lgirdwood and lyakh May 14, 2021 21:28
@ranj063
ranj063 force-pushed the fix/dmic_shared branch from fb1ed4c to 466ac35 Compare May 14, 2021 21:31
@ranj063 ranj063 closed this May 14, 2021
@ranj063 ranj063 reopened this May 14, 2021
@ranj063
ranj063 force-pushed the fix/dmic_shared branch 2 times, most recently from a8fcd5f to f2e73dc Compare May 14, 2021 22:33
When the DMIC pipeline is moved to a core other than the
primary core, the DMIC params and dmic_active_fifos are
not valid. Modify them to be shared data and handle
cache management when accessing/modifying them.

Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
@ranj063
ranj063 force-pushed the fix/dmic_shared branch from f2e73dc to 073f1f9 Compare May 14, 2021 22:36
@lgirdwood

Copy link
Copy Markdown
Member

@ranj063 good find, I'll cherry-pick for rc2

@lgirdwood
lgirdwood merged commit eb9b04a into thesofproject:main May 16, 2021
@keyonjie

Copy link
Copy Markdown
Contributor

Hi @ranj063 we are still seeing DSP panic with this, looks we need to use uncached mode of dmic_prm[] also.
@aiChaoSONG to confirm.

@keyonjie

Copy link
Copy Markdown
Contributor

@ranj063 @lgirdwood one more change needed per my observation:
#4196

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