Re: [PATCH 11/11] drm/i915: Enable PSR by default.

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

 



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@xxxxxxxxxxxxxxxxxx> 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@xxxxxxxxx>
> >>
> >> 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
_______________________________________________
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