linux-tegra.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v3 00/28] DRM API Conversions
@ 2017-08-11 12:32 Cihangir Akturk
  2017-08-11 12:33 ` [PATCH v3 21/28] drm/tegra: switch to drm_*_get(), drm_*_put() helpers Cihangir Akturk
  2017-08-11 14:24 ` [PATCH v3 00/28] DRM API Conversions Deucher, Alexander
  0 siblings, 2 replies; 5+ messages in thread
From: Cihangir Akturk @ 2017-08-11 12:32 UTC (permalink / raw)
  Cc: devel-gWbeCf7V1WCQmaza687I9mD2FQJk+8+b,
	linux-arm-msm-u79uwXL29TY76Z2rM5mHXA,
	intel-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	etnaviv-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW, Cihangir Akturk,
	amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW, daniel-/w4YWyX8dFk,
	nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	virtualization-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA,
	freedreno-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW

Changes since v2:

- Patch series is based on *drm-misc-next* as suggested by Sean Paul.

- Dropped patch 05 (drm/atmel-hlcdc) and patch 25 (drm/vc4) from v2,
  since they were already pulled in the drm-misc-next

Changes since v1:

- This time patches were generated with coccinelle instead of my own
  script, as suggested by Daniel Vetter.

- Fixed the typo in commit messages. s/adn/and

Note: I've included r-b, a-b tags, as these patches are identical to v1
except for the file: drivers/gpu/drm/i915/i915_gem_object.h

This patch set replaces the occurrences of drm_*_reference() and
drm_*_unreference() with the new drm_*_get() and drm_*_put() functions.
All patches in the series do the same thing, converting to the new APIs.
I created patches per DRM driver as suggested by Daniel Vetter.

This patch set was generated by scripts/coccinelle/api/drm-get-put.cocci

Previous thread can be reached at:
https://marc.info/?l=dri-devel&m=150178288816047

Background:

In the kernel, reference counting APIs use *_get(), *_put() style naming
to reference-count the objects. But DRM subsystem uses a different
naming for them such as *_reference(), *_unreference() which is
inconsistent with the other reference counting APIs in the kernel. To
solve this consistency issue, Thierry Reding introduced a couple of
functions and compatibility aliases in the following commits for them.

commit 020a218f95bd3ceff7dd1022ff7ebc0497bc7bf9
Author: Thierry Reding <treding@nvidia.com>
Date:   Tue Feb 28 15:46:38 2017 +0100

    drm: Introduce drm_mode_object_{get,put}()

commit ad09360750afa18a0a0ce0253d6ea6033abc22e7
Author: Thierry Reding <treding@nvidia.com>
Date:   Tue Feb 28 15:46:39 2017 +0100

    drm: Introduce drm_connector_{get,put}()

commit a4a69da06bc11a937a6e417938b1bb698ee1fa46
Author: Thierry Reding <treding@nvidia.com>
Date:   Tue Feb 28 15:46:40 2017 +0100

    drm: Introduce drm_framebuffer_{get,put}()

commit e6b62714e87c8811d5564b6a0738dcde63a51774
Author: Thierry Reding <treding@nvidia.com>
Date:   Tue Feb 28 15:46:41 2017 +0100

    drm: Introduce drm_gem_object_{get,put}()

commit 6472e5090be7c78749a3c279b4faae87ab835c40
Author: Thierry Reding <treding@nvidia.com>
Date:   Tue Feb 28 15:46:42 2017 +0100

    drm: Introduce drm_property_blob_{get,put}()

Cihangir Akturk (28):
  drm/amdgpu: switch to drm_*_get(), drm_*_put() helpers
  drm: mali-dp: switch to drm_*_get(), drm_*_put() helpers
  drm/armada: switch to drm_*_get(), drm_*_put() helpers
  drm/ast: switch to drm_*_get(), drm_*_put() helpers
  drm/bochs: switch to drm_*_get(), drm_*_put() helpers
  drm/cirrus: switch to drm_*_get(), drm_*_put() helpers
  drm/etnaviv: switch to drm_*_get(), drm_*_put() helpers
  drm/exynos: switch to drm_*_get(), drm_*_put() helpers
  drm/gma500: switch to drm_*_get(), drm_*_put() helpers
  drm/hisilicon: switch to drm_*_get(), drm_*_put() helpers
  drm/i915: switch to drm_*_get(), drm_*_put() helpers
  drm/imx: switch to drm_*_get(), drm_*_put() helpers
  drm/mediatek: switch to drm_*_get(), drm_*_put() helpers
  drm/mgag200: switch to drm_*_get(), drm_*_put() helpers
  drm/msm: switch to drm_*_get(), drm_*_put() helpers
  drm/nouveau: switch to drm_*_get(), drm_*_put() helpers
  drm/omapdrm: switch to drm_*_get(), drm_*_put() helpers
  drm/qxl: switch to drm_*_get(), drm_*_put() helpers
  drm/radeon: switch to drm_*_get(), drm_*_put() helpers
  drm/rockchip: switch to drm_*_get(), drm_*_put() helpers
  drm/tegra: switch to drm_*_get(), drm_*_put() helpers
  drm/tilcdc: switch to drm_*_get(), drm_*_put() helpers
  drm/udl: switch to drm_*_get(), drm_*_put() helpers
  drm/vc4: switch to drm_*_get(), drm_*_put() helpers
  drm/vgem: switch to drm_*_get(), drm_*_put() helpers
  drm/virtio: switch to drm_*_get(), drm_*_put() helpers
  drm/vmwgfx: switch to drm_*_get(), drm_*_put() helpers
  drm: vboxvideo: switch to drm_*_get(), drm_*_put() helpers

 drivers/gpu/drm/amd/amdgpu/amdgpu_bo_list.c       |  2 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c            |  2 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_display.c       |  6 ++---
 drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c            |  4 +--
 drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c           | 22 ++++++++---------
 drivers/gpu/drm/amd/amdgpu/dce_v10_0.c            |  6 ++---
 drivers/gpu/drm/amd/amdgpu/dce_v11_0.c            |  6 ++---
 drivers/gpu/drm/amd/amdgpu/dce_v6_0.c             |  6 ++---
 drivers/gpu/drm/amd/amdgpu/dce_v8_0.c             |  6 ++---
 drivers/gpu/drm/arm/malidp_planes.c               |  2 +-
 drivers/gpu/drm/armada/armada_crtc.c              | 22 ++++++++---------
 drivers/gpu/drm/armada/armada_drv.c               |  2 +-
 drivers/gpu/drm/armada/armada_fb.c                |  8 +++---
 drivers/gpu/drm/armada/armada_fbdev.c             |  6 ++---
 drivers/gpu/drm/armada/armada_gem.c               | 14 +++++------
 drivers/gpu/drm/armada/armada_overlay.c           |  4 +--
 drivers/gpu/drm/ast/ast_fb.c                      |  2 +-
 drivers/gpu/drm/ast/ast_main.c                    | 10 ++++----
 drivers/gpu/drm/ast/ast_mode.c                    |  6 ++---
 drivers/gpu/drm/bochs/bochs_fbdev.c               |  2 +-
 drivers/gpu/drm/bochs/bochs_mm.c                  | 10 ++++----
 drivers/gpu/drm/cirrus/cirrus_fbdev.c             |  2 +-
 drivers/gpu/drm/cirrus/cirrus_main.c              | 10 ++++----
 drivers/gpu/drm/etnaviv/etnaviv_drv.c             |  8 +++---
 drivers/gpu/drm/etnaviv/etnaviv_gem.c             | 20 +++++++--------
 drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c       |  2 +-
 drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c      |  4 +--
 drivers/gpu/drm/exynos/exynos_drm_fb.c            |  4 +--
 drivers/gpu/drm/exynos/exynos_drm_gem.c           | 12 ++++-----
 drivers/gpu/drm/exynos/exynos_drm_plane.c         |  2 +-
 drivers/gpu/drm/gma500/framebuffer.c              |  4 +--
 drivers/gpu/drm/gma500/gem.c                      |  4 +--
 drivers/gpu/drm/gma500/gma_display.c              |  6 ++---
 drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c |  4 +--
 drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c       |  8 +++---
 drivers/gpu/drm/i915/i915_gem_object.h            |  4 +--
 drivers/gpu/drm/i915/intel_display.c              | 24 +++++++++---------
 drivers/gpu/drm/i915/intel_dp_mst.c               |  2 +-
 drivers/gpu/drm/i915/intel_fbdev.c                |  4 +--
 drivers/gpu/drm/imx/ipuv3-crtc.c                  |  2 +-
 drivers/gpu/drm/mediatek/mtk_drm_fb.c             |  4 +--
 drivers/gpu/drm/mediatek/mtk_drm_gem.c            |  2 +-
 drivers/gpu/drm/mgag200/mgag200_cursor.c          |  2 +-
 drivers/gpu/drm/mgag200/mgag200_fb.c              |  4 +--
 drivers/gpu/drm/mgag200/mgag200_main.c            | 10 ++++----
 drivers/gpu/drm/msm/adreno/a5xx_gpu.c             | 10 ++++----
 drivers/gpu/drm/msm/adreno/a5xx_power.c           |  2 +-
 drivers/gpu/drm/msm/adreno/adreno_gpu.c           |  2 +-
 drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c          |  6 ++---
 drivers/gpu/drm/msm/mdp/mdp4/mdp4_kms.c           |  2 +-
 drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c          |  2 +-
 drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c         |  4 +--
 drivers/gpu/drm/msm/msm_drv.c                     |  8 +++---
 drivers/gpu/drm/msm/msm_fb.c                      |  4 +--
 drivers/gpu/drm/msm/msm_fbdev.c                   |  2 +-
 drivers/gpu/drm/msm/msm_gem.c                     |  8 +++---
 drivers/gpu/drm/msm/msm_gem_submit.c              |  4 +--
 drivers/gpu/drm/msm/msm_gpu.c                     |  4 +--
 drivers/gpu/drm/msm/msm_ringbuffer.c              |  2 +-
 drivers/gpu/drm/nouveau/dispnv04/crtc.c           |  2 +-
 drivers/gpu/drm/nouveau/nouveau_abi16.c           |  2 +-
 drivers/gpu/drm/nouveau/nouveau_display.c         |  8 +++---
 drivers/gpu/drm/nouveau/nouveau_fbcon.c           |  2 +-
 drivers/gpu/drm/nouveau/nouveau_gem.c             | 14 +++++------
 drivers/gpu/drm/nouveau/nv50_display.c            |  2 +-
 drivers/gpu/drm/omapdrm/omap_drv.c                |  2 +-
 drivers/gpu/drm/omapdrm/omap_fb.c                 |  4 +--
 drivers/gpu/drm/omapdrm/omap_fbdev.c              |  2 +-
 drivers/gpu/drm/omapdrm/omap_gem.c                |  4 +--
 drivers/gpu/drm/omapdrm/omap_gem_dmabuf.c         |  2 +-
 drivers/gpu/drm/qxl/qxl_display.c                 |  4 +--
 drivers/gpu/drm/qxl/qxl_dumb.c                    |  2 +-
 drivers/gpu/drm/qxl/qxl_fb.c                      |  6 ++---
 drivers/gpu/drm/qxl/qxl_gem.c                     |  2 +-
 drivers/gpu/drm/qxl/qxl_ioctl.c                   |  4 +--
 drivers/gpu/drm/qxl/qxl_object.c                  |  6 ++---
 drivers/gpu/drm/radeon/radeon_cs.c                |  2 +-
 drivers/gpu/drm/radeon/radeon_cursor.c            |  6 ++---
 drivers/gpu/drm/radeon/radeon_display.c           | 12 ++++-----
 drivers/gpu/drm/radeon/radeon_fb.c                |  4 +--
 drivers/gpu/drm/radeon/radeon_gem.c               | 30 +++++++++++------------
 drivers/gpu/drm/radeon/radeon_object.c            |  2 +-
 drivers/gpu/drm/rockchip/rockchip_drm_fb.c        |  6 ++---
 drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c     |  2 +-
 drivers/gpu/drm/rockchip/rockchip_drm_gem.c       |  2 +-
 drivers/gpu/drm/rockchip/rockchip_drm_vop.c       |  4 +--
 drivers/gpu/drm/tegra/drm.c                       | 12 ++++-----
 drivers/gpu/drm/tegra/fb.c                        |  8 +++---
 drivers/gpu/drm/tegra/gem.c                       | 10 ++++----
 drivers/gpu/drm/tilcdc/tilcdc_crtc.c              |  6 ++---
 drivers/gpu/drm/udl/udl_fb.c                      |  6 ++---
 drivers/gpu/drm/udl/udl_gem.c                     |  4 +--
 drivers/gpu/drm/vc4/vc4_bo.c                      |  2 +-
 drivers/gpu/drm/vgem/vgem_drv.c                   |  4 +--
 drivers/gpu/drm/vgem/vgem_fence.c                 |  2 +-
 drivers/gpu/drm/virtio/virtgpu_display.c          |  4 +--
 drivers/gpu/drm/virtio/virtgpu_gem.c              |  4 +--
 drivers/gpu/drm/virtio/virtgpu_ioctl.c            | 14 +++++------
 drivers/gpu/drm/vmwgfx/vmwgfx_fb.c                |  2 +-
 drivers/gpu/drm/vmwgfx/vmwgfx_ioctl.c             |  4 +--
 drivers/staging/vboxvideo/vbox_fb.c               |  2 +-
 drivers/staging/vboxvideo/vbox_main.c             |  8 +++---
 drivers/staging/vboxvideo/vbox_mode.c             |  2 +-
 103 files changed, 294 insertions(+), 294 deletions(-)

-- 
2.7.4

_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH v3 21/28] drm/tegra: switch to drm_*_get(), drm_*_put() helpers
  2017-08-11 12:32 [PATCH v3 00/28] DRM API Conversions Cihangir Akturk
@ 2017-08-11 12:33 ` Cihangir Akturk
       [not found]   ` <1502454794-28558-22-git-send-email-cakturk-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
  2017-08-11 14:24 ` [PATCH v3 00/28] DRM API Conversions Deucher, Alexander
  1 sibling, 1 reply; 5+ messages in thread
From: Cihangir Akturk @ 2017-08-11 12:33 UTC (permalink / raw)
  Cc: daniel, linux-kernel, dri-devel, Cihangir Akturk, Thierry Reding,
	David Airlie, Jonathan Hunter, linux-tegra

Use drm_*_get() and drm_*_put() helpers instead of drm_*_reference()
and drm_*_unreference() helpers.

drm_*_reference() and drm_*_unreference() functions are just
compatibility alias for drm_*_get() and drm_*_put() and should not be
used by new code. So convert all users of compatibility functions to
use the new APIs.

Generated by: scripts/coccinelle/api/drm-get-put.cocci

Signed-off-by: Cihangir Akturk <cakturk@gmail.com>
---
 drivers/gpu/drm/tegra/drm.c | 12 ++++++------
 drivers/gpu/drm/tegra/fb.c  |  8 ++++----
 drivers/gpu/drm/tegra/gem.c | 10 +++++-----
 3 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/drivers/gpu/drm/tegra/drm.c b/drivers/gpu/drm/tegra/drm.c
index 3ba659a..e70c67b 100644
--- a/drivers/gpu/drm/tegra/drm.c
+++ b/drivers/gpu/drm/tegra/drm.c
@@ -306,7 +306,7 @@ host1x_bo_lookup(struct drm_file *file, u32 handle)
 	if (!gem)
 		return NULL;
 
-	drm_gem_object_unreference_unlocked(gem);
+	drm_gem_object_put_unlocked(gem);
 
 	bo = to_tegra_bo(gem);
 	return &bo->base;
@@ -593,7 +593,7 @@ static int tegra_gem_mmap(struct drm_device *drm, void *data,
 
 	args->offset = drm_vma_node_offset_addr(&bo->gem.vma_node);
 
-	drm_gem_object_unreference_unlocked(gem);
+	drm_gem_object_put_unlocked(gem);
 
 	return 0;
 }
@@ -860,7 +860,7 @@ static int tegra_gem_set_tiling(struct drm_device *drm, void *data,
 	bo->tiling.mode = mode;
 	bo->tiling.value = value;
 
-	drm_gem_object_unreference_unlocked(gem);
+	drm_gem_object_put_unlocked(gem);
 
 	return 0;
 }
@@ -900,7 +900,7 @@ static int tegra_gem_get_tiling(struct drm_device *drm, void *data,
 		break;
 	}
 
-	drm_gem_object_unreference_unlocked(gem);
+	drm_gem_object_put_unlocked(gem);
 
 	return err;
 }
@@ -925,7 +925,7 @@ static int tegra_gem_set_flags(struct drm_device *drm, void *data,
 	if (args->flags & DRM_TEGRA_GEM_BOTTOM_UP)
 		bo->flags |= TEGRA_BO_BOTTOM_UP;
 
-	drm_gem_object_unreference_unlocked(gem);
+	drm_gem_object_put_unlocked(gem);
 
 	return 0;
 }
@@ -947,7 +947,7 @@ static int tegra_gem_get_flags(struct drm_device *drm, void *data,
 	if (bo->flags & TEGRA_BO_BOTTOM_UP)
 		args->flags |= DRM_TEGRA_GEM_BOTTOM_UP;
 
-	drm_gem_object_unreference_unlocked(gem);
+	drm_gem_object_put_unlocked(gem);
 
 	return 0;
 }
diff --git a/drivers/gpu/drm/tegra/fb.c b/drivers/gpu/drm/tegra/fb.c
index 25acb73..80540c1 100644
--- a/drivers/gpu/drm/tegra/fb.c
+++ b/drivers/gpu/drm/tegra/fb.c
@@ -88,7 +88,7 @@ static void tegra_fb_destroy(struct drm_framebuffer *framebuffer)
 			if (bo->pages)
 				vunmap(bo->vaddr);
 
-			drm_gem_object_unreference_unlocked(&bo->gem);
+			drm_gem_object_put_unlocked(&bo->gem);
 		}
 	}
 
@@ -195,7 +195,7 @@ struct drm_framebuffer *tegra_fb_create(struct drm_device *drm,
 
 unreference:
 	while (i--)
-		drm_gem_object_unreference_unlocked(&planes[i]->gem);
+		drm_gem_object_put_unlocked(&planes[i]->gem);
 
 	return ERR_PTR(err);
 }
@@ -242,7 +242,7 @@ static int tegra_fbdev_probe(struct drm_fb_helper *helper,
 	info = drm_fb_helper_alloc_fbi(helper);
 	if (IS_ERR(info)) {
 		dev_err(drm->dev, "failed to allocate framebuffer info\n");
-		drm_gem_object_unreference_unlocked(&bo->gem);
+		drm_gem_object_put_unlocked(&bo->gem);
 		return PTR_ERR(info);
 	}
 
@@ -251,7 +251,7 @@ static int tegra_fbdev_probe(struct drm_fb_helper *helper,
 		err = PTR_ERR(fbdev->fb);
 		dev_err(drm->dev, "failed to allocate DRM framebuffer: %d\n",
 			err);
-		drm_gem_object_unreference_unlocked(&bo->gem);
+		drm_gem_object_put_unlocked(&bo->gem);
 		return PTR_ERR(fbdev->fb);
 	}
 
diff --git a/drivers/gpu/drm/tegra/gem.c b/drivers/gpu/drm/tegra/gem.c
index 7a39a35..fe77bcd 100644
--- a/drivers/gpu/drm/tegra/gem.c
+++ b/drivers/gpu/drm/tegra/gem.c
@@ -24,7 +24,7 @@ static void tegra_bo_put(struct host1x_bo *bo)
 {
 	struct tegra_bo *obj = host1x_to_tegra_bo(bo);
 
-	drm_gem_object_unreference_unlocked(&obj->gem);
+	drm_gem_object_put_unlocked(&obj->gem);
 }
 
 static dma_addr_t tegra_bo_pin(struct host1x_bo *bo, struct sg_table **sgt)
@@ -95,7 +95,7 @@ static struct host1x_bo *tegra_bo_get(struct host1x_bo *bo)
 {
 	struct tegra_bo *obj = host1x_to_tegra_bo(bo);
 
-	drm_gem_object_reference(&obj->gem);
+	drm_gem_object_get(&obj->gem);
 
 	return bo;
 }
@@ -325,7 +325,7 @@ struct tegra_bo *tegra_bo_create_with_handle(struct drm_file *file,
 		return ERR_PTR(err);
 	}
 
-	drm_gem_object_unreference_unlocked(&bo->gem);
+	drm_gem_object_put_unlocked(&bo->gem);
 
 	return bo;
 }
@@ -439,7 +439,7 @@ int tegra_bo_dumb_map_offset(struct drm_file *file, struct drm_device *drm,
 
 	*offset = drm_vma_node_offset_addr(&bo->gem.vma_node);
 
-	drm_gem_object_unreference_unlocked(gem);
+	drm_gem_object_put_unlocked(gem);
 
 	return 0;
 }
@@ -654,7 +654,7 @@ struct drm_gem_object *tegra_gem_prime_import(struct drm_device *drm,
 		struct drm_gem_object *gem = buf->priv;
 
 		if (gem->dev == drm) {
-			drm_gem_object_reference(gem);
+			drm_gem_object_get(gem);
 			return gem;
 		}
 	}
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* RE: [PATCH v3 00/28] DRM API Conversions
  2017-08-11 12:32 [PATCH v3 00/28] DRM API Conversions Cihangir Akturk
  2017-08-11 12:33 ` [PATCH v3 21/28] drm/tegra: switch to drm_*_get(), drm_*_put() helpers Cihangir Akturk
@ 2017-08-11 14:24 ` Deucher, Alexander
  2017-08-11 15:21   ` Cihangir Akturk
  1 sibling, 1 reply; 5+ messages in thread
From: Deucher, Alexander @ 2017-08-11 14:24 UTC (permalink / raw)
  To: 'Cihangir Akturk'
  Cc: devel@driverdev.osuosl.org, linux-arm-msm@vger.kernel.org,
	intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	dri-devel@lists.freedesktop.org, etnaviv@lists.freedesktop.org,
	amd-gfx@lists.freedesktop.org, daniel@ffwll.ch,
	nouveau@lists.freedesktop.org, linux-tegra@vger.kernel.org,
	virtualization@lists.linux-foundation.org,
	freedreno@lists.freedesktop.org

> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces@lists.freedesktop.org] On Behalf
> Of Cihangir Akturk
> Sent: Friday, August 11, 2017 8:33 AM
> Cc: devel@driverdev.osuosl.org; linux-arm-msm@vger.kernel.org; intel-
> gfx@lists.freedesktop.org; linux-kernel@vger.kernel.org; dri-
> devel@lists.freedesktop.org; etnaviv@lists.freedesktop.org; Cihangir Akturk;
> amd-gfx@lists.freedesktop.org; daniel@ffwll.ch;
> nouveau@lists.freedesktop.org; linux-tegra@vger.kernel.org;
> virtualization@lists.linux-foundation.org; freedreno@lists.freedesktop.org
> Subject: [PATCH v3 00/28] DRM API Conversions
> 
> Changes since v2:
> 
> - Patch series is based on *drm-misc-next* as suggested by Sean Paul.
> 
> - Dropped patch 05 (drm/atmel-hlcdc) and patch 25 (drm/vc4) from v2,
>   since they were already pulled in the drm-misc-next
> 
> Changes since v1:
> 
> - This time patches were generated with coccinelle instead of my own
>   script, as suggested by Daniel Vetter.
> 
> - Fixed the typo in commit messages. s/adn/and
> 

FWIW, I already picked up v1 of these patches for radeon and amdgpu.

Alex

> Note: I've included r-b, a-b tags, as these patches are identical to v1
> except for the file: drivers/gpu/drm/i915/i915_gem_object.h
> 
> This patch set replaces the occurrences of drm_*_reference() and
> drm_*_unreference() with the new drm_*_get() and drm_*_put()
> functions.
> All patches in the series do the same thing, converting to the new APIs.
> I created patches per DRM driver as suggested by Daniel Vetter.
> 
> This patch set was generated by scripts/coccinelle/api/drm-get-put.cocci
> 
> Previous thread can be reached at:
> https://marc.info/?l=dri-devel&m=150178288816047
> 
> Background:
> 
> In the kernel, reference counting APIs use *_get(), *_put() style naming
> to reference-count the objects. But DRM subsystem uses a different
> naming for them such as *_reference(), *_unreference() which is
> inconsistent with the other reference counting APIs in the kernel. To
> solve this consistency issue, Thierry Reding introduced a couple of
> functions and compatibility aliases in the following commits for them.
> 
> commit 020a218f95bd3ceff7dd1022ff7ebc0497bc7bf9
> Author: Thierry Reding <treding@nvidia.com>
> Date:   Tue Feb 28 15:46:38 2017 +0100
> 
>     drm: Introduce drm_mode_object_{get,put}()
> 
> commit ad09360750afa18a0a0ce0253d6ea6033abc22e7
> Author: Thierry Reding <treding@nvidia.com>
> Date:   Tue Feb 28 15:46:39 2017 +0100
> 
>     drm: Introduce drm_connector_{get,put}()
> 
> commit a4a69da06bc11a937a6e417938b1bb698ee1fa46
> Author: Thierry Reding <treding@nvidia.com>
> Date:   Tue Feb 28 15:46:40 2017 +0100
> 
>     drm: Introduce drm_framebuffer_{get,put}()
> 
> commit e6b62714e87c8811d5564b6a0738dcde63a51774
> Author: Thierry Reding <treding@nvidia.com>
> Date:   Tue Feb 28 15:46:41 2017 +0100
> 
>     drm: Introduce drm_gem_object_{get,put}()
> 
> commit 6472e5090be7c78749a3c279b4faae87ab835c40
> Author: Thierry Reding <treding@nvidia.com>
> Date:   Tue Feb 28 15:46:42 2017 +0100
> 
>     drm: Introduce drm_property_blob_{get,put}()
> 
> Cihangir Akturk (28):
>   drm/amdgpu: switch to drm_*_get(), drm_*_put() helpers
>   drm: mali-dp: switch to drm_*_get(), drm_*_put() helpers
>   drm/armada: switch to drm_*_get(), drm_*_put() helpers
>   drm/ast: switch to drm_*_get(), drm_*_put() helpers
>   drm/bochs: switch to drm_*_get(), drm_*_put() helpers
>   drm/cirrus: switch to drm_*_get(), drm_*_put() helpers
>   drm/etnaviv: switch to drm_*_get(), drm_*_put() helpers
>   drm/exynos: switch to drm_*_get(), drm_*_put() helpers
>   drm/gma500: switch to drm_*_get(), drm_*_put() helpers
>   drm/hisilicon: switch to drm_*_get(), drm_*_put() helpers
>   drm/i915: switch to drm_*_get(), drm_*_put() helpers
>   drm/imx: switch to drm_*_get(), drm_*_put() helpers
>   drm/mediatek: switch to drm_*_get(), drm_*_put() helpers
>   drm/mgag200: switch to drm_*_get(), drm_*_put() helpers
>   drm/msm: switch to drm_*_get(), drm_*_put() helpers
>   drm/nouveau: switch to drm_*_get(), drm_*_put() helpers
>   drm/omapdrm: switch to drm_*_get(), drm_*_put() helpers
>   drm/qxl: switch to drm_*_get(), drm_*_put() helpers
>   drm/radeon: switch to drm_*_get(), drm_*_put() helpers
>   drm/rockchip: switch to drm_*_get(), drm_*_put() helpers
>   drm/tegra: switch to drm_*_get(), drm_*_put() helpers
>   drm/tilcdc: switch to drm_*_get(), drm_*_put() helpers
>   drm/udl: switch to drm_*_get(), drm_*_put() helpers
>   drm/vc4: switch to drm_*_get(), drm_*_put() helpers
>   drm/vgem: switch to drm_*_get(), drm_*_put() helpers
>   drm/virtio: switch to drm_*_get(), drm_*_put() helpers
>   drm/vmwgfx: switch to drm_*_get(), drm_*_put() helpers
>   drm: vboxvideo: switch to drm_*_get(), drm_*_put() helpers
> 
>  drivers/gpu/drm/amd/amdgpu/amdgpu_bo_list.c       |  2 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c            |  2 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_display.c       |  6 ++---
>  drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c            |  4 +--
>  drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c           | 22 ++++++++---------
>  drivers/gpu/drm/amd/amdgpu/dce_v10_0.c            |  6 ++---
>  drivers/gpu/drm/amd/amdgpu/dce_v11_0.c            |  6 ++---
>  drivers/gpu/drm/amd/amdgpu/dce_v6_0.c             |  6 ++---
>  drivers/gpu/drm/amd/amdgpu/dce_v8_0.c             |  6 ++---
>  drivers/gpu/drm/arm/malidp_planes.c               |  2 +-
>  drivers/gpu/drm/armada/armada_crtc.c              | 22 ++++++++---------
>  drivers/gpu/drm/armada/armada_drv.c               |  2 +-
>  drivers/gpu/drm/armada/armada_fb.c                |  8 +++---
>  drivers/gpu/drm/armada/armada_fbdev.c             |  6 ++---
>  drivers/gpu/drm/armada/armada_gem.c               | 14 +++++------
>  drivers/gpu/drm/armada/armada_overlay.c           |  4 +--
>  drivers/gpu/drm/ast/ast_fb.c                      |  2 +-
>  drivers/gpu/drm/ast/ast_main.c                    | 10 ++++----
>  drivers/gpu/drm/ast/ast_mode.c                    |  6 ++---
>  drivers/gpu/drm/bochs/bochs_fbdev.c               |  2 +-
>  drivers/gpu/drm/bochs/bochs_mm.c                  | 10 ++++----
>  drivers/gpu/drm/cirrus/cirrus_fbdev.c             |  2 +-
>  drivers/gpu/drm/cirrus/cirrus_main.c              | 10 ++++----
>  drivers/gpu/drm/etnaviv/etnaviv_drv.c             |  8 +++---
>  drivers/gpu/drm/etnaviv/etnaviv_gem.c             | 20 +++++++--------
>  drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c       |  2 +-
>  drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c      |  4 +--
>  drivers/gpu/drm/exynos/exynos_drm_fb.c            |  4 +--
>  drivers/gpu/drm/exynos/exynos_drm_gem.c           | 12 ++++-----
>  drivers/gpu/drm/exynos/exynos_drm_plane.c         |  2 +-
>  drivers/gpu/drm/gma500/framebuffer.c              |  4 +--
>  drivers/gpu/drm/gma500/gem.c                      |  4 +--
>  drivers/gpu/drm/gma500/gma_display.c              |  6 ++---
>  drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c |  4 +--
>  drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c       |  8 +++---
>  drivers/gpu/drm/i915/i915_gem_object.h            |  4 +--
>  drivers/gpu/drm/i915/intel_display.c              | 24 +++++++++---------
>  drivers/gpu/drm/i915/intel_dp_mst.c               |  2 +-
>  drivers/gpu/drm/i915/intel_fbdev.c                |  4 +--
>  drivers/gpu/drm/imx/ipuv3-crtc.c                  |  2 +-
>  drivers/gpu/drm/mediatek/mtk_drm_fb.c             |  4 +--
>  drivers/gpu/drm/mediatek/mtk_drm_gem.c            |  2 +-
>  drivers/gpu/drm/mgag200/mgag200_cursor.c          |  2 +-
>  drivers/gpu/drm/mgag200/mgag200_fb.c              |  4 +--
>  drivers/gpu/drm/mgag200/mgag200_main.c            | 10 ++++----
>  drivers/gpu/drm/msm/adreno/a5xx_gpu.c             | 10 ++++----
>  drivers/gpu/drm/msm/adreno/a5xx_power.c           |  2 +-
>  drivers/gpu/drm/msm/adreno/adreno_gpu.c           |  2 +-
>  drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c          |  6 ++---
>  drivers/gpu/drm/msm/mdp/mdp4/mdp4_kms.c           |  2 +-
>  drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c          |  2 +-
>  drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c         |  4 +--
>  drivers/gpu/drm/msm/msm_drv.c                     |  8 +++---
>  drivers/gpu/drm/msm/msm_fb.c                      |  4 +--
>  drivers/gpu/drm/msm/msm_fbdev.c                   |  2 +-
>  drivers/gpu/drm/msm/msm_gem.c                     |  8 +++---
>  drivers/gpu/drm/msm/msm_gem_submit.c              |  4 +--
>  drivers/gpu/drm/msm/msm_gpu.c                     |  4 +--
>  drivers/gpu/drm/msm/msm_ringbuffer.c              |  2 +-
>  drivers/gpu/drm/nouveau/dispnv04/crtc.c           |  2 +-
>  drivers/gpu/drm/nouveau/nouveau_abi16.c           |  2 +-
>  drivers/gpu/drm/nouveau/nouveau_display.c         |  8 +++---
>  drivers/gpu/drm/nouveau/nouveau_fbcon.c           |  2 +-
>  drivers/gpu/drm/nouveau/nouveau_gem.c             | 14 +++++------
>  drivers/gpu/drm/nouveau/nv50_display.c            |  2 +-
>  drivers/gpu/drm/omapdrm/omap_drv.c                |  2 +-
>  drivers/gpu/drm/omapdrm/omap_fb.c                 |  4 +--
>  drivers/gpu/drm/omapdrm/omap_fbdev.c              |  2 +-
>  drivers/gpu/drm/omapdrm/omap_gem.c                |  4 +--
>  drivers/gpu/drm/omapdrm/omap_gem_dmabuf.c         |  2 +-
>  drivers/gpu/drm/qxl/qxl_display.c                 |  4 +--
>  drivers/gpu/drm/qxl/qxl_dumb.c                    |  2 +-
>  drivers/gpu/drm/qxl/qxl_fb.c                      |  6 ++---
>  drivers/gpu/drm/qxl/qxl_gem.c                     |  2 +-
>  drivers/gpu/drm/qxl/qxl_ioctl.c                   |  4 +--
>  drivers/gpu/drm/qxl/qxl_object.c                  |  6 ++---
>  drivers/gpu/drm/radeon/radeon_cs.c                |  2 +-
>  drivers/gpu/drm/radeon/radeon_cursor.c            |  6 ++---
>  drivers/gpu/drm/radeon/radeon_display.c           | 12 ++++-----
>  drivers/gpu/drm/radeon/radeon_fb.c                |  4 +--
>  drivers/gpu/drm/radeon/radeon_gem.c               | 30 +++++++++++------------
>  drivers/gpu/drm/radeon/radeon_object.c            |  2 +-
>  drivers/gpu/drm/rockchip/rockchip_drm_fb.c        |  6 ++---
>  drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c     |  2 +-
>  drivers/gpu/drm/rockchip/rockchip_drm_gem.c       |  2 +-
>  drivers/gpu/drm/rockchip/rockchip_drm_vop.c       |  4 +--
>  drivers/gpu/drm/tegra/drm.c                       | 12 ++++-----
>  drivers/gpu/drm/tegra/fb.c                        |  8 +++---
>  drivers/gpu/drm/tegra/gem.c                       | 10 ++++----
>  drivers/gpu/drm/tilcdc/tilcdc_crtc.c              |  6 ++---
>  drivers/gpu/drm/udl/udl_fb.c                      |  6 ++---
>  drivers/gpu/drm/udl/udl_gem.c                     |  4 +--
>  drivers/gpu/drm/vc4/vc4_bo.c                      |  2 +-
>  drivers/gpu/drm/vgem/vgem_drv.c                   |  4 +--
>  drivers/gpu/drm/vgem/vgem_fence.c                 |  2 +-
>  drivers/gpu/drm/virtio/virtgpu_display.c          |  4 +--
>  drivers/gpu/drm/virtio/virtgpu_gem.c              |  4 +--
>  drivers/gpu/drm/virtio/virtgpu_ioctl.c            | 14 +++++------
>  drivers/gpu/drm/vmwgfx/vmwgfx_fb.c                |  2 +-
>  drivers/gpu/drm/vmwgfx/vmwgfx_ioctl.c             |  4 +--
>  drivers/staging/vboxvideo/vbox_fb.c               |  2 +-
>  drivers/staging/vboxvideo/vbox_main.c             |  8 +++---
>  drivers/staging/vboxvideo/vbox_mode.c             |  2 +-
>  103 files changed, 294 insertions(+), 294 deletions(-)
> 
> --
> 2.7.4
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v3 00/28] DRM API Conversions
  2017-08-11 14:24 ` [PATCH v3 00/28] DRM API Conversions Deucher, Alexander
@ 2017-08-11 15:21   ` Cihangir Akturk
  0 siblings, 0 replies; 5+ messages in thread
From: Cihangir Akturk @ 2017-08-11 15:21 UTC (permalink / raw)
  To: Deucher, Alexander
  Cc: devel@driverdev.osuosl.org, linux-arm-msm@vger.kernel.org,
	intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	dri-devel@lists.freedesktop.org, etnaviv@lists.freedesktop.org,
	amd-gfx@lists.freedesktop.org, nouveau@lists.freedesktop.org,
	linux-tegra@vger.kernel.org,
	virtualization@lists.linux-foundation.org,
	freedreno@lists.freedesktop.org

On Fri, Aug 11, 2017 at 02:24:19PM +0000, Deucher, Alexander wrote:
> > -----Original Message-----
> > From: amd-gfx [mailto:amd-gfx-bounces@lists.freedesktop.org] On Behalf
> > Of Cihangir Akturk
> > Sent: Friday, August 11, 2017 8:33 AM
> > Cc: devel@driverdev.osuosl.org; linux-arm-msm@vger.kernel.org; intel-
> > gfx@lists.freedesktop.org; linux-kernel@vger.kernel.org; dri-
> > devel@lists.freedesktop.org; etnaviv@lists.freedesktop.org; Cihangir Akturk;
> > amd-gfx@lists.freedesktop.org; daniel@ffwll.ch;
> > nouveau@lists.freedesktop.org; linux-tegra@vger.kernel.org;
> > virtualization@lists.linux-foundation.org; freedreno@lists.freedesktop.org
> > Subject: [PATCH v3 00/28] DRM API Conversions
> > 
> > Changes since v2:
> > 
> > - Patch series is based on *drm-misc-next* as suggested by Sean Paul.
> > 
> > - Dropped patch 05 (drm/atmel-hlcdc) and patch 25 (drm/vc4) from v2,
> >   since they were already pulled in the drm-misc-next
> > 
> > Changes since v1:
> > 
> > - This time patches were generated with coccinelle instead of my own
> >   script, as suggested by Daniel Vetter.
> > 
> > - Fixed the typo in commit messages. s/adn/and
> > 
> 
> FWIW, I already picked up v1 of these patches for radeon and amdgpu.

I think you can skip these patches of v3 for amdgpu and radeon, as
they have remained unchanged since v1.

> 
> Alex
> 
> > Note: I've included r-b, a-b tags, as these patches are identical to v1
> > except for the file: drivers/gpu/drm/i915/i915_gem_object.h
> > 
> > This patch set replaces the occurrences of drm_*_reference() and
> > drm_*_unreference() with the new drm_*_get() and drm_*_put()
> > functions.
> > All patches in the series do the same thing, converting to the new APIs.
> > I created patches per DRM driver as suggested by Daniel Vetter.
> > 
> > This patch set was generated by scripts/coccinelle/api/drm-get-put.cocci
> > 
> > Previous thread can be reached at:
> > https://marc.info/?l=dri-devel&m=150178288816047
> > 
> > Background:
> > 
> > In the kernel, reference counting APIs use *_get(), *_put() style naming
> > to reference-count the objects. But DRM subsystem uses a different
> > naming for them such as *_reference(), *_unreference() which is
> > inconsistent with the other reference counting APIs in the kernel. To
> > solve this consistency issue, Thierry Reding introduced a couple of
> > functions and compatibility aliases in the following commits for them.
> > 
> > commit 020a218f95bd3ceff7dd1022ff7ebc0497bc7bf9
> > Author: Thierry Reding <treding@nvidia.com>
> > Date:   Tue Feb 28 15:46:38 2017 +0100
> > 
> >     drm: Introduce drm_mode_object_{get,put}()
> > 
> > commit ad09360750afa18a0a0ce0253d6ea6033abc22e7
> > Author: Thierry Reding <treding@nvidia.com>
> > Date:   Tue Feb 28 15:46:39 2017 +0100
> > 
> >     drm: Introduce drm_connector_{get,put}()
> > 
> > commit a4a69da06bc11a937a6e417938b1bb698ee1fa46
> > Author: Thierry Reding <treding@nvidia.com>
> > Date:   Tue Feb 28 15:46:40 2017 +0100
> > 
> >     drm: Introduce drm_framebuffer_{get,put}()
> > 
> > commit e6b62714e87c8811d5564b6a0738dcde63a51774
> > Author: Thierry Reding <treding@nvidia.com>
> > Date:   Tue Feb 28 15:46:41 2017 +0100
> > 
> >     drm: Introduce drm_gem_object_{get,put}()
> > 
> > commit 6472e5090be7c78749a3c279b4faae87ab835c40
> > Author: Thierry Reding <treding@nvidia.com>
> > Date:   Tue Feb 28 15:46:42 2017 +0100
> > 
> >     drm: Introduce drm_property_blob_{get,put}()
> > 
> > Cihangir Akturk (28):
> >   drm/amdgpu: switch to drm_*_get(), drm_*_put() helpers
> >   drm: mali-dp: switch to drm_*_get(), drm_*_put() helpers
> >   drm/armada: switch to drm_*_get(), drm_*_put() helpers
> >   drm/ast: switch to drm_*_get(), drm_*_put() helpers
> >   drm/bochs: switch to drm_*_get(), drm_*_put() helpers
> >   drm/cirrus: switch to drm_*_get(), drm_*_put() helpers
> >   drm/etnaviv: switch to drm_*_get(), drm_*_put() helpers
> >   drm/exynos: switch to drm_*_get(), drm_*_put() helpers
> >   drm/gma500: switch to drm_*_get(), drm_*_put() helpers
> >   drm/hisilicon: switch to drm_*_get(), drm_*_put() helpers
> >   drm/i915: switch to drm_*_get(), drm_*_put() helpers
> >   drm/imx: switch to drm_*_get(), drm_*_put() helpers
> >   drm/mediatek: switch to drm_*_get(), drm_*_put() helpers
> >   drm/mgag200: switch to drm_*_get(), drm_*_put() helpers
> >   drm/msm: switch to drm_*_get(), drm_*_put() helpers
> >   drm/nouveau: switch to drm_*_get(), drm_*_put() helpers
> >   drm/omapdrm: switch to drm_*_get(), drm_*_put() helpers
> >   drm/qxl: switch to drm_*_get(), drm_*_put() helpers
> >   drm/radeon: switch to drm_*_get(), drm_*_put() helpers
> >   drm/rockchip: switch to drm_*_get(), drm_*_put() helpers
> >   drm/tegra: switch to drm_*_get(), drm_*_put() helpers
> >   drm/tilcdc: switch to drm_*_get(), drm_*_put() helpers
> >   drm/udl: switch to drm_*_get(), drm_*_put() helpers
> >   drm/vc4: switch to drm_*_get(), drm_*_put() helpers
> >   drm/vgem: switch to drm_*_get(), drm_*_put() helpers
> >   drm/virtio: switch to drm_*_get(), drm_*_put() helpers
> >   drm/vmwgfx: switch to drm_*_get(), drm_*_put() helpers
> >   drm: vboxvideo: switch to drm_*_get(), drm_*_put() helpers
> > 
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_bo_list.c       |  2 +-
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c            |  2 +-
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_display.c       |  6 ++---
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c            |  4 +--
> >  drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c           | 22 ++++++++---------
> >  drivers/gpu/drm/amd/amdgpu/dce_v10_0.c            |  6 ++---
> >  drivers/gpu/drm/amd/amdgpu/dce_v11_0.c            |  6 ++---
> >  drivers/gpu/drm/amd/amdgpu/dce_v6_0.c             |  6 ++---
> >  drivers/gpu/drm/amd/amdgpu/dce_v8_0.c             |  6 ++---
> >  drivers/gpu/drm/arm/malidp_planes.c               |  2 +-
> >  drivers/gpu/drm/armada/armada_crtc.c              | 22 ++++++++---------
> >  drivers/gpu/drm/armada/armada_drv.c               |  2 +-
> >  drivers/gpu/drm/armada/armada_fb.c                |  8 +++---
> >  drivers/gpu/drm/armada/armada_fbdev.c             |  6 ++---
> >  drivers/gpu/drm/armada/armada_gem.c               | 14 +++++------
> >  drivers/gpu/drm/armada/armada_overlay.c           |  4 +--
> >  drivers/gpu/drm/ast/ast_fb.c                      |  2 +-
> >  drivers/gpu/drm/ast/ast_main.c                    | 10 ++++----
> >  drivers/gpu/drm/ast/ast_mode.c                    |  6 ++---
> >  drivers/gpu/drm/bochs/bochs_fbdev.c               |  2 +-
> >  drivers/gpu/drm/bochs/bochs_mm.c                  | 10 ++++----
> >  drivers/gpu/drm/cirrus/cirrus_fbdev.c             |  2 +-
> >  drivers/gpu/drm/cirrus/cirrus_main.c              | 10 ++++----
> >  drivers/gpu/drm/etnaviv/etnaviv_drv.c             |  8 +++---
> >  drivers/gpu/drm/etnaviv/etnaviv_gem.c             | 20 +++++++--------
> >  drivers/gpu/drm/etnaviv/etnaviv_gem_prime.c       |  2 +-
> >  drivers/gpu/drm/etnaviv/etnaviv_gem_submit.c      |  4 +--
> >  drivers/gpu/drm/exynos/exynos_drm_fb.c            |  4 +--
> >  drivers/gpu/drm/exynos/exynos_drm_gem.c           | 12 ++++-----
> >  drivers/gpu/drm/exynos/exynos_drm_plane.c         |  2 +-
> >  drivers/gpu/drm/gma500/framebuffer.c              |  4 +--
> >  drivers/gpu/drm/gma500/gem.c                      |  4 +--
> >  drivers/gpu/drm/gma500/gma_display.c              |  6 ++---
> >  drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c |  4 +--
> >  drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c       |  8 +++---
> >  drivers/gpu/drm/i915/i915_gem_object.h            |  4 +--
> >  drivers/gpu/drm/i915/intel_display.c              | 24 +++++++++---------
> >  drivers/gpu/drm/i915/intel_dp_mst.c               |  2 +-
> >  drivers/gpu/drm/i915/intel_fbdev.c                |  4 +--
> >  drivers/gpu/drm/imx/ipuv3-crtc.c                  |  2 +-
> >  drivers/gpu/drm/mediatek/mtk_drm_fb.c             |  4 +--
> >  drivers/gpu/drm/mediatek/mtk_drm_gem.c            |  2 +-
> >  drivers/gpu/drm/mgag200/mgag200_cursor.c          |  2 +-
> >  drivers/gpu/drm/mgag200/mgag200_fb.c              |  4 +--
> >  drivers/gpu/drm/mgag200/mgag200_main.c            | 10 ++++----
> >  drivers/gpu/drm/msm/adreno/a5xx_gpu.c             | 10 ++++----
> >  drivers/gpu/drm/msm/adreno/a5xx_power.c           |  2 +-
> >  drivers/gpu/drm/msm/adreno/adreno_gpu.c           |  2 +-
> >  drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c          |  6 ++---
> >  drivers/gpu/drm/msm/mdp/mdp4/mdp4_kms.c           |  2 +-
> >  drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c          |  2 +-
> >  drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c         |  4 +--
> >  drivers/gpu/drm/msm/msm_drv.c                     |  8 +++---
> >  drivers/gpu/drm/msm/msm_fb.c                      |  4 +--
> >  drivers/gpu/drm/msm/msm_fbdev.c                   |  2 +-
> >  drivers/gpu/drm/msm/msm_gem.c                     |  8 +++---
> >  drivers/gpu/drm/msm/msm_gem_submit.c              |  4 +--
> >  drivers/gpu/drm/msm/msm_gpu.c                     |  4 +--
> >  drivers/gpu/drm/msm/msm_ringbuffer.c              |  2 +-
> >  drivers/gpu/drm/nouveau/dispnv04/crtc.c           |  2 +-
> >  drivers/gpu/drm/nouveau/nouveau_abi16.c           |  2 +-
> >  drivers/gpu/drm/nouveau/nouveau_display.c         |  8 +++---
> >  drivers/gpu/drm/nouveau/nouveau_fbcon.c           |  2 +-
> >  drivers/gpu/drm/nouveau/nouveau_gem.c             | 14 +++++------
> >  drivers/gpu/drm/nouveau/nv50_display.c            |  2 +-
> >  drivers/gpu/drm/omapdrm/omap_drv.c                |  2 +-
> >  drivers/gpu/drm/omapdrm/omap_fb.c                 |  4 +--
> >  drivers/gpu/drm/omapdrm/omap_fbdev.c              |  2 +-
> >  drivers/gpu/drm/omapdrm/omap_gem.c                |  4 +--
> >  drivers/gpu/drm/omapdrm/omap_gem_dmabuf.c         |  2 +-
> >  drivers/gpu/drm/qxl/qxl_display.c                 |  4 +--
> >  drivers/gpu/drm/qxl/qxl_dumb.c                    |  2 +-
> >  drivers/gpu/drm/qxl/qxl_fb.c                      |  6 ++---
> >  drivers/gpu/drm/qxl/qxl_gem.c                     |  2 +-
> >  drivers/gpu/drm/qxl/qxl_ioctl.c                   |  4 +--
> >  drivers/gpu/drm/qxl/qxl_object.c                  |  6 ++---
> >  drivers/gpu/drm/radeon/radeon_cs.c                |  2 +-
> >  drivers/gpu/drm/radeon/radeon_cursor.c            |  6 ++---
> >  drivers/gpu/drm/radeon/radeon_display.c           | 12 ++++-----
> >  drivers/gpu/drm/radeon/radeon_fb.c                |  4 +--
> >  drivers/gpu/drm/radeon/radeon_gem.c               | 30 +++++++++++------------
> >  drivers/gpu/drm/radeon/radeon_object.c            |  2 +-
> >  drivers/gpu/drm/rockchip/rockchip_drm_fb.c        |  6 ++---
> >  drivers/gpu/drm/rockchip/rockchip_drm_fbdev.c     |  2 +-
> >  drivers/gpu/drm/rockchip/rockchip_drm_gem.c       |  2 +-
> >  drivers/gpu/drm/rockchip/rockchip_drm_vop.c       |  4 +--
> >  drivers/gpu/drm/tegra/drm.c                       | 12 ++++-----
> >  drivers/gpu/drm/tegra/fb.c                        |  8 +++---
> >  drivers/gpu/drm/tegra/gem.c                       | 10 ++++----
> >  drivers/gpu/drm/tilcdc/tilcdc_crtc.c              |  6 ++---
> >  drivers/gpu/drm/udl/udl_fb.c                      |  6 ++---
> >  drivers/gpu/drm/udl/udl_gem.c                     |  4 +--
> >  drivers/gpu/drm/vc4/vc4_bo.c                      |  2 +-
> >  drivers/gpu/drm/vgem/vgem_drv.c                   |  4 +--
> >  drivers/gpu/drm/vgem/vgem_fence.c                 |  2 +-
> >  drivers/gpu/drm/virtio/virtgpu_display.c          |  4 +--
> >  drivers/gpu/drm/virtio/virtgpu_gem.c              |  4 +--
> >  drivers/gpu/drm/virtio/virtgpu_ioctl.c            | 14 +++++------
> >  drivers/gpu/drm/vmwgfx/vmwgfx_fb.c                |  2 +-
> >  drivers/gpu/drm/vmwgfx/vmwgfx_ioctl.c             |  4 +--
> >  drivers/staging/vboxvideo/vbox_fb.c               |  2 +-
> >  drivers/staging/vboxvideo/vbox_main.c             |  8 +++---
> >  drivers/staging/vboxvideo/vbox_mode.c             |  2 +-
> >  103 files changed, 294 insertions(+), 294 deletions(-)
> > 
> > --
> > 2.7.4
> > 
> > _______________________________________________
> > amd-gfx mailing list
> > amd-gfx@lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH v3 21/28] drm/tegra: switch to drm_*_get(), drm_*_put() helpers
       [not found]   ` <1502454794-28558-22-git-send-email-cakturk-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
@ 2017-08-17 15:33     ` Thierry Reding
  0 siblings, 0 replies; 5+ messages in thread
From: Thierry Reding @ 2017-08-17 15:33 UTC (permalink / raw)
  To: Cihangir Akturk
  Cc: daniel-/w4YWyX8dFk, linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW, David Airlie,
	Jonathan Hunter, linux-tegra-u79uwXL29TY76Z2rM5mHXA

[-- Attachment #1: Type: text/plain, Size: 795 bytes --]

On Fri, Aug 11, 2017 at 03:33:07PM +0300, Cihangir Akturk wrote:
> Use drm_*_get() and drm_*_put() helpers instead of drm_*_reference()
> and drm_*_unreference() helpers.
> 
> drm_*_reference() and drm_*_unreference() functions are just
> compatibility alias for drm_*_get() and drm_*_put() and should not be
> used by new code. So convert all users of compatibility functions to
> use the new APIs.
> 
> Generated by: scripts/coccinelle/api/drm-get-put.cocci
> 
> Signed-off-by: Cihangir Akturk <cakturk-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
>  drivers/gpu/drm/tegra/drm.c | 12 ++++++------
>  drivers/gpu/drm/tegra/fb.c  |  8 ++++----
>  drivers/gpu/drm/tegra/gem.c | 10 +++++-----
>  3 files changed, 15 insertions(+), 15 deletions(-)

Applied, thanks.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-08-17 15:33 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-08-11 12:32 [PATCH v3 00/28] DRM API Conversions Cihangir Akturk
2017-08-11 12:33 ` [PATCH v3 21/28] drm/tegra: switch to drm_*_get(), drm_*_put() helpers Cihangir Akturk
     [not found]   ` <1502454794-28558-22-git-send-email-cakturk-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-08-17 15:33     ` Thierry Reding
2017-08-11 14:24 ` [PATCH v3 00/28] DRM API Conversions Deucher, Alexander
2017-08-11 15:21   ` Cihangir Akturk

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).