[PATCH] int10/vbe: don't use xf86Screens. (ABI) (v2)
Adam Jackson
ajax at nwnk.net
Thu May 17 11:13:13 PDT 2012
On 5/16/12 5:16 AM, Dave Airlie wrote:
>>
>>> @@ -340,7 +341,7 @@ vbeDoEDID(vbeInfoPtr pVbe, pointer pDDCModule)
>>> if (!DDC_data)
>>> return NULL;
>>>
>>> - pMonitor = xf86InterpretEDID(pVbe->pInt10->scrnIndex, DDC_data);
>>> + pMonitor = xf86InterpretEDID(pVbe->pInt10->pScrn->scrnIndex,
>>> DDC_data);
>>
>>
>> The callee here wants index->ptr conversion too, doesn't it? I don't think
>> I see that in subsequent patches.
>
> That gets into an area I've been thinking about but mostly avoiding for now,
>
> That is pretty much a logging function, i.e. scrnIndex only goes into
> logging functions,
>
> Now I'm tempted to leave logging functions just passing indices, but
> I'm thinking its probably a bad idea long term, just not sure what it
> is short-term.
Point taken, hadn't thought about that. You do actually want _some_
integer there, otherwise how do you distinguish two nouveau instances.
I suppose as long as the screen arrays become things that it's
physically not possible to index into then it really doesn't matter what
scrnIndex is numerically, and it'd be safe to keep using as a unique ID.
R-b for the original patch.
- ajax
More information about the xorg-devel
mailing list