diff options
| author | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2016-01-14 13:53:34 +0100 |
|---|---|---|
| committer | Matt Roper <matthew.d.roper@intel.com> | 2016-01-15 03:33:25 +0100 |
| commit | f1ecaf8f9c9924badd4c8f690e39632d31edb500 (patch) | |
| tree | 386a94552b10ba3229c3b49451a9f0cd8db257a7 /drivers/gpu/drm/i915 | |
| parent | drm/i915: add onoff utility function (diff) | |
| download | linux-f1ecaf8f9c9924badd4c8f690e39632d31edb500.tar.xz linux-f1ecaf8f9c9924badd4c8f690e39632d31edb500.zip | |
drm/i915: Start WM computation from scratch on ILK-BDW
ilk_compute_pipe_wm() assumes as zeroed pipe_wm structure when it
starts. We used to pass such a zeroed struct in, but this got broken
when the pipe_wm structure got embedded in the crtc state.
To fix it without too much fuzz, we need to resort to a memset().
Fixes: 4e0963c7663b ("drm/i915: Calculate pipe watermarks into CRTC state (v3)")
Cc: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Matt Roper <matthew.d.roper@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1452776015-22076-1-git-send-email-ville.syrjala@linux.intel.com
Diffstat (limited to 'drivers/gpu/drm/i915')
| -rw-r--r-- | drivers/gpu/drm/i915/intel_pm.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c index 465ca76df201..966c4c2c416f 100644 --- a/drivers/gpu/drm/i915/intel_pm.c +++ b/drivers/gpu/drm/i915/intel_pm.c @@ -2317,6 +2317,7 @@ static int ilk_compute_pipe_wm(struct intel_crtc *intel_crtc, return PTR_ERR(cstate); pipe_wm = &cstate->wm.optimal.ilk; + memset(pipe_wm, 0, sizeof(*pipe_wm)); for_each_intel_plane_on_crtc(dev, intel_crtc, intel_plane) { ps = drm_atomic_get_plane_state(state, |