[Intel-gfx] [PATCH] drm/i915: remove extraneous VGA power domain put calls

Daniel Vetter daniel at ffwll.ch
Tue May 6 21:31:18 CEST 2014


On Fri, Apr 25, 2014 at 07:14:42PM +0300, Ville Syrjälä wrote:
> On Fri, Apr 25, 2014 at 05:28:00PM +0300, Imre Deak wrote:
> > In recent dmesg logs reported for unrelated issues I noticed some power
> > domain WARNs caused by the following.
> > 
> > The workaround
> > 
> > commit ce352550327b394f3072a07c9cd9d27af9276f15
> > Author: Ville Syrjälä <ville.syrjala at linux.intel.com>
> > Date:   Fri Sep 20 10:14:23 2013 +0300
> > 
> >     drm/i915: Fix unclaimed register access due to delayed VGA memory disable
> > 
> > and following fixup of it
> > 
> > commit a14853206517b0c8102accbc77401805a0dbdb9e
> > Author: Ville Syrjälä <ville.syrjala at linux.intel.com>
> > Date:   Mon Sep 16 17:38:34 2013 +0300
> > 
> >     drm/i915: Move power well init earlier during driver load
> > 
> > was partially reverted by
> > 
> > commit 7f16e5c1416070dc590dd333a2d677700046a4ab
> > Merge: 9d1cb91 5e01dc7
> > Author: Daniel Vetter <daniel.vetter at ffwll.ch>
> > Date:   Mon Nov 4 16:28:47 2013 +0100
> > 
> >     Merge tag 'v3.12' into drm-intel-next
> > 
> > but kept the power domain put calls on the error path.
> > 
> > I think for now we can keep things as-is (not reintroduce the w/a) and just fix
> > the error path, since
> > - nobody complained seeing this issue
> > - according to Ville someone is reworking the VGA arbitration scheme at the
> >   moment and when that's ready we have to rethink this part anyway
> > 
> > So fix this by just removing the put calls from the error path as well.
> > 
> > Signed-off-by: Imre Deak <imre.deak at intel.com>
> 
> Reviewed-by: Ville Syrjälä <ville.syrjala at linux.intel.com>

Oops, forgotten to mention that this one is merged to dinq.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch



More information about the Intel-gfx mailing list