Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
IronPort-SDR: mUEOjIbPACRYj1nzmCBLTuxLmJibW9swawMSSt7jE05SUMMReCnJWlXX95nfK5ZfEVR1J5S2B1
 oJlA3cuowBfQ==
IronPort-SDR: ORroYAM1KHIe5G6nl1CYBrtbrzPa8jYhFyIQ6qqrD+QGHGs+Q8O/LQlNvfrX8z0P0CccpA88s0
 Om4sJrb4I/Sw==
Subject: Re: snd_hda_intel/sst-acpi sound breakage on suspend/resume since
 5.6-rc1
To:     Dominik Brodowski <linux@dominikbrodowski.net>, tiwai@suse.com
Cc:     pierre-louis.bossart@linux.intel.com,
        liam.r.girdwood@linux.intel.com, yang.jie@linux.intel.com,
        broonie@kernel.org, perex@perex.cz, alsa-devel@alsa-project.org,
        linux-kernel@vger.kernel.org
References: <20200318063022.GA116342@light.dominikbrodowski.net>
From:   Cezary Rojewski <cezary.rojewski@intel.com>
Message-ID: <66c719b3-a66e-6a9f-fab8-721ba48d7ad8@intel.com>
Date:   Wed, 18 Mar 2020 10:41:42 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101
 Thunderbird/68.5.0
MIME-Version: 1.0
In-Reply-To: <20200318063022.GA116342@light.dominikbrodowski.net>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On 2020-03-18 07:30, Dominik Brodowski wrote:
> Hi!
> 
> While 5.5.x works fine, mainline as of ac309e7744be (v5.6-rc6+) causes me
> some sound-related trouble: after boot, the sound works fine -- but once I
> suspend and resume my broadwell-based XPS13, I need to switch to headphone
> and back to speaker to hear something. But what I hear isn't music but
> garbled output.
> 
> A few dmesg snippets from v5.6-rc6-9-gac309e7744be which might be of
> interest. I've highlighted the lines differing from v.5.5.x which might be
> of special interest:
> 

Thank you for the report, Dominik. You definitely got our attention.

I've checked the market: Dell XPS 13 9343, yes? Once you confirm model 
id, I'll order a piece immediately to our site.

In regard to logs, thanks for highlighting important lines. Build is of 
'rc' so bugs can still be in plenty - any reason for switching to 
cutting-edge kernel on production stuff? Our CI didn't detect any 
anomalies yet as it is running on 5.5.

I'll direct your ticket on todays meeting. On the first look, issue 
seems to be connected with recent changes to /drivers/dma/dmaengine.c. 
DesignWare DMA controller drv - which HSW/BDW makes use of - might not 
have been updated accordingly. Will dig further on that.

One more, just to make it clear for the rest of the viewers:

 > 	haswell-pcm-audio haswell-pcm-audio: Direct firmware load for 
intel/IntcPP01.bin failed with error -2
 > 	haswell-pcm-audio haswell-pcm-audio: fw image intel/IntcPP01.bin not 
available(-2)

Back in the ancient days of DSP (HSW/BDW are actually the very first 
audio DSP hws for Intel) topology was part of FW - SW could not 
configure it and probably that's why library IntcPP01 is attempted to be 
loaded on every boot, even if it's not part of configuration for given 
hw. Maybe we could make it quieter though..

> 
> (these last two messages already are printed a couple of time after boot, and then
> again during a suspend/resume cycle. On v.5.5.y, there are similar messages
> "no context buffer need to restore!"). Everything is built-in, no modules
> are loaded.
> 
> Unfortunately, I cannot bisect this issue easily -- i915 was broken for
> quite some time on this system[*], prohibiting boot...

Hmm, sounds like that issue is quite old. DSP for Haswell and Broadwell 
is available for I2S devices only, so this relates directly to legacy 
HDA driver. Compared to Skylake+, HDAudio controller for older platforms 
is found within GPU. My advice is to notify the DRM guys about this issue.

Takashi, are you aware of problems with HDMI on HSW/ BDW or should I 
just loop Jani and other DRM peps here?

Czarek