Re: snd_hda_intel/sst-acpi sound breakage on suspend/resume since 5.6-rc1

From: Pierre-Louis Bossart
Date: Thu Mar 19 2020 - 15:29:22 EST




On 3/19/20 1:35 PM, Cezary Rojewski wrote:
On 2020-03-19 19:24, Dominik Brodowski wrote:
On Thu, Mar 19, 2020 at 06:33:50PM +0100, Cezary Rojewski wrote:

Could you confirm the same happens on your machine when revert of mentioned
patch is not applied ("stream is NULL" messages occur)? Issue may be
harmless but explained sequence does not look right.

Indeed, I still see

haswell-pcm-audio haswell-pcm-audio: warning: stream is NULL, no stream to reset, ignore it.
haswell-pcm-audio haswell-pcm-audio: warning: stream is NULL, no stream to free, ignore it.
haswell-pcm-audio haswell-pcm-audio: FW loaded, mailbox readback FW info: type 01, - version: 00.00, build 77, source commit id: 876ac6906f31a43b6772b23c7c983ce9dcb18a19
haswell-pcm-audio haswell-pcm-audio: warning: stream is NULL, no stream to reset, ignore it.
haswell-pcm-audio haswell-pcm-audio: warning: stream is NULL, no stream to free, ignore it.

though sounds continues to work.


Thanks once again for your input and time!

I'll prepare patches for both issues. My guess is haswell-pcm could be updated to handle 'platform' component param just fine, but it is probably a change of more than few lines. I'd rather revert non-SOF broadwell to its previous behavior and start a separate task from there.

It'd be good to know why a dummy platform component is required though.