Bug 871 - [PATCH] Add "ForceLegacyCRT" option to Radeon driver for older VGA monitors
Summary: [PATCH] Add "ForceLegacyCRT" option to Radeon driver for older VGA monitors
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Driver/Radeon (show other bugs)
Version: git
Hardware: x86 (IA32) Linux (All)
: high normal
Assignee: Xorg Project Team
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 351
  Show dependency treegraph
 
Reported: 2004-07-14 23:15 UTC by Mike A. Harris
Modified: 2004-08-08 19:05 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
xorg-x11-CVS-ati-radeon-forcelegacycrt.patch (3.02 KB, patch)
2004-07-15 00:43 UTC, Mike A. Harris
no flags Details | Splinter Review

Description Mike A. Harris 2004-07-14 23:15:50 UTC
The Radeon driver does not work with older VGA paperwhite displays and
certain other older fixed frequency displays.  The driver simply can't
detect the monitor and so it bails.

Here is the original Red Hat bug report for this issue:
    http://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=69978

I've implemented a patch to add a new config file option named "ForceLegacyCRT"
which overrides the autodetection logic and allows the user to use an older
CRT that can't be autodetected.  This should be safe for single-head setups,
or multihead setups where only one monitor is connected to the Radeon
hardware, and the other head(s) are on other video cards.  In a dualhead
setup using both heads on one Radeon card however, if this option is
enabled and a legacy CRT is plugged in, it is possible that one of the
monitors might be damaged due to receiving out of range signals.  That isn't
a bug however, it's a limitation in the old legacy monitor.

Please apply to CVS for next release
Comment 1 Mike A. Harris 2004-07-15 00:43:08 UTC
Created attachment 482 [details] [review]
xorg-x11-CVS-ati-radeon-forcelegacycrt.patch
Comment 2 Mike A. Harris 2004-08-06 23:31:03 UTC
This patch applied clean to CVS when it was submitted.  The Radeon driver has
subsequently been modified and the patch no longer applies.  If someone looks
at this patch please, and tells me if they think it is clean enough to go in
or not, I'd appreciate it.  If it is acceptable, I will port it forward so
it can be committed to CVS.  If it is not, please let me know why, so I can
either clean it up to be acceptable, or drop it on the floor.

Thanks in advance.
Comment 3 Alex Deucher 2004-08-07 13:24:01 UTC
Couldn't you force it to CRT using:
option "monitorlayout" "crt"
rather than adding a new option.  you should also work on dualhead cards if you
hard code the monitor type for both crtc1 and crtc2.
Comment 4 Kevin E. Martin 2004-08-09 12:05:31 UTC
This patch is no longer needed as explained in comment #3.
Closing.


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.