[PATCH xf86-video-chips] Build fixes for ABI_VIDEODRV_VERSION 12

Jeremy Huddleston jeremyhu at apple.com
Sat Sep 24 04:13:14 PDT 2011


On Sep 24, 2011, at 03:11, Mark Kettenis wrote:

>> From: Jeremy Huddleston <jeremyhu at apple.com>
>> Date: Sat, 24 Sep 2011 00:03:45 -0700
>> 
>> 1) Include <pciaccess.h> for IOADDRESS
> 
> How does this help you? Nothing in libpciaccess defines IOADDRESS,
> and adding that typedef to <pciaccess.h> seems wrong to me.  See my
> previous mail for an easier solution.

Weird, for some reason I thought IOADDRESS came from pciaccess.  You're right.  That is wrong.

> 
>> 2) Use CARD32 instead of PCITAG
> 
> See comment inline below.


> 
>> src/ct_driver.h |    5 +++++
>> 1 files changed, 5 insertions(+), 0 deletions(-)
>> 
>> diff --git a/src/ct_driver.h b/src/ct_driver.h
>> index 10a24d4..06c8aa8 100644
>> --- a/src/ct_driver.h
>> +++ b/src/ct_driver.h
>> @@ -271,7 +272,11 @@ typedef struct {
>> 
>> typedef struct _CHIPSRec {
>>     pciVideoPtr		PciInfo;
>> +#if GET_ABI_MAJOR(ABI_VIDEODRV_VERSION) < 12
>>     PCITAG		PciTag;
>> +#else
>> +    CARD32		PciTag;
>> +#endif
> 
> The PciTag member is only used in the !XSERVER_LIBPCIACCESS case, so a
> better slution would be:
> 
> typedef struct _CHIPSRec {
>     pciVideoPtr		PciInfo;
> +#ifndef XSERVER_LIBPCIACCESS
>     PCITAG		PciTag;
> +#endif
>     int			Chipset;
>     EntityInfoPtr       pEnt;
>     IOADDRESS		PIOBase;
> 


That doesn't help us.  PCITAG is gone, and CARD32 is being used instead.  XSERVER_LIBPCIACCESS is still defined.  Perhaps *both* changes are necessary.




More information about the xorg-devel mailing list