[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freedesktop-xorg
Subject:    Re: [PATCH] Add PCI IDs from ati website
From:       Roland Scheidegger <sroland () tungstengraphics ! com>
Date:       2007-02-27 12:46:07
Message-ID: 45E4280F.8040709 () tungstengraphics ! com
[Download RAW message or body]

Roland Scheidegger wrote:
> Christoph Brill wrote:
>> Hi list,
>>
>> attached are 4 files that update the PCI ID's of the xf86-video-ati
>> driver with values takes from the ATi website.
> I didn't look at it too closely, but some comments. First, it's a bit
> pointless to add all the X1xxx ids as they aren't supported anyway.
> Also, adding secondary ids is useless too. Lastly, if these are real ids
> for cards which actually exist, then they should be added to the
> relevant code too - for xorg that is radeon_driver.c, radeon_probe.c,
> atichip.c, for drm that would be drm_pciids.txt, and for mesa dri that
> would be radeon_chipset.h and radeon_screen.c - maybe I forgot some :-) ...
Forgot to mention, you could also use the .inf file from a driver for
some other OS to get the pci ids. The official ati pci id list contains
entries which are somewhat bogus, i.e. ids which don't really exist -
for instance firegl v3300 with rv410 AFAIK never made it to market
(firegl v3300 on the market use rv515), likewise things like x700xt
don't exist outside some samples. If it's not supported by the windows
driver, there's a good chance it can be safely ignored (unless it's an
entry for a mobility chip or some mac card or something). Though
ultimately I guess it doesn't really matter which list you use, they are
similar enough.

Roland

[prev in list] [next in list] [prev in thread] [next in thread]