[PATCH vmware] Build fix for ABI version 12
Jeremy Huddleston
jeremyhu at apple.com
Thu Nov 17 11:25:07 PST 2011
Hi Thomas,
I didn't realize you had restrictions on your trees. I just wanted to get your driver building again. My apologies, I won't do that again.
Sorry for any inconvenience this may have caused you.
On Nov 17, 2011, at 02:40, Thomas Hellstrom wrote:
> Jeremy,
>
> When submitting and commiting patches for the vmware driver please follow the guidelines outlined in
>
> http://wiki.x.org/wiki/vmware
>
> Not following this messes up our internal testing.
>
> Thanks,
> Thomas
>
>
> On 10/07/2011 10:22 PM, Thomas Hellstrom wrote:
>> On 10/07/2011 10:19 PM, Jeremy Huddleston wrote:
>>> This results in the vmware driver assuming there is only one PCI domain,
>>> which I think is true for everywhere this is expected to run anyway.
>>>
>>> Signed-off-by: Jeremy Huddleston<jeremyhu at apple.com>
>>> ---
>>> This gets vmware building with my recent bus changes, but I haven't tested it.
>>>
>>> I'm not sure what BUILD_FOR_420 is. Is that to disable that code when building for XFree86-4.2 and prior?
>>>
>>> src/vmware.c | 2 ++
>>> 1 files changed, 2 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/src/vmware.c b/src/vmware.c
>>> index 1be0990..55bde22 100644
>>> --- a/src/vmware.c
>>> +++ b/src/vmware.c
>>> @@ -612,9 +612,11 @@ VMWAREPreInit(ScrnInfoPtr pScrn, int flags)
>>> uint32 width = 0, height = 0;
>>> Bool defaultMode;
>>>
>>> +#if GET_ABI_MAJOR(ABI_VIDEODRV_VERSION)< 12
>>> #ifndef BUILD_FOR_420
>>> domainIOBase = pScrn->domainIOBase;
>>> #endif
>>> +#endif
>>>
>>> if (flags& PROBE_DETECT) {
>>> return FALSE;
>>
>> To be honest, I'm not sure myself. I'll look into it.
>> Thanks for the patch. It might take a while before I push it. Need to get latest Xserver building first to check.
>>
>> /Thomas
>>
>> _______________________________________________
>> xorg-devel at lists.x.org: X.Org development
>> Archives: http://lists.x.org/archives/xorg-devel
>> Info: http://lists.x.org/mailman/listinfo/xorg-devel
>
More information about the xorg-devel
mailing list