Commit 496d0cc6 authored by Emil Velikov's avatar Emil Velikov Committed by Emil Velikov

drm/panfrost: remove _unlocked suffix in drm_gem_object_put_unlocked

Spelling out _unlocked for each and every driver is a annoying.
Especially if we consider how many drivers, do not know (or need to)
about the horror stories involving struct_mutex.

Just drop the suffix. It makes the API cleaner.

Done via the following script:

__from=drm_gem_object_put_unlocked
__to=drm_gem_object_put
for __file in $(git grep --name-only $__from); do
  sed -i  "s/$__from/$__to/g" $__file;
done

Cc: Rob Herring <robh@kernel.org>
Cc: Tomeu Vizoso <tomeu.vizoso@collabora.com>
Cc: Steven Price <steven.price@arm.com>
Signed-off-by: default avatarEmil Velikov <emil.velikov@collabora.com>
Acked-by: default avatarSam Ravnborg <sam@ravnborg.org>
Reviewed-by: default avatarSteven Price <steven.price@arm.com>
Acked-by: default avatarThomas Zimmermann <tzimmermann@suse.de>
Link: https://patchwork.freedesktop.org/patch/msgid/20200515095118.2743122-28-emil.l.velikov@gmail.com
parent d742cdd6
...@@ -99,7 +99,7 @@ static int panfrost_ioctl_create_bo(struct drm_device *dev, void *data, ...@@ -99,7 +99,7 @@ static int panfrost_ioctl_create_bo(struct drm_device *dev, void *data,
mapping = panfrost_gem_mapping_get(bo, priv); mapping = panfrost_gem_mapping_get(bo, priv);
if (!mapping) { if (!mapping) {
drm_gem_object_put_unlocked(&bo->base.base); drm_gem_object_put(&bo->base.base);
return -EINVAL; return -EINVAL;
} }
...@@ -317,7 +317,7 @@ panfrost_ioctl_wait_bo(struct drm_device *dev, void *data, ...@@ -317,7 +317,7 @@ panfrost_ioctl_wait_bo(struct drm_device *dev, void *data,
if (!ret) if (!ret)
ret = timeout ? -ETIMEDOUT : -EBUSY; ret = timeout ? -ETIMEDOUT : -EBUSY;
drm_gem_object_put_unlocked(gem_obj); drm_gem_object_put(gem_obj);
return ret; return ret;
} }
...@@ -351,7 +351,7 @@ static int panfrost_ioctl_mmap_bo(struct drm_device *dev, void *data, ...@@ -351,7 +351,7 @@ static int panfrost_ioctl_mmap_bo(struct drm_device *dev, void *data,
args->offset = drm_vma_node_offset_addr(&gem_obj->vma_node); args->offset = drm_vma_node_offset_addr(&gem_obj->vma_node);
out: out:
drm_gem_object_put_unlocked(gem_obj); drm_gem_object_put(gem_obj);
return ret; return ret;
} }
...@@ -372,7 +372,7 @@ static int panfrost_ioctl_get_bo_offset(struct drm_device *dev, void *data, ...@@ -372,7 +372,7 @@ static int panfrost_ioctl_get_bo_offset(struct drm_device *dev, void *data,
bo = to_panfrost_bo(gem_obj); bo = to_panfrost_bo(gem_obj);
mapping = panfrost_gem_mapping_get(bo, priv); mapping = panfrost_gem_mapping_get(bo, priv);
drm_gem_object_put_unlocked(gem_obj); drm_gem_object_put(gem_obj);
if (!mapping) if (!mapping)
return -EINVAL; return -EINVAL;
...@@ -438,7 +438,7 @@ static int panfrost_ioctl_madvise(struct drm_device *dev, void *data, ...@@ -438,7 +438,7 @@ static int panfrost_ioctl_madvise(struct drm_device *dev, void *data,
mutex_unlock(&bo->mappings.lock); mutex_unlock(&bo->mappings.lock);
mutex_unlock(&pfdev->shrinker_lock); mutex_unlock(&pfdev->shrinker_lock);
drm_gem_object_put_unlocked(gem_obj); drm_gem_object_put(gem_obj);
return ret; return ret;
} }
......
...@@ -93,7 +93,7 @@ static void panfrost_gem_mapping_release(struct kref *kref) ...@@ -93,7 +93,7 @@ static void panfrost_gem_mapping_release(struct kref *kref)
mapping = container_of(kref, struct panfrost_gem_mapping, refcount); mapping = container_of(kref, struct panfrost_gem_mapping, refcount);
panfrost_gem_teardown_mapping(mapping); panfrost_gem_teardown_mapping(mapping);
drm_gem_object_put_unlocked(&mapping->obj->base.base); drm_gem_object_put(&mapping->obj->base.base);
kfree(mapping); kfree(mapping);
} }
...@@ -261,7 +261,7 @@ panfrost_gem_create_with_handle(struct drm_file *file_priv, ...@@ -261,7 +261,7 @@ panfrost_gem_create_with_handle(struct drm_file *file_priv,
*/ */
ret = drm_gem_handle_create(file_priv, &shmem->base, handle); ret = drm_gem_handle_create(file_priv, &shmem->base, handle);
/* drop reference from allocate - handle holds it now. */ /* drop reference from allocate - handle holds it now. */
drm_gem_object_put_unlocked(&shmem->base); drm_gem_object_put(&shmem->base);
if (ret) if (ret)
return ERR_PTR(ret); return ERR_PTR(ret);
......
...@@ -281,7 +281,7 @@ static void panfrost_job_cleanup(struct kref *ref) ...@@ -281,7 +281,7 @@ static void panfrost_job_cleanup(struct kref *ref)
if (job->bos) { if (job->bos) {
for (i = 0; i < job->bo_count; i++) for (i = 0; i < job->bo_count; i++)
drm_gem_object_put_unlocked(job->bos[i]); drm_gem_object_put(job->bos[i]);
kvfree(job->bos); kvfree(job->bos);
} }
......
...@@ -538,7 +538,7 @@ static int panfrost_mmu_map_fault_addr(struct panfrost_device *pfdev, int as, ...@@ -538,7 +538,7 @@ static int panfrost_mmu_map_fault_addr(struct panfrost_device *pfdev, int as,
err_pages: err_pages:
drm_gem_shmem_put_pages(&bo->base); drm_gem_shmem_put_pages(&bo->base);
err_bo: err_bo:
drm_gem_object_put_unlocked(&bo->base.base); drm_gem_object_put(&bo->base.base);
return ret; return ret;
} }
......
...@@ -156,7 +156,7 @@ static int panfrost_perfcnt_enable_locked(struct panfrost_device *pfdev, ...@@ -156,7 +156,7 @@ static int panfrost_perfcnt_enable_locked(struct panfrost_device *pfdev,
gpu_write(pfdev, GPU_PRFCNT_TILER_EN, 0xffffffff); gpu_write(pfdev, GPU_PRFCNT_TILER_EN, 0xffffffff);
/* The BO ref is retained by the mapping. */ /* The BO ref is retained by the mapping. */
drm_gem_object_put_unlocked(&bo->base); drm_gem_object_put(&bo->base);
return 0; return 0;
...@@ -167,7 +167,7 @@ static int panfrost_perfcnt_enable_locked(struct panfrost_device *pfdev, ...@@ -167,7 +167,7 @@ static int panfrost_perfcnt_enable_locked(struct panfrost_device *pfdev,
err_close_bo: err_close_bo:
panfrost_gem_close(&bo->base, file_priv); panfrost_gem_close(&bo->base, file_priv);
err_put_bo: err_put_bo:
drm_gem_object_put_unlocked(&bo->base); drm_gem_object_put(&bo->base);
return ret; return ret;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment