diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-11-26 08:17:05 -0500 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-12-03 03:35:17 -0500 |
commit | a4b3a5713d9f1ca94762b468117f918d3b15e5c4 (patch) | |
tree | bb941b2534eebefa526d6e84c4c8486ecab941a1 /drivers/gpu/drm/i915/intel_overlay.c | |
parent | 9bfc01a29b7d4d6b965a596b047b405bf6f58be1 (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.c | 11 |
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. */ |
368 | static int intel_overlay_recover_from_interrupt(struct intel_overlay *overlay) | 367 | static 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); |