drm/i915: split PIN_GLOBAL out from PIN_MAPPABLE

With abitrary pin flags it makes sense to split out a "please bind
this into global gtt" from the "please allocate in the mappable
range".

Use this unconditionally in our global gtt pin helper since this is
what its callers want. Later patches will drop PIN_MAPPABLE where it's
not strictly needed.

Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

1 files changed, 2 insertions, 2 deletions

diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c
index aa263e371ebc..b3a15c9fa034 100644
--- a/drivers/gpu/drm/i915/i915_gem.c
+++ b/drivers/gpu/drm/i915/i915_gem.c
@@ -3838,7 +3838,7 @@ i915_gem_object_pin(struct drm_i915_gem_object *obj,
         struct i915_vma *vma;
         int ret;
 
-        if (WARN_ON(flags & PIN_MAPPABLE && !i915_is_ggtt(vm)))
+        if (WARN_ON(flags & (PIN_GLOBAL | PIN_MAPPABLE) && !i915_is_ggtt(vm)))
                 return -EINVAL;
 
         vma = i915_gem_obj_to_vma(obj, vm);
@@ -3872,7 +3872,7 @@ i915_gem_object_pin(struct drm_i915_gem_object *obj,
         vma = i915_gem_obj_to_vma(obj, vm);
 
         vma->bind_vma(vma, obj->cache_level,
-                      flags & PIN_MAPPABLE ? GLOBAL_BIND : 0);
+                      flags & PIN_GLOBAL ? GLOBAL_BIND : 0);
 
         i915_gem_obj_to_vma(obj, vm)->pin_count++;
         if (flags & PIN_MAPPABLE)