[patch] Fix refresh calculation for SavageMatchBiosMode

Alex Deucher alexdeucher at gmail.com
Tue Oct 19 17:15:33 PDT 2004

On Wed, 20 Oct 2004 00:28:08 +0200, Felix Kühling <fxkuehl at gmx.de> wrote:
> I found a problem which made the Savage driver choose BIOS modes with
> frequencies that are way outside the refresh range specified in
> xorg.conf. This happened when X tried to set a double scanned mode (in
> my case) and the Savage driver computed the wrong vertical refresh value
> based on Clock, HTotal and VTotal. Some code in common/xf86Mode.c also
> considers the DoubleScan and Interlace flags. Adding similar code to the
> Savage driver makes it choose sane BIOS modes again. A patch is
> attached. Line numbers are off due to other changes I'm working on ATM
> but it should apply cleanly otherwise.

Looks good Felix.  I'll test it out tonight.  feel free to apply it.


> Regards,
>   Felix
> --
> | Felix Kühling <fxkuehl at gmx.de>                     http://fxk.de.vu |
> | PGP Fingerprint: 6A3C 9566 5B30 DDED 73C3  B152 151C 5CC1 D888 E595 |

More information about the xorg mailing list