[patch1/2] xf86modes: fix max clock unit

Jesse Barnes jbarnes at virtuousgeek.org
Tue Feb 5 13:04:24 PST 2008


On Friday, February 01, 2008 5:44 pm Hong Liu wrote:
> max_clock is 0 at that time...
> Yeah, unifying the unit seems to make more sense.
>
> Here is the 2nd patch.
>
> diff --git a/hw/xfree86/modes/xf86Crtc.c b/hw/xfree86/modes/xf86Crtc.c
> index 10db862..4ecf4b3 100644
> --- a/hw/xfree86/modes/xf86Crtc.c
> +++ b/hw/xfree86/modes/xf86Crtc.c
> @@ -1362,8 +1362,8 @@ xf86ProbeOutputModes (ScrnInfoPtr scrn, int maxX, int
> maxY) if (sync_source == sync_default)
>  			    sync_source = sync_edid;
>  		    }
> -		    if (ranges->max_clock > max_clock)
> -			max_clock = ranges->max_clock;
> +		    if (ranges->max_clock * 1000 > max_clock)
> +			max_clock = ranges->max_clock * 1000;
>  		}
>  	    }
>  	}

Applied and pushed, thanks.

This one probably belongs in the 1.4 branch as well?

Jesse




More information about the xorg mailing list