| Summary: | CVT modeline calculator. | ||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | xorg | Reporter: | Luc Verhaegen <libv> | ||||||||||||
| Component: | * Other | Assignee: | Luc Verhaegen <libv> | ||||||||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||||||||
| Severity: | enhancement | ||||||||||||||
| Priority: | lowest | CC: | alexdeucher, cloos | ||||||||||||
| Version: | git | ||||||||||||||
| Hardware: | All | ||||||||||||||
| OS: | All | ||||||||||||||
| Whiteboard: | |||||||||||||||
| i915 platform: | i915 features: | ||||||||||||||
| Attachments: |
|
||||||||||||||
|
Description
Luc Verhaegen
2005-11-25 06:14:58 UTC
Created attachment 3897 [details] [review] Code, manpage and Makefile.am for cvt Links against dummylib. To be put in xserver/xorg/hw/xfree86/utils. Created attachment 3900 [details] [review] Add cvt to xserver/xorg/ build. Created attachment 4154 [details] [review] Code, manpage and Makefile.am for cvt. This fixes the incorrect xf86.h inclusion. The inclusion was external only and required an (possibly different) X tree installed. Now it includes "xf86.h" and ../../xfree86/common/xf86.h is -Ied. Created attachment 4165 [details] [review] Code, manpage and Makefile.am for cvt. - Removes reduced keyword and modeflag: this can be detected by HBlank length. - Allows only 60Hz modes for reduced blanking. Created attachment 4166 [details] [review] Tree side changes. This code adds what's needed for cvt to build to the configure.ac and the xfree86/utils/Makefile.am. This also adds a few warning fixes to xf86Config.c. And it adds the ReducedBlanking option to Monitor subsection, reduced blanking checking, etc. The gist from the xorg@list discussion, where only skk and benh were fielding questions and raising concerns, seems to have boiled down with the following conclusion: - we do need to check for reduced blanking support (done). - use an option (done). - use the edid digital bit, with a few exceptions (todo). I'm going to commit the cvt generator, with all the server side stuff in that patch, but keep this bug open until the edid side stuff has been resolved too. I will make this bug depend on #5386 until then. Comment on attachment 4165 [details] [review] Code, manpage and Makefile.am for cvt. Committed. Comment on attachment 4166 [details] [review] Tree side changes. Committed. Suggestion for next release: Now that this algorithm is part of the X source code, could we build it into the xorg.conf parser so people can manually specify mode data by providing resolution and refresh rate (you know, the way they have wanted to for the last 20 years :-). Something like: SensibleMode "1920x1080@57" ScreenWidth 1920 ScreenHeight 1080 RefreshRate 57 ReducedBlanking yes EndSensibleMode To anyone who thinks manual intervention is no longer required, I direct you to the host of Windows video setting tweakers on the net as proof that not even Windows (the main market for all video driver developers) can't get things to work automagically all the time. If that isn't enough of an incentive to make it simpler to manually intervene, you may feel my pain by reviewing my experiences: http://home.att.net/~Tom.Horsley/easy-linux.html cvt was made global very silently. I doubt that anyone uses it except my edid code. People just weren't told. Whatever. |
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.