[patch1/2] xf86modes: fix max clock unit

Eric Anholt eric at anholt.net
Fri Feb 1 17:14:39 PST 2008


On Mon, 2008-01-28 at 14:16 +0800, Hong Liu wrote:
> max clock from EDID data is in MHz, while we need KHz to validate modes.
> 
> Thanks,
> Hong
> 
> diff --git a/hw/xfree86/modes/xf86Crtc.c b/hw/xfree86/modes/xf86Crtc.c
> index 10db862..890d055 100644
> --- a/hw/xfree86/modes/xf86Crtc.c
> +++ b/hw/xfree86/modes/xf86Crtc.c
> @@ -1363,7 +1363,7 @@ xf86ProbeOutputModes (ScrnInfoPtr scrn, int maxX, int maxY)
>  			    sync_source = sync_edid;
>  		    }
>  		    if (ranges->max_clock > max_clock)
> -			max_clock = ranges->max_clock;
> +			max_clock = ranges->max_clock * 1000;
>  		}
>  	    }
>  	}

What about the line just above the line you changed?

-- 
Eric Anholt                             anholt at FreeBSD.org
eric at anholt.net                         eric.anholt at intel.com

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 187 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg/attachments/20080201/4ead5643/attachment.pgp>


More information about the xorg mailing list