Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DNM]Drivers: DAI: Intel: Move ACE DMIC start reset clear to earlier #74328

Conversation

singalsu
Copy link
Collaborator

@singalsu singalsu commented Jun 14, 2024

The unreset of ACE PDM controllers is moved to beginning of dai_dmic_start() function and done side-by-side with the trace print removed from in between.

This change is done for adhering to recommended hardware initialization flow to avoid intermittent failures on ARL-S.

The unreset of ACE PDM controllers is moved to beginning of
dai_dmic_start() function and done side-by-side with the trace
print removed from in between.

This change is done for adhering to recommended hardware
initialization flow to avoid intermittent failures on ARL-S.

Signed-off-by: Seppo Ingalsuo <seppo.ingalsuo@linux.intel.com>
@singalsu singalsu force-pushed the dai_intel_dmic_trigger_start_ace_fix branch from 1818b7f to 9238fe6 Compare June 14, 2024 15:37
@singalsu
Copy link
Collaborator Author

Fixed a typo in commit message.

This patch was created to address issue reported in thesofproject/sof#9205 .

@kv2019i kv2019i added the platform: Intel ADSP Intel Audio platforms label Jun 17, 2024
@singalsu
Copy link
Collaborator Author

Please wait, I need to add more changes. With this change a very small probability for corrupt still exists.

@singalsu singalsu changed the title Drivers: DAI: Intel: Move ACE DMIC start reset clear to earlier [DNM]Drivers: DAI: Intel: Move ACE DMIC start reset clear to earlier Jun 17, 2024
@singalsu
Copy link
Collaborator Author

Closing, #74414 replaces this.

@singalsu singalsu closed this Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants