Skip to content

Conversation

@ranj063
Copy link
Collaborator

@ranj063 ranj063 commented Aug 30, 2021

Tearing down pipelines is only supported with firmware
ABI 3.19 or greater.

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

Tearing down pipelines is only supported with firmware
ABI 3.19 or greater.

Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
@ranj063 ranj063 force-pushed the fix/pause_resume_v2 branch from 087a8e5 to 763d7aa Compare August 30, 2021 20:26
@plbossart
Copy link
Member

I tested this with firmware 1.8, without this patch either the device does not wake at all, or I get his error on resume

[  101.715109] sof-audio-pci-intel-cnl 0000:00:1f.3: ipc tx: 0x30100000: GLB_TPLG_MSG: PIPE_NEW
[  101.715202] sof-audio-pci-intel-cnl 0000:00:1f.3: ipc tx error for 0x30100000 (msg/reply size: 48/20): -22
[  101.715204] sof-audio-pci-intel-cnl 0000:00:1f.3: error: failed to load widget PIPELINE.1.ALH2.OUT
[  101.715206] sof-audio-pci-intel-cnl 0000:00:1f.3: error: failed to restore pipeline after resume -22
[  101.715208] PM: dpm_run_callback(): pci_pm_resume+0x0/0x80 returns -22
[  101.715216] sof-audio-pci-intel-cnl 0000:00:1f.3: PM: failed to resume async: error -22

@plbossart
Copy link
Member

One suspend-resume timeout in https://sof-ci.01.org/linuxpr/PR3126/build6253/devicetest/?model=TGLU_RVP_SDW&testcase=check-suspend-resume-with-playback, let's redo the tests. @bardliao can you take a look?

@plbossart
Copy link
Member

SOFCI TEST

@bardliao
Copy link
Collaborator

Thanks @ranj063 This fixes the issue I saw in #3098 (comment)

Copy link
Collaborator

@bardliao bardliao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM and no issue found on pr-device-test.

@plbossart plbossart merged commit bed17ef into thesofproject:topic/sof-dev Aug 31, 2021
@ranj063 ranj063 deleted the fix/pause_resume_v2 branch February 13, 2022 05:10
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