Re: [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]

 



On Wed, Mar 11, 2009 at 10:27:51AM -0400, Tom spot Callaway wrote:
>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

Not that this is your fault, but shouldn't this be something
like __ARCH_HAS_AGP?

josh

_______________________________________________
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