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

Shaohua Li shaohua.li at intel.com
Fri Aug 15 08:10:24 CEST 2008


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) {





More information about the Intel-gfx mailing list