On Tue, 2018-01-23 at 08:42 +1100, Keith Packard wrote: > Adam Jackson <[email protected]> writes: > > > - if (!xf86Info.disableRandR) > > + if (!rrGetScrPriv(xf86Screens[i]->pScreen)) > > xf86RandRInit(screenInfo.screens[scr_index]); > > A comment here would help me understand this change; I assume we won't > generally hit this path, right?
Yeah fair, this is a bit idiomatic. What this is saying is, if the driver hasn't already initialized randr support for this screen - meaning, if there's not already a randr screen private - then fill in the randr hooks for the screen with the 1.1 compat code. 1.2+ drivers will have called xf86CrtcScreenInit() in their ScreenInit hook before this point. - ajax _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel
