Bug#572311: No display output from Radeon RV610 on Alpha
On Sun, Mar 7, 2010 at 3:47 AM, Michael Cree <mcree@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@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
Reply to: