Radeon driver : some comments about the source code

[loic.mahe@xxxxxxxxxxxxxx]

Last mail for today,

I checked ATI's fglrx source code and
the pci.ids database and I found some discrepancies with
XFree86's code (radeon_probe.c) for
the identification of the chipsets.
I would normally tell ATI is right,
but ...

PCI_CHIP_RV380_3152      
 "ATI Radeon Mobility X300 (M24) 3152 (PCIE)"
=> fglrx : ATI MOBILITY RADEON X300
(M22 3152), so RV370 chipset (instead of RV380)
=> pci.ids : ATI|M22 [Radeon Mobility
X300]

PCI_CHIP_RV360_AR      
 "ATI Radeon 9600XT AR (AGP)"
=> fglrx : ATI RADEON 9600 Series
(RV350/RV360 4152)
=> pci.ids : ATI|RV350 AR [Radeon
9600]

PCI_CHIP_RV370_5462      
 "ATI Radeon Mobility X600 SE (M24C) 5462 (PCIE)"
=> fglrx and pci.ids agree, but according
to the radeon(4x) man page, the M24 chipset is comparable to a
RV380 (instead of RV370)

PCI_CHIP_RV370_5B62      
 "ATI Radeon X600 (RV370) 5B62 (PCIE)"
=> pci.ids : ATI|RV380 [Radeon X600
(PCIE)]
=> according to the radeon(4x) man
page, the X600 has a RV380 chipset (instead of RV370)
=> but fglrx also says : RADEON X600
Series (RV370 5B62) whereas other X600 are given as RV380

PCI_CHIP_RS350_7835     "ATI
Radeon Mobility 9200 IGP 7835"
=> fglrx : MOBILITY RADEON 9000 IGP
       RL300MB (what is RL300 ???)

PCI_CHIP_RS300_5835     "ATI
Radeon Mobility 9100 IGP (U3) 5835"
=> fglrx says : RS300M chipset ???

Moreover, the man page and source code
should indicate clearly where the DRI is supported or not,
because RADEONPreInitDRI(), is not always
clean with that.

Loïc Mahé - Toulouse, France