<div dir="ltr">Any comments?<div><br></div><div>Without this, plugging one of the older Chromebook models into a Dell U3011 monitor produces a garbled display at the default 2048x1280 resolution.</div><div><br></div><div>The original threshold was apparently fairly arbitrary:</div>
<div><br></div><div><a href="http://cgit.freedesktop.org/~anholt/xf86-video-intel/commit/?id=8fcf9a81179ee8577ddab5e904c58fbfd14cf59c">http://cgit.freedesktop.org/~anholt/xf86-video-intel/commit/?id=8fcf9a81179ee8577ddab5e904c58fbfd14cf59c</a>.</div>
<div><br></div><div style>Stuart</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, May 20, 2013 at 11:15 AM, Stuart Abercrombie <span dir="ltr"><<a href="mailto:sabercrombie@chromium.org" target="_blank">sabercrombie@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">90% of core speed (=180MHz dot clock) is too high for 2048x1280 to get<br>
pixel doubling on Pineview, which it needs to avoid underruns, so<br>
lower this to 85%.<br>
<br>
Signed-off-by: Stuart Abercrombie <<a href="mailto:sabercrombie@chromium.org">sabercrombie@chromium.org</a>><br>
---<br>
 drivers/gpu/drm/i915/intel_display.c | 4 ++--<br>
 1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c<br>
index efe8299..9c924e9 100644<br>
--- a/drivers/gpu/drm/i915/intel_display.c<br>
+++ b/drivers/gpu/drm/i915/intel_display.c<br>
@@ -4564,14 +4564,14 @@ static void i9xx_set_pipeconf(struct intel_crtc *intel_crtc)<br>
        pipeconf = I915_READ(PIPECONF(intel_crtc->pipe));<br>
<br>
        if (intel_crtc->pipe == 0 && INTEL_INFO(dev)->gen < 4) {<br>
-               /* Enable pixel doubling when the dot clock is > 90% of the (display)<br>
+               /* Enable pixel doubling when the dot clock is > 85% of the (display)<br>
                 * core speed.<br>
                 *<br>
                 * XXX: No double-wide on 915GM pipe B. Is that the only reason for the<br>
                 * pipe == 0 check?<br>
                 */<br>
                if (intel_crtc->config.requested_mode.clock ><br>
-                   dev_priv->display.get_display_clock_speed(dev) * 9 / 10)<br>
+                   dev_priv->display.get_display_clock_speed(dev) * 17 / 20)<br>
                        pipeconf |= PIPECONF_DOUBLE_WIDE;<br>
                else<br>
                        pipeconf &= ~PIPECONF_DOUBLE_WIDE;<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.8.2.1<br>
<br>
</font></span></blockquote></div><br></div>