On Thu, Jan 26, 2012 at 10:32 PM, Daniel Stone <[email protected]> wrote: > Hi, > > On 27 January 2012 09:28, Aaron Plattner <[email protected]> wrote: >> On 01/26/2012 01:46 PM, Daniel Stone wrote: >>> Can you not drop the ScreenPtr argument and just say that the function >>> relies on ScrnInfoPtr::pScreen being valid? >> >> It would presumably be typically called from ScreenInit, where >> pScrn->pScreen is NULL. >> >> If you want to have the server wire that up earlier, that'd be another >> matter. > > Rather. A few drivers (at least radeon, nouveau, and vmware, as well > as the one I just wrote) wire it up during ScreenInit, which I'd > thought was more widespread. I don't see any downside to doing it in > the server before ScreenInit though.
Yeah thats not the shed I was painting right now. So we can either change it to passing pScreen, and doing the xf86Screens[pScreen->myNum] dance, or we can leave as it is. Dave. > > Cheers, > Daniel _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
