[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

[PATCH] Don't try to use agp symbols if we don't have AGP (*cough*sparc*cough*)



The radeon_buffer.c addition that gets pulled into the DRM driver as
part of Kernel Mode Setting (even if you disable KMS for the Radeon
driver) was assuming that it was safe to use symbols from the AGP code,
but that isn't safe on platforms where __OS_HAS_AGP is 0. Such as
sparc64. :)

This patch gets the code building again. It probably should go into the
actual kms patch, but if you would prefer this go as a separate patch, I
can apply it. With this, we can get rawhide building for sparc64, so I'm
eager to see it applied in one way or another.

Signed-off-by: Tom "spot" Callaway <tcallawa redhat com>
diff -up linux-2.6.28.sparc64/drivers/gpu/drm/radeon/radeon_buffer.c.BAD linux-2.6.28.sparc64/drivers/gpu/drm/radeon/radeon_buffer.c
--- linux-2.6.28.sparc64/drivers/gpu/drm/radeon/radeon_buffer.c.BAD	2009-03-11 09:11:01.000000000 -0500
+++ linux-2.6.28.sparc64/drivers/gpu/drm/radeon/radeon_buffer.c	2009-03-11 09:11:27.000000000 -0500
@@ -37,9 +37,11 @@ struct drm_ttm_backend *radeon_create_tt
 {
 	drm_radeon_private_t *dev_priv = dev->dev_private;
 
+#if __OS_HAS_AGP
 	if (dev_priv->flags & RADEON_IS_AGP)
 		return drm_agp_init_ttm(dev);
 	else
+#endif
 		return ati_pcigart_init_ttm(dev, &dev_priv->gart_info, radeon_gart_flush);
 }
 

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]