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

Pierre-Louis Bossart
2020-03-19T19:29:22Z

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.