[Intel-gfx] [PATCH 1/2] ALSA: x86: Fix runtime PM for hdmi-lpe-audio

[Ville Syrjälä]

On Thu, Oct 25, 2018 at 07:50:29AM +0100, Chris Wilson wrote:
> Quoting Chris Wilson (2018-10-24 17:32:04)
> > Quoting Ville Syrjälä (2018-10-24 17:28:45)
> > > On Wed, Oct 24, 2018 at 04:52:22PM +0100, Chris Wilson wrote:
> > > > Quoting Ville Syrjala (2018-10-24 16:48:24)
> > > > > From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> > > > > 
> > > > > Commit 46e831abe864 ("drm/i915/lpe: Mark LPE audio runtime pm as
> > > > > "no callbacks"") broke runtime PM with lpe audio. We can no longer
> > > > > runtime suspend the GPU since the sysfs  power/control for the
> > > > > lpe-audio device no longer exists and the device is considered
> > > > > always active. We can fix this by not marking the device as
> > > > > active.
> > > > 
> > > > Do you also want to send with a HAX patch to force selection of SND_X86
> > > > for CI?
> > > 
> > > Hmm. Would have been a good idea a few minutes ago. Not sure I want
> > > to spam the list again. Depends how long it'll take to adjust the ci
> > > .config I suppose.
> > 
> > Trybot and link to the results?
> 
> https://intel-gfx-ci.01.org/tree/drm-tip/Trybot_3118/issues.html
> 
> The fi-byt-clapper pm tests passed which is the most significant result.
> For enabling LPE audio in CI, it appears we need to trick
> snd_hdmi_lpe_audio.ko into unloading on demand.

Re-ran the test for good measure after fixing the module unload in igt.
Now all green.

https://intel-gfx-ci.01.org/tree/drm-tip/Trybot_3140/issues.html

-- 
Ville Syrjälä
Intel