aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm/i915/intel_overlay.c
diff options
context:
space:
mode:
authorDaniel Vetter <daniel.vetter@ffwll.ch>2014-11-26 08:17:05 -0500
committerDaniel Vetter <daniel.vetter@ffwll.ch>2014-12-03 03:35:17 -0500
commita4b3a5713d9f1ca94762b468117f918d3b15e5c4 (patch)
treebb941b2534eebefa526d6e84c4c8486ecab941a1 /drivers/gpu/drm/i915/intel_overlay.c
parent9bfc01a29b7d4d6b965a596b047b405bf6f58be1 (diff)
drm/i915: Convert i915_wait_seqno to i915_wait_request
Updated i915_wait_seqno() to take a request structure instead of a seqno value and renamed it accordingly. Internally, it just pulls the seqno out of the request and calls on to __wait_seqno() as before. However, all the code further up the stack is now simplified as it can just pass the request object straight through without having to peek inside. For: VIZ-4377 Signed-off-by: John Harrison <John.C.Harrison@Intel.com> Reviewed-by: Thomas Daniel <Thomas.Daniel@intel.com> [danvet: Squash in hunk from an earlier patch which was rebased wrongly.] Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_overlay.c')
-rw-r--r--drivers/gpu/drm/i915/intel_overlay.c11
1 files changed, 3 insertions, 8 deletions
diff --git a/drivers/gpu/drm/i915/intel_overlay.c b/drivers/gpu/drm/i915/intel_overlay.c
index 5defc37d8adb..6c530e298b64 100644
--- a/drivers/gpu/drm/i915/intel_overlay.c
+++ b/drivers/gpu/drm/i915/intel_overlay.c
@@ -224,8 +224,7 @@ static int intel_overlay_do_wait_request(struct intel_overlay *overlay,
224 return ret; 224 return ret;
225 225
226 overlay->flip_tail = tail; 226 overlay->flip_tail = tail;
227 ret = i915_wait_seqno(ring, 227 ret = i915_wait_request(overlay->last_flip_req);
228 i915_gem_request_get_seqno(overlay->last_flip_req));
229 if (ret) 228 if (ret)
230 return ret; 229 return ret;
231 i915_gem_retire_requests(dev); 230 i915_gem_retire_requests(dev);
@@ -367,19 +366,15 @@ static int intel_overlay_off(struct intel_overlay *overlay)
367 * We have to be careful not to repeat work forever an make forward progess. */ 366 * We have to be careful not to repeat work forever an make forward progess. */
368static int intel_overlay_recover_from_interrupt(struct intel_overlay *overlay) 367static int intel_overlay_recover_from_interrupt(struct intel_overlay *overlay)
369{ 368{
370 struct drm_device *dev = overlay->dev;
371 struct drm_i915_private *dev_priv = dev->dev_private;
372 struct intel_engine_cs *ring = &dev_priv->ring[RCS];
373 int ret; 369 int ret;
374 370
375 if (overlay->last_flip_req == NULL) 371 if (overlay->last_flip_req == NULL)
376 return 0; 372 return 0;
377 373
378 ret = i915_wait_seqno(ring, 374 ret = i915_wait_request(overlay->last_flip_req);
379 i915_gem_request_get_seqno(overlay->last_flip_req));
380 if (ret) 375 if (ret)
381 return ret; 376 return ret;
382 i915_gem_retire_requests(dev); 377 i915_gem_retire_requests(overlay->dev);
383 378
384 if (overlay->flip_tail) 379 if (overlay->flip_tail)
385 overlay->flip_tail(overlay); 380 overlay->flip_tail(overlay);