[Intel-gfx] [PATCH]avoid duplicate mode set in lvds

Zhenyu Wang zhenyu.z.wang at intel.com
Mon Aug 18 05:03:54 CEST 2008


On 2008.08.15 14:10:24 +0800, Shaohua Li wrote:
> xf86SetDesiredModes() already sets lvds to full mode. later when
> xf86CrtcScreenInit() initialized randr12, i830_lvds_set_property will
> recall xf86CrtcSetMode and set mode to full. This patch is to remove the
> duplication. In my test, this can save about 0.2 - 0.4s x startup time.
> ---
>  src/i830_lvds.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> Index: intel-video/src/i830_lvds.c
> ===================================================================
> --- intel-video.orig/src/i830_lvds.c
> +++ intel-video/src/i830_lvds.c
> @@ -1122,6 +1122,8 @@ i830_lvds_set_property(xf86OutputPtr out
>  	if (ret < 0)
>  	    return FALSE;
>  
> +	if (dev_priv->fitting_mode == ret)
> +		return TRUE;
>  	dev_priv->fitting_mode = ret;
>  
>  	if (output->crtc) {
> 

Pushed, thanks.

-- 
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20080818/7df5b482/attachment.sig>


More information about the Intel-gfx mailing list