diff options
author | Ville Syrjälä <ville.syrjala@linux.intel.com> | 2013-09-10 04:40:40 -0400 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2013-09-10 05:15:39 -0400 |
commit | 46ba614c0045b0b5354397010578e8b56d621251 (patch) | |
tree | 4113142623a87b069b7d013b46644f9e2fa677f1 /drivers/gpu/drm/i915/intel_sprite.c | |
parent | da66146425c3136943452988afd3d64cd551da58 (diff) |
drm/i915: Pass crtc to intel_update_watermarks()
Passing the appropriate crtc to intel_update_watermarks() should help
in avoiding needless work in the future.
v2: Avoid clash with internal 'crtc' variable in some wm functions
Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_sprite.c')
-rw-r--r-- | drivers/gpu/drm/i915/intel_sprite.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/gpu/drm/i915/intel_sprite.c b/drivers/gpu/drm/i915/intel_sprite.c index ad6ec4b39005..d9c7a667553a 100644 --- a/drivers/gpu/drm/i915/intel_sprite.c +++ b/drivers/gpu/drm/i915/intel_sprite.c | |||
@@ -288,7 +288,7 @@ ivb_update_plane(struct drm_plane *plane, struct drm_crtc *crtc, | |||
288 | dev_priv->sprite_scaling_enabled |= 1 << pipe; | 288 | dev_priv->sprite_scaling_enabled |= 1 << pipe; |
289 | 289 | ||
290 | if (!scaling_was_enabled) { | 290 | if (!scaling_was_enabled) { |
291 | intel_update_watermarks(dev); | 291 | intel_update_watermarks(crtc); |
292 | intel_wait_for_vblank(dev, pipe); | 292 | intel_wait_for_vblank(dev, pipe); |
293 | } | 293 | } |
294 | sprscale = SPRITE_SCALE_ENABLE | (src_w << 16) | src_h; | 294 | sprscale = SPRITE_SCALE_ENABLE | (src_w << 16) | src_h; |
@@ -323,7 +323,7 @@ ivb_update_plane(struct drm_plane *plane, struct drm_crtc *crtc, | |||
323 | 323 | ||
324 | /* potentially re-enable LP watermarks */ | 324 | /* potentially re-enable LP watermarks */ |
325 | if (scaling_was_enabled && !dev_priv->sprite_scaling_enabled) | 325 | if (scaling_was_enabled && !dev_priv->sprite_scaling_enabled) |
326 | intel_update_watermarks(dev); | 326 | intel_update_watermarks(crtc); |
327 | } | 327 | } |
328 | 328 | ||
329 | static void | 329 | static void |
@@ -349,7 +349,7 @@ ivb_disable_plane(struct drm_plane *plane, struct drm_crtc *crtc) | |||
349 | 349 | ||
350 | /* potentially re-enable LP watermarks */ | 350 | /* potentially re-enable LP watermarks */ |
351 | if (scaling_was_enabled && !dev_priv->sprite_scaling_enabled) | 351 | if (scaling_was_enabled && !dev_priv->sprite_scaling_enabled) |
352 | intel_update_watermarks(dev); | 352 | intel_update_watermarks(crtc); |
353 | } | 353 | } |
354 | 354 | ||
355 | static int | 355 | static int |