RE: [PATCH v3 00/28] DRM API Conversions
> -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Cihangir Akturk > Sent: Friday, August 11, 2017 8:33 AM > Cc: de...@driverdev.osuosl.org; linux-arm-...@vger.kernel.org; intel- > g...@lists.freedesktop.org; linux-ker...@vger.kernel.org; dri- > de...@lists.freedesktop.org; etna...@lists.freedesktop.org; Cihangir Akturk; > amd-...@lists.freedesktop.org; dan...@ffwll.ch; > nouv...@lists.freedesktop.org; linux-te...@vger.kernel.org; > virtualizat...@lists.linux-foundation.org; freedr...@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=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 <tred...@nvidia.com> > Date: Tue Feb 28 15:46:38 2017 +0100 > > drm: Introduce drm_mode_object_{get,put}() > > commit ad09360750afa18a0a0ce0253d6ea6033abc22e7 > Author: Thierry Reding <tred...@nvidia.com> > Date: Tue Feb 28 15:46:39 2017 +0100 > > drm: Introduce drm_connector_{get,put}() > > commit a4a69da06bc11a937a6e417938b1bb698ee1fa46 > Author: Thierry Reding <tred...@nvidia.com> > Date: Tue Feb 28 15:46:40 2017 +0100 > > drm: Introduce drm_framebuffer_{get,put}() > > commit e6b62714e87c8811d5564b6a0738dcde63a51774 > Author: Thierry Reding <tred...@nvidia.com> > Date: Tue Feb 28 15:46:41 2017 +0100 > > drm: Introduce drm_gem_object_{get,put}() > > commit 6472e5090be7c78749a3c279b4faae87ab835c40 > Author: Thierry Reding <tred...@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_*
Re: [PATCH v3 00/28] DRM API Conversions
On Fri, Aug 11, 2017 at 02:24:19PM +, Deucher, Alexander wrote: > > -Original Message- > > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > > Of Cihangir Akturk > > Sent: Friday, August 11, 2017 8:33 AM > > Cc: de...@driverdev.osuosl.org; linux-arm-...@vger.kernel.org; intel- > > g...@lists.freedesktop.org; linux-ker...@vger.kernel.org; dri- > > de...@lists.freedesktop.org; etna...@lists.freedesktop.org; Cihangir Akturk; > > amd-...@lists.freedesktop.org; dan...@ffwll.ch; > > nouv...@lists.freedesktop.org; linux-te...@vger.kernel.org; > > virtualizat...@lists.linux-foundation.org; freedr...@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=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 <tred...@nvidia.com> > > Date: Tue Feb 28 15:46:38 2017 +0100 > > > > drm: Introduce drm_mode_object_{get,put}() > > > > commit ad09360750afa18a0a0ce0253d6ea6033abc22e7 > > Author: Thierry Reding <tred...@nvidia.com> > > Date: Tue Feb 28 15:46:39 2017 +0100 > > > > drm: Introduce drm_connector_{get,put}() > > > > commit a4a69da06bc11a937a6e417938b1bb698ee1fa46 > > Author: Thierry Reding <tred...@nvidia.com> > > Date: Tue Feb 28 15:46:40 2017 +0100 > > > > drm: Introduce drm_framebuffer_{get,put}() > > > > commit e6b62714e87c8811d5564b6a0738dcde63a51774 > > Author: Thierry Reding <tred...@nvidia.com> > > Date: Tue Feb 28 15:46:41 2017 +0100 > > > > drm: Introduce drm_gem_object_{get,put}() > > > > commit 6472e5090be7c78749a3c279b4faae87ab835c40 > > Author: Thierry Reding <tred...@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_*_
[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 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=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 RedingDate: Tue Feb 28 15:46:38 2017 +0100 drm: Introduce drm_mode_object_{get,put}() commit ad09360750afa18a0a0ce0253d6ea6033abc22e7 Author: Thierry Reding Date: Tue Feb 28 15:46:39 2017 +0100 drm: Introduce drm_connector_{get,put}() commit a4a69da06bc11a937a6e417938b1bb698ee1fa46 Author: Thierry Reding Date: Tue Feb 28 15:46:40 2017 +0100 drm: Introduce drm_framebuffer_{get,put}() commit e6b62714e87c8811d5564b6a0738dcde63a51774 Author: Thierry Reding Date: Tue Feb 28 15:46:41 2017 +0100 drm: Introduce drm_gem_object_{get,put}() commit 6472e5090be7c78749a3c279b4faae87ab835c40 Author: Thierry Reding 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