Bug 16085 - nv driver should have option to overide EDID panel size
Summary: nv driver should have option to overide EDID panel size
Status: RESOLVED DUPLICATE of bug 3654
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/nVidia (open) (show other bugs)
Version: 7.3 (2007.09)
Hardware: Other All
: medium normal
Assignee: Aaron Plattner
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-05-24 08:53 UTC by Markus Koßmann
Modified: 2008-05-26 11:59 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
xorg.log showing the problem (53.25 KB, text/plain)
2008-05-24 08:53 UTC, Markus Koßmann
no flags Details

Description Markus Koßmann 2008-05-24 08:53:16 UTC
Created attachment 16719 [details]
xorg.log showing the problem 

My DFP provides wrong EDID data :-(.This includes wrong panel dimension
data. The nv driver refuses any modeline which is using pixelcounts > paneldimension. So I can't run that DFP with its
native resolution (1600x1200) when using nv. The propriatary nvidia driver doesn't do that check , but there are randon system freezes without any log entries when running it with kernel 2.6.25 (on SUSE 11.0 beta). So it will also be problematic for use with SUSE 11.0.
Comment 1 Aaron Plattner 2008-05-26 01:22:23 UTC
The EDID looks okay to me.  The problem here is that the BIOS configured the card's scaler for 1280x1024 and the driver is not able to reconfigure it on GPUs prior to the GeForce 8 family.

*** This bug has been marked as a duplicate of bug 3654 ***
Comment 2 Markus Koßmann 2008-05-26 09:38:55 UTC
Then you should fix the messages.They are badly midleading.  The xorg.log files says: 'Not using mode "1600x1200" (exceeds panel dimensions)'. And not something like "cards BIOS doesn't support that mode", which would be right in suxh a case. 
Comment 3 Aaron Plattner 2008-05-26 11:59:20 UTC
You're right, I guess it is confusing.  It used to say

  Mode "foo" is larger than BIOS programmed panel size of W x H.  Removing.

but got changed to the current behavior in this commit:


commit 98372d0f94a9d658a7257d2199ef37a2449b9961
Author: Adam Jackson <ajax@benzedrine.nwnk.net>
Date:   Fri Aug 11 16:02:28 2006 -0400

    Replace panel message + MODE_BAD with MODE_PANEL.

:100644 100644 f094e6b... cb759be... M  src/nv_driver.c


I'll change the message where it prints the BIOS-configured panel size to make it clearer that that's what it's going to use for mode validation.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.