diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2011-04-13 09:28:23 +0100 |
|---|---|---|
| committer | Keith Packard <keithp@keithp.com> | 2011-04-13 09:28:41 -0700 |
| commit | 5c72d064f7ead1126bed6faab0c2bfb7418036e2 (patch) | |
| tree | c5278936857a313edacb354536546b563d500d9c | |
| parent | f6e5b1603b8bb7131b6778d0d4e2e5dda120a379 (diff) | |
| download | talos-op-linux-5c72d064f7ead1126bed6faab0c2bfb7418036e2.tar.gz talos-op-linux-5c72d064f7ead1126bed6faab0c2bfb7418036e2.zip | |
drm/i915: Initialise g4x watermarks for disabled pipes
We were using uninitialised watermarks values for disabled pipes which
were combined into a single WM register and so corrupting the values for
the enabled pipe and upsetting the display hardware.
Reported-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com>
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=32612
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Keith Packard <keithp@keithp.com>
| -rw-r--r-- | drivers/gpu/drm/i915/intel_display.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 4fc21e047a2d..e522c702b04e 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -3771,8 +3771,11 @@ static bool g4x_compute_wm0(struct drm_device *dev, int entries, tlb_miss; crtc = intel_get_crtc_for_plane(dev, plane); - if (crtc->fb == NULL || !crtc->enabled) + if (crtc->fb == NULL || !crtc->enabled) { + *cursor_wm = cursor->guard_size; + *plane_wm = display->guard_size; return false; + } htotal = crtc->mode.htotal; hdisplay = crtc->mode.hdisplay; |
