On Sun, Mar 7, 2010 at 3:47 AM, Michael Cree <mc...@orcon.net.nz> wrote: > On 07/03/10 03:34, Alex Deucher wrote: >> >> On Fri, Mar 5, 2010 at 8:10 PM, Michael Cree<mc...@orcon.net.nz> wrote: >>> >>> If I compile the ati/radeon video driver from fdo git master (commit >>> 4975658f05) with default CFLAGS (i.e. no byte-word extension), except for >>> the module src/AtomBios/CD_Operations.c which I compile with the -mbwx >>> compiler option, then I get a working video driver. >>> >>> Looks like the problem is in src/AtomBios/CD_Operations.c. >> >> You probably need something like this drm patch ported to the ddx: >> http://marc.info/?l=dri-devel&m=126611019424637&w=2 > > Thanks, that hint was helpful. I have drummed up a patch (attached) that > replaces some use of the UINT16LE_TO_CPU(), etc., macros with generic > interfaces from the Xserver's compiler.h header file. Now works correctly > on RV610 video card on an Alpha XP1000. Have also verified that the driver > still works on an RV710 card on AMD64 architecture. > > The patch applies cleanly against the 6.12.5 branch and also upstream git > master. > > Alex: may I presume that you will handle getting it upstream for review and > hopefully acceptance into the fdo git master.
I'll take a look at it. My only concern would be making sure these changes don't break big endian which is the reason the macros were added in the first place. Alex -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org