On Tue, Jun 17, 2014 at 09:23:48AM +0200, Daniel Vetter wrote: > On Tue, Jun 17, 2014 at 07:43:53AM +0100, Chris Wilson wrote: > > On Mon, Jun 16, 2014 at 05:03:02PM -0700, Rodrigo Vivi wrote: > > > On Mon, Jun 16, 2014 at 10:51 AM, Daniel Vetter > > > <[1]daniel.vetter@xxxxxxxx> wrote: > > > > > > static void intel_edp_psr_enable_sink(struct intel_dp *intel_dp) > > > @@ -1911,9 +1906,6 @@ void intel_edp_psr_exit(struct drm_device *dev, > > > bool schedule_back) > > > if (!HAS_PSR(dev)) > > > return; > > > > Whilst you are here, this should just be a if (dev_priv->psr.enabled) > > check. > > I have fixup for psr_invalidate/psr_flush, as this ends up being called to > do exactly that. It blew up ;-) So another bisect failure in the series. I'll fix this up. -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