Alan Hourihane wrote:

Why does the mga_drv.c set DRIVER_REQUIRE_AGP when it doesn't 'require' it
if there's a PCI Matrox card ?

It does still require it. The PCI cards are not (yet) supported. The whole reason I added the call to drm_device_is_agp in that place and added the drm_driver::device_is_agp hook was so that the PCI G450 cards (which are AGP cards with a universal PCI-to-PCI bridge) could be detected and failed gracefully.

Couldn't we do....

   if (drm_device_is_agp(dev) || DRIVER_REQUIRE_AGP)
       dev->agp = drm_agp_init(dev);

and remove DRIVER_REQUIRE_AGP from mga_drv.c ?



-------------------------------------------------------
This SF.Net email is sponsored by Yahoo.
Introducing Yahoo! Search Developer Network - Create apps using Yahoo!
Search APIs Find out how you can build Yahoo! directly into your own
Applications - visit http://developer.yahoo.net/?fr=offad-ysdn-ostg-q22005
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to