Bug#572311: No display output from Radeon RV610 on Alpha

[Alex Deucher]

On Sun, Mar 7, 2010 at 3:47 AM, Michael Cree <mcree at orcon.net.nz> wrote:
> On 07/03/10 03:34, Alex Deucher wrote:
>>
>> On Fri, Mar 5, 2010 at 8:10 PM, Michael Cree<mcree at 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