Bug 3718 - DRI & Xorg ATI headers can't agree on format of register defines
Summary: DRI & Xorg ATI headers can't agree on format of register defines
Alias: None
Product: Mesa
Classification: Unclassified
Component: Drivers/DRI/r200 (show other bugs)
Version: 5.0.2
Hardware: All All
: high normal
Assignee: Xorg Project Team
QA Contact:
Keywords: janitor
Depends on:
Reported: 2005-07-06 20:32 UTC by Alan Coopersmith
Modified: 2008-10-06 02:33 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Description Alan Coopersmith 2005-07-06 20:32:03 UTC
[Originally reported by Peter  Breitenlohner in attachment #2324 [details] [review] to bug #2901.
 I've split it out into a separate bug here so the maintainers of the ATI 
 code in Xorg & DRI can decide which syntax to agree on and who to change.]

The ATI register defines in the DRI & Xorg headers don't agree on whether or
not to use spaces for a number of registers, and this makes gcc cry.

For example, in extras/Mesa/src/mesa/drivers/dri/r200/r200_reg.h:
#define     R200_TXC_ARG_B_ZERO                (0<<5)
#define     R200_TXC_ARG_B_CURRENT_COLOR       (2<<5)
#define     R200_TXC_ARG_B_CURRENT_ALPHA       (3<<5)
#define     R200_TXC_ARG_B_DIFFUSE_COLOR       (4<<5)

While in programs/Xserver/hw/xfree86/drivers/ati/radeon_reg.h:
#       define R200_TXC_ARG_B_ZERO              (0 << 5)
#       define R200_TXC_ARG_B_CURRENT_COLOR     (2 << 5)
#       define R200_TXC_ARG_B_CURRENT_ALPHA     (3 << 5)
#       define R200_TXC_ARG_B_DIFFUSE_COLOR     (4 << 5)

Peter Breitenlohner counted 462 pairs of warnings from disagreements like this
in a build of Xorg 6.8.2.  Clearing those out to make the real problems more
visible would be good.
Comment 1 T. Hood 2005-09-26 08:37:45 UTC
Please sync extras/Mesa/src/mesa/drivers/dri/r200/r200_reg.h to
Comment 2 Michel Dänzer 2008-09-23 00:04:14 UTC
Is this still an issue? I don't think xf86-video-ati includes headers from Mesa drivers or vice versa...
Comment 3 Peter Breitenlohner 2008-10-06 02:03:55 UTC
Yes, this can certainly be closed.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.