Re: [PATCH] drm/i915: remove extraneous VGA power domain put calls

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxxxxx>
> > 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@xxxxxxxxxxxxxxx>
> > 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@xxxxxxxx>
> > 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@xxxxxxxxx>
> 
> Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>

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
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx





[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux