Attachment 22620 Details for Bug 19972 – patch from F9 & U8.04 that makes it work

[patch] patch from F9 & U8.04 that makes it work

102-fedora-vesa-1.3.0-range-hack.patch (text/plain), 1.64 KB, created by Mario Limonciello on 2009-02-05 11:00:15 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Mario Limonciello

Created: 2009-02-05 11:00:15 UTC

Size: 1.64 KB

patch

obsolete

>--- 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");

Actions: View | Splinter Review

Attachments on bug 19972: 22619 | 22620 | 22622

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.