diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-02-14 08:01:11 -0500 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2014-02-14 08:16:58 -0500 |
commit | 1ec9e26ddab06459e89a890431b2de064c5d1056 (patch) | |
tree | 6ea1f8cfcffa2acf9ffec2fc12b197cc65e1d978 /drivers/gpu/drm/i915/intel_pm.c | |
parent | 931c1c26983b4f84e33b78579fc8d57e4a14c6b4 (diff) |
drm/i915: Consolidate binding parameters into flags
Anything more than just one bool parameter is just a pain to read,
symbolic constants are much better.
Split out from Chris' vma-binding rework patch.
v2: Undo the behaviour change in object_pin that Chris spotted.
v3: Split out misplaced hunk to handle set_cache_level errors,
spotted by Jani.
v4: Keep the current over-zealous binding logic in the execbuffer code
working with a quick hack while the overall binding code gets shuffled
around.
v5: Reorder the PIN_ flags for more natural patch splitup.
v6: Pull out the PIN_GLOBAL split-up again.
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Ben Widawsky <benjamin.widawsky@intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_pm.c')
-rw-r--r-- | drivers/gpu/drm/i915/intel_pm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c index e4a0c9cc226d..136647037244 100644 --- a/drivers/gpu/drm/i915/intel_pm.c +++ b/drivers/gpu/drm/i915/intel_pm.c | |||
@@ -2741,7 +2741,7 @@ intel_alloc_context_page(struct drm_device *dev) | |||
2741 | return NULL; | 2741 | return NULL; |
2742 | } | 2742 | } |
2743 | 2743 | ||
2744 | ret = i915_gem_obj_ggtt_pin(ctx, 4096, true, false); | 2744 | ret = i915_gem_obj_ggtt_pin(ctx, 4096, PIN_MAPPABLE); |
2745 | if (ret) { | 2745 | if (ret) { |
2746 | DRM_ERROR("failed to pin power context: %d\n", ret); | 2746 | DRM_ERROR("failed to pin power context: %d\n", ret); |
2747 | goto err_unref; | 2747 | goto err_unref; |