On Thu, 3 Mar 2005, Rune Petersen wrote:

With the latest changes to drm in r300_driver I was reminded of a few problems with R420 cards and the r300 driver.

1) the Xorg driver defaults to R200 for R420 cards.
RADEONDRIKernelInit() in radeon_dri.c

Could you test that your patch works with regular freedesktop.org DRM driver as far as 2d is concerned ? It should print a message in the log that DRI is enabled, though, of course, you would not be able to use R300 Mesa driver.



2) RADEONInitDispBandwidth() is disabled for r420 cards. I have no problem enabling this. are there a reason for this?

Well, the reason is that we don't have documentation for R4xx cards to update it - the latest ATI ever released to us is R3xx docs.


As you can see there are a lot of arrays tabulating "magic" values - these can only be obtained from designers of the hardware, or, perhaps, with some diagnostics equipment.

There are also special cases for R300 cards and we do not know what they should be for your card.

However, if you do need it enabled, we could change the code flow and put a comment warning that this does not come from documentation. This would also need to be discussed on X.org devel list before committing, maybe Hue or Michel or someone else will contribute actual code for R4xx cards.



3) I can enable more pipes for r420 cards, but am unable to test it on 420 cards with fewer pipes.
Do I just add this and wait for someone to complain?

A better way would be to read carefully specs of R4xx cards on ATI website to find out how many actual pipes are available :)




Regarding Xorg are there an easy way to make a diff without needing a duplicate tree? And where should I send the Xorg patches?

If you are working from X.org CVS (as anonymous) and you are modifying files inside the tree you can use the following command to obtain the patch:


cvs -z3 diff -u

E-mail the patch to me - the reason that R4xx cards are not enabled there is that I was not certain whether R3xx microcode would work for them.

                      best

                        Vladimir Dergachev



Rune Petersen



-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
--
_______________________________________________
Dri-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to