Re: Patch for review and testing

quoted

Then I will remove it.

Not yet, not until the new version is in. It does work someway with the
current code. Let me deal with those OF things please.

Sorry. I will leave this alone then. I posted it just case there where 
conflicts with peoples work here. I will wait to send that patch. 
 

Actually, that may not be a good approach neither... You probably want
to check that you are indeed dealing with the default VGA device so an
additional card don't get an unrelated EDID, no ?

Also, other archs may want to implement this function too. Keep the
struct device as an argument, check for bus_type before casting to PCI,
and we should probably, in the x86 PCI code, "remember" the pci_dev of
the default VGA (if not done already) and compare it on calls to this
function. (To be completely clean, I also need to know if I'm the
primary VGA in radeonfb and aty128fb).

I knew this was going to be more complex than I written.

What about a cmdline option then ? it's too early during boot to check
for it ? 

Yes. The code is run in real mode before linux is started. 

vendors will build kernels with or without the CONFIG_ option,
and people won't change it, so I'm afraid it will be useless... Is the
BIOS call standard ? 

Yes it is standard. 

There may be a way to workaround the hang, no ? Or
it's one of those calls that Windows never uses and are broken in half
of the BIOSes around ?

I don't know of any work arounds. Its a small precentage but I like to 
make life easier for people. 

P.S.
   Have any idea to know which card is the primary one on intel. I have 
been trying to figure that one out for quite some time.



-------------------------------------------------------
The SF.Net email is sponsored by EclipseCon 2004
Premiere Conference on Open Tools Development and Integration
See the breadth of Eclipse activity. February 3-5 in Anaheim, CA.
http://www.eclipsecon.org/osdn