[xorg-bugzilla-noise] [Bug 803] New: DRI merge broke Radeon driver on non-DRI platforms

[bugzilla-daemon at pdx.freedesktop.org]

Please do not reply to this email: if you want to comment on the bug, go to     
the URL shown below and enter your comments there.  
  
http://freedesktop.org/bugzilla/show_bug.cgi?id=803   
   
           Summary: DRI merge broke Radeon driver on non-DRI platforms
           Product: xorg
           Version: CVS_head
          Platform: PC
        OS/Version: Solaris
            Status: NEW
          Severity: major
          Priority: P2
         Component: Driver/Radeon
        AssignedTo: xorg-bugzilla-noise at freedesktop.org
        ReportedBy: alan.coopersmith at sun.com
                CC: eta at lclark.edu


radeon_driver.c can no longer build on non-DRI platforms:
"radeon_driver.c", line 2047: undefined struct/union member: fbLocation
"radeon_driver.c", line 2059: improper member use: fbLocation
"radeon_driver.c", line 2061: improper member use: fbLocation
"radeon_driver.c", line 2062: improper member use: fbLocation

In radeon.h, fbLocation is only defined #ifdef XF86DRI, but there are no 
#ifdefs around the new code in radeon_driver.c to keep it from being used
in those cases.

The simple fix would seem to be adding #ifdefs around those lines, but I don't
know if that would break the driver.   
   
--    
Configure bugmail: http://freedesktop.org/bugzilla/userprefs.cgi?tab=email   
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.