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

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

 



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@xxxxxxxxxx>
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);
 }
 
_______________________________________________
Fedora-kernel-list mailing list
Fedora-kernel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-kernel-list

[Index of Archives]     [Fedora General Discussion]     [Older Fedora Users Archive]     [Fedora Advisory Board]     [Fedora Security]     [Fedora Devel Java]     [Fedora Legacy]     [Fedora Desktop]     [ATA RAID]     [Fedora Marketing]     [Fedora Mentors]     [Fedora Package Announce]     [Fedora Package Review]     [Fedora Music]     [Fedora Packaging]     [Centos]     [Fedora SELinux]     [Coolkey]     [Yum Users]     [Tux]     [Yosemite News]     [KDE Users]     [Fedora Art]     [Fedora Docs]     [USB]     [Asterisk PBX]

  Powered by Linux