Red Hat Bugzilla – Bug 69259
adding support for NVIDIA GeForce2 Go
Last modified: 2007-04-18 12:44:18 EDT
From Bugzilla Helper: User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.1a) Gecko/20020610 Description of problem: XFree86 as of Red Hat Linux 7.3 did not support NVIDIA GeForce2 Go cards. Code to support it, as well as many other cards, was installed in the XFree86 CVS tree a few months ago. The attached patch, against XFree86-4.2.0-8, merges those changes, so that the `nv' driver works on my Dell Inspiron 8000.
Created attachment 66024 [details] Patch that merges code for the nv driver from XFree86 CVS mainline as of today
I've investigated the above patch, and it is unsuitable for application to our tree. It contains way too much needless whitespace, indenting, restructuring changes which make it quite unreadable. I generated my own diff against CVS with whitespace/gratuitous changes omitted, and investgated the resultant diff. The driver changes add support for a lot of hardware, and change numerous other things as well. The changes are too drastic for inclusion in our tree at this stage, especially considering the driver freeze was approx 10 days ago. Any driver changes of large magnitude need to get into the first public beta. Smaller changes definitely need to get into the second public beta at least. I'm closing this WONTFIX rather than deferring it, as it'll all be in XFree86 4.3.0 within 6 months anyway. I'll be making CVS RPM packages of X probably sometime in the next 2 months for testing.
FWIW, I agree entirely with Mark. Yesterday, I had a very unfortunate experience with the very XFree86 binaries I built, when I had to give a presentation to a hundred people and I couldn't get my laptop to use the external display properly: the screen would be full of garbage and wouldn't come back, so I had to reboot. Before starting the presentation, fortunately. I switched to the VESA driver and it worked fine, so it's probably safer not to put this code in for now.
Oops, s/Mark/Mike/. Mark is the NVidia guy who wrote the patch I merged. Oops :-)