drm/i915: Apply the g4x TLB miss w/a to SR watermarks as well
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 21 Apr 2017 18:14:28 +0000 (21:14 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 10 May 2017 13:48:31 +0000 (16:48 +0300)
The documentation I've seen doesn't actually specify which watermarks
need the TLB miss w/a. Currently we only apply the w/a to the normal
watermarks for both primary and cursor planes. Since the documentation
doesn't explicitly say anything I'm going to assume that the w/a should
equally apply to the SR/HPLL watermarks. So let's do that.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170421181432.15216-12-ville.syrjala@linux.intel.com
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
drivers/gpu/drm/i915/intel_pm.c

index c07f3b2b097289feef1b5b5bf7cbaf47a9339318..61b67994c4a8284a4b571b91ebf2277e84ec9bc1 100644 (file)
@@ -1006,7 +1006,7 @@ static bool g4x_compute_srwm(struct drm_i915_private *dev_priv,
        struct intel_crtc *crtc;
        const struct drm_display_mode *adjusted_mode;
        const struct drm_framebuffer *fb;
-       int hdisplay, htotal, cpp, clock;
+       int plane_width, cursor_width, htotal, cpp, clock;
        int small, large;
        int entries;
 
@@ -1020,20 +1020,23 @@ static bool g4x_compute_srwm(struct drm_i915_private *dev_priv,
        fb = crtc->base.primary->state->fb;
        clock = adjusted_mode->crtc_clock;
        htotal = adjusted_mode->crtc_htotal;
-       hdisplay = crtc->config->pipe_src_w;
+       plane_width = crtc->config->pipe_src_w;
+       cursor_width = crtc->base.cursor->state->crtc_w;
        cpp = fb->format->cpp[0];
 
        /* Use the minimum of the small and large buffer method for primary */
        small = intel_wm_method1(clock, cpp, latency_ns / 100);
-       large = intel_wm_method2(clock, htotal, hdisplay, cpp,
+       large = intel_wm_method2(clock, htotal, plane_width, cpp,
                                 latency_ns / 100);
-       entries = DIV_ROUND_UP(min(small, large), display->cacheline_size);
+       entries = min(small, large);
+       entries += g4x_tlb_miss_wa(display->fifo_size, plane_width, cpp);
+       entries = DIV_ROUND_UP(entries, display->cacheline_size);
        *display_wm = entries + display->guard_size;
 
        /* calculate the self-refresh watermark for display cursor */
-       entries = intel_wm_method2(clock, htotal,
-                                  crtc->base.cursor->state->crtc_w, 4,
+       entries = intel_wm_method2(clock, htotal, cursor_width, 4,
                                   latency_ns / 100);
+       entries += g4x_tlb_miss_wa(cursor->fifo_size, cursor_width, 4);
        entries = DIV_ROUND_UP(entries, cursor->cacheline_size);
        *cursor_wm = entries + cursor->guard_size;