On Sat, Sep 04, 2004 at 11:30:40AM +0100, Dave Airlie wrote: > > it looks you can nuke most of those. > > > -#if __REALLY_HAVE_AGP > > > +#if __OS_HAS_AGP > > > drm_agp_head_t *agp; /**< AGP data */ > > > #endif > > > > for example does this extra pointer really hurt in the *really* unlikely > > case you don't have AGP ? > > actually it's not just the pointer it is the structure drm_agp_head_t I > think it needs some agp includes to work .. ... or a dummy drm_agp_head_t typedef.