[PATCH v2 3/8] randr: Add ability to turn PRIME sync off

Adam Jackson ajax at nwnk.net
Tue Jan 26 08:41:55 PST 2016


On Mon, 2016-01-25 at 19:17 -0800, Alex Goins wrote:
> Hey Adam,
> 
> I recently got around to implementing this when putting the v3 patch set
> together (with some changes of my own,) and I'm finding that it harms the user
> experience significantly.
> 
> It's sort of annoying that you currently have to select from either 0 or 
> instead of 0 or 1, but more importantly, xrandr uses hex to print provider XIDs
> and decimal to print/set output properties.

I wasn't intending to insist that you pick the XID, quite the reverse,
I was happy to have 1 mean "pick one". Nothing <32 is a valid XID
anyway so it's never ambiguous.

> Would you object to simply including a /* TODO */ for making
> RR(Init/Fini)PrimeSyncProps correctly handle different sources for different
> outputs, if and when that is implemented?

That sounds fine to me.  Sorry to have led you down a blind alley.

- ajax


More information about the xorg-devel mailing list