[Intel-gfx] [PATCH 11/11] drm/i915: Enable PSR by default.

Chris Wilson chris at chris-wilson.co.uk
Sat Jul 12 14:02:02 CEST 2014


On Sat, Jul 12, 2014 at 01:20:10PM +0200, Daniel Vetter wrote:
> On Sat, Jul 12, 2014 at 12:42 PM, Chris Wilson <chris at chris-wilson.co.uk> wrote:
> > On Sat, Jul 12, 2014 at 12:05:13PM +0200, Daniel Vetter wrote:
> >> On Fri, Jul 11, 2014 at 10:30:19AM -0700, Rodrigo Vivi wrote:
> >> > Panel Self Refresh is an eDP power saving feature specified by VESA's eDP v1.3,
> >> > that allows some panel componets to shutdown while you still see static images on
> >> > the screen. Besides being supported on the platform it must be supported by the
> >> > eDP panel itself.
> >> >
> >> > Now that we have the propper frontbuffer tracking support and correct locks on place
> >> > we can enabled this feature by default.
> >> >
> >> > Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
> >>
> >> Yay!
> >>
> >> Entire series pulled in, thanks a lot for the testing and review.
> >
> > Is there any chance we can have the output property to know when PSR is
> > available and active on the eDP?
> 
> Oh right, forgotten about this. I guess now that the hw stuff is there
> we can resurrect this. And with the universal planes support we could
> move the "prefer backbuffer rendering" hint to planes even where it
> imo belongs (since e.g. fbc is on the plane). Or maybe we don't care
> about that level of granularity and just have one per pipe.

But PSR is definitely an output property... Please-dont-touch-me is
indeed a plane property.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre



More information about the Intel-gfx mailing list