On Sat, Jul 12, 2014 at 8:55 PM, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote: > On Sat, Jul 12, 2014 at 08:15:42PM +0200, Daniel Vetter wrote: >> On Sat, Jul 12, 2014 at 7:51 PM, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote: >> > On Sat, Jul 12, 2014 at 07:14:33PM +0200, Daniel Vetter wrote: >> >> On Sat, Jul 12, 2014 at 2:02 PM, Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> wrote: >> >> > But PSR is definitely an output property... Please-dont-touch-me is >> >> > indeed a plane property. >> >> >> >> Well I guess we also want this for fbc, and fbc isn't an output >> >> property. I guess in the end it doesn't really matter that much as >> >> long as it's there, but traditional userspace exposes all output >> >> properties to userspace (on X), hence why I think we should hide it a >> >> bit ;-) >> > >> > Hence why I want PSR as an output property. I want to see the status >> > exposed in xrandr. >> >> Hm, why that? I've thought we only want this to give hints to the >> compositor/ddx? > > An interesting factoid to present to the user. Also makes it easy for me > to check everything works in X. Hm not sure we want users to know this stuff. Next they ask to adjust it and then we have the same mess as with kernel options ;-) But I don't care strongly enough really either way. In-kernel I still think we want to keep this on the crtc so that fbc and psr work the same way. The connector property would then just chase the connected crtc to read out the property. -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
