Blob Blame History Raw
--- xf86-video-vesa-1.3.0/src/vesa.c.jx	2007-01-30 10:31:32.000000000 -0500
+++ xf86-video-vesa-1.3.0/src/vesa.c	2007-01-30 10:56:15.000000000 -0500
@@ -638,8 +638,40 @@
 	xf86UnloadSubModule(pDDCModule);
     }
 
-    if ((pScrn->monitor->DDC = pVesa->monitor) != NULL)
-	xf86SetDDCproperties(pScrn, pVesa->monitor);
+    /* UNCLEAN */
+    do {
+	Bool had_cfg_ranges, have_ddc_ranges;
+
+	had_cfg_ranges = (pScrn->monitor->nHsync || pScrn->monitor->nVrefresh);
+	if ((pScrn->monitor->DDC = pVesa->monitor) != NULL)
+	    xf86SetDDCproperties(pScrn, pVesa->monitor);
+	have_ddc_ranges = (pScrn->monitor->nHsync || pScrn->monitor->nVrefresh);
+
+	if (have_ddc_ranges && !had_cfg_ranges) {
+	    int i;
+	    for (i = 0; i < DET_TIMINGS; i++) {
+		if (pVesa->monitor->det_mon[i].type == DS_RANGES)
+		    break;
+	    }
+	    if (i < DET_TIMINGS)
+		break;
+
+	    /*
+	     * DDC added ranges, but they're inferred from the modes, not
+	     * copied from a range section.  So in the worst case, where it's
+	     * not a range but a single frequency, stretch things out to be
+	     * a range.  Panels with only one mode listed in DDC will
+	     * otherwise probably not match any VESA modes.  Hooray for
+	     * conformance.
+	     */
+	    if (pScrn->monitor->hsync[0].lo == pScrn->monitor->hsync[0].hi
+		 && pScrn->monitor->hsync[0].lo > 31.5)
+		pScrn->monitor->hsync[0].lo = 31.5;
+	    if (pScrn->monitor->vrefresh[0].lo == pScrn->monitor->vrefresh[0].hi
+		 && pScrn->monitor->vrefresh[0].lo > 50.0)
+		pScrn->monitor->vrefresh[0].lo = 50;
+	}
+    } while (0);
 
     xf86DrvMsgVerb(pScrn->scrnIndex, X_INFO, DEBUG_VERB,
 			"Searching for matching VESA mode(s):\n");