Attachment 15659 Details for Bug 15313 – Panel fitting fixes

[patch] Panel fitting fixes

i830-855-pfit-fixes.patch (text/plain), 1.86 KB, created by Jesse Barnes on 2008-04-03 11:33:55 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jesse Barnes

Created: 2008-04-03 11:33:55 UTC

Size: 1.86 KB

patch

obsolete

>diff --git a/src/i830_lvds.c b/src/i830_lvds.c
>index 33b4f26..9f3207e 100644
>--- a/src/i830_lvds.c
>+++ b/src/i830_lvds.c
>@@ -542,23 +542,22 @@ i830_lvds_mode_fixup(xf86OutputPtr output, DisplayModePtr mode,
>     adjusted_mode->Clock = dev_priv->panel_fixed_mode->Clock;
>     xf86SetModeCrtc(adjusted_mode, INTERLACE_HALVE_V);
> 
>+    /* Make sure pre-965s set dither correctly */
>+    if (!IS_I965G(pI830) && dev_priv->panel_wants_dither)
>+	pfit_control |= PANEL_8TO6_DITHER_ENABLE;
>+
>     /* Native modes don't need fitting */
>     if (adjusted_mode->HDisplay == mode->HDisplay &&
> 	adjusted_mode->VDisplay == mode->VDisplay) {
>-	pfit_control = 0;
> 	pfit_pgm_ratios = 0;
> 	border = 0;
> 	goto out;
>     }
> 
>-    /* Basic panel fitting options */
>-    if (!IS_I965G(pI830)) {
>-	if (dev_priv->panel_wants_dither)
>-	    pfit_control |= PANEL_8TO6_DITHER_ENABLE;
>-    } else {
>+    /* 965+ wants fuzzy fitting */
>+    if (IS_I965G(pI830))
> 	pfit_control |= (intel_crtc->pipe << PFIT_PIPE_SHIFT) |
> 	    PFIT_FILTER_FUZZY;
>-    }
> 
>     /*
>      * Deal with panel fitting options.  Figure out how to stretch the image
>@@ -742,7 +741,6 @@ i830_lvds_prepare(xf86OutputPtr output)
> 
>     lvds = INREG(LVDS);
> 
>-    i830_lvds_dpms(output, DPMSModeOff);
>     /*
>      * ->prepare will be called after the CRTC is off but before
>      * we set the mode, so program the PFIT regs here.
>@@ -751,6 +749,8 @@ i830_lvds_prepare(xf86OutputPtr output)
> 	OUTREG(LVDS, lvds | LVDS_BORDER_ENABLE);
>     else
> 	OUTREG(LVDS, lvds & (~LVDS_BORDER_ENABLE));
>+
>+    i830_lvds_dpms(output, DPMSModeOff);
> }
> 
> static void
>@@ -767,8 +767,6 @@ i830_lvds_mode_set(xf86OutputPtr output, DisplayModePtr mode,
>      */
>     OUTREG(PFIT_PGM_RATIOS, dev_priv->pfit_pgm_ratios);
>     OUTREG(PFIT_CONTROL, dev_priv->pfit_control);
>-    /* It's harmless to turn on the LVDS if it's already on */
>-    i830_lvds_dpms(output, DPMSModeOn);
> }
> 
> /**

Actions: View | Splinter Review

Attachments on bug 15313: 15615 | 15616 | 15621 | 15622 | 15631 | 15637 | 15651 | 15655 | 15657 | 15658 | 15659 | 15662 | 15665 | 15681

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.