On Mon, Jun 16, 2014 at 07:51:34PM +0200, Daniel Vetter wrote: > +static void intel_mark_fb_busy(struct drm_device *dev, > + unsigned frontbuffer_bits, > + struct intel_engine_cs *ring) > { > - struct drm_device *dev = obj->base.dev; > enum pipe pipe; > > - intel_edp_psr_exit(dev); > - > if (!i915.powersave) > return; I think this wants to be moved from here down to intel_increase_pllclock. It's a little dangerous here as we use it inconsistently, and we certainly want to keep tracking fb bits for other reasons. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx