[Intel-gfx] [PATCH] drm/i915: Acquire intel_runtime_pm for HD-Audio registers

[Chris Wilson]

On Fri, Jul 15, 2016 at 02:40:42AM +0000, Yang, Libin wrote:
> Hi Wilson,
> 
> > -----Original Message-----
> > From: Chris Wilson [mailto:chris at chris-wilson.co.uk]
> > Sent: Thursday, July 14, 2016 10:34 PM
> > To: Mika Kuoppala <mika.kuoppala at linux.intel.com>; intel-
> > gfx at lists.freedesktop.org; Takashi Iwai <tiwai at suse.de>; Yang, Libin
> > <libin.yang at intel.com>
> > Subject: Re: [Intel-gfx] [PATCH] drm/i915: Acquire intel_runtime_pm for HD-
> > Audio registers
> > 
> > On Wed, Jul 13, 2016 at 05:01:02PM +0300, Marius Vlad wrote:
> > > Did try when you submitted the patch...but can't seem to replicate
> > > with latest nightly on other SKLs, and currently do not have access on
> > > the machine that caused it.
> > 
> > So fwiw, Hans de Geode confirmed that only reverting 03b135cebc47 (which
> > enables the display powerwell as well) was enough.
> > 
> > Libin, do you want to handle the revert or explain the mistake in
> 
> What's the problem with the patch? I can find the details from the email thread.
> 
> HSW/BSW need acquire the power because it is inside the gpu. So we 
> limited need i915 power for controller to HSW/BDW. 

Please see
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=96214

Judging from the reports, Skylake uses the same display powerwell as
HSW/BDW.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre