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

ASoC: SOF: Intel: fix suspend/resume for non-HDA mode. #152

Merged
merged 1 commit into from
Sep 20, 2018

Conversation

keyonjie
Copy link

For non-HDA mode, we don't handle the pp and init/stop chip related,
here correct it.

Signed-off-by: Keyon Jie yang.jie@linux.intel.com

For non-HDA mode, we don't handle the pp and init/stop chip related,
here correct it.

Signed-off-by: Keyon Jie <yang.jie@linux.intel.com>
@keyonjie
Copy link
Author

@ZhendanYang @ranj063 this should fix the non-HDA compiling issue.

@keyonjie
Copy link
Author

@lgirdwood please help review.

@juimonen
Copy link

for me in this patch works nicely. I can compile the kernel without disabling sof_hda and pipeline elements are destroyed after streaming in up2 (I guess that means pm works).

@lgirdwood lgirdwood merged commit 0064eda into thesofproject:topic/sof-dev Sep 20, 2018
@juimonen
Copy link

I guess this still has some issues. I made some changes to processing components which might hide the pm not still working... anyway this at least enables the compilation

@keyonjie
Copy link
Author

I guess this still has some issues. I made some changes to processing components which might hide the pm not still working... anyway this at least enables the compilation

Hi @juimonen , you are right, maybe the subject here is somewhat confusing, the commit here is only fixing compiling issue.

For the pm not working case, can you share more info which may help on debugging it?

paulstelian97 pushed a commit to paulstelian97/linux that referenced this pull request May 4, 2020
…ion/add-contribution-page

Documentation: Add contributions page
aiChaoSONG pushed a commit to aiChaoSONG/linux that referenced this pull request May 6, 2021
Remove the remaining instances of `KernelResult<()>`.
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