mbox series

[00/28] drm: Remove DRM aperture helpers

Message ID 20240930130921.689876-1-tzimmermann@suse.de (mailing list archive)
Headers show
Series drm: Remove DRM aperture helpers | expand

Message

Thomas Zimmermann Sept. 30, 2024, 1:02 p.m. UTC 
DRM's aperture helpers are merely wrappers around the respective
helpers in drivers/video/. The DRM interface existed first, but was
later generalized for use with fbdev and a few other graphics-related
places in the kernel. It is time to remove the DRM wrappers and call
the video helpers directly.

No functional changes intended. This series should replace one interface
with another. Individual patches could go into the various trees and the
DRM wrappers can be removed when all driver changes have been merged.

Thomas Zimmermann (28):
  drm/amdgpu: Use video aperture helpers
  drm/arm/hdlcd: Use video aperture helpers
  drm/armada: Use video aperture helpers
  drm/ast: Use video aperture helpers
  drm/hisilicon/hibmc: Use video aperture helpers
  drm/hyperv-drm: Use video aperture helpers
  drm/i915: Use video aperture helpers
  drm/loongson: Use video aperture helpers
  drm/meson: Use video aperture helpers
  drm/mgag200: Use video aperture helpers
  drm/msm: Use video aperture helpers
  drm/nouveau: Use video aperture helpers
  drm/ofdrm: Use video aperture helpers
  drm/qxl: Use video aperture helpers
  drm/radeon: Use video aperture helpers
  drm/rockchip: Use video aperture helpers
  drm/simpledrm: Use video aperture helpers
  drm/stm: Use video aperture helpers
  drm/sun4i: Use video aperture helpers
  drm/tegra: Use video aperture helpers
  drm/bochs: Use video aperture helpers
  drm/cirrus: Use video aperture helpers
  drm/vboxvideo: Use video aperture helpers
  drm/vc4: Use video aperture helpers
  drm/virtgpu: Use video aperture helpers
  drm/vmwgfx: Use video aperture helpers
  drm/xe: Use video aperture helpers
  drm: Remove DRM aperture helpers

 Documentation/gpu/drm-internals.rst           |  12 --
 MAINTAINERS                                   |   2 -
 drivers/gpu/drm/Makefile                      |   1 -
 drivers/gpu/drm/amd/amdgpu/amdgpu_device.c    |   5 +-
 drivers/gpu/drm/arm/hdlcd_drv.c               |   4 +-
 drivers/gpu/drm/armada/armada_drv.c           |   4 +-
 drivers/gpu/drm/ast/ast_drv.c                 |   4 +-
 drivers/gpu/drm/drm_aperture.c                | 192 ------------------
 .../gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c   |   4 +-
 drivers/gpu/drm/hyperv/hyperv_drm_drv.c       |   4 +-
 drivers/gpu/drm/i915/i915_driver.c            |   4 +-
 drivers/gpu/drm/loongson/lsdc_drv.c           |   8 +-
 drivers/gpu/drm/meson/meson_drv.c             |   4 +-
 drivers/gpu/drm/mgag200/mgag200_drv.c         |   4 +-
 drivers/gpu/drm/msm/msm_kms.c                 |   4 +-
 drivers/gpu/drm/nouveau/nouveau_drm.c         |   4 +-
 drivers/gpu/drm/qxl/qxl_drv.c                 |   4 +-
 drivers/gpu/drm/radeon/radeon_drv.c           |   5 +-
 drivers/gpu/drm/rockchip/rockchip_drm_drv.c   |   4 +-
 drivers/gpu/drm/stm/drv.c                     |   4 +-
 drivers/gpu/drm/sun4i/sun4i_drv.c             |   4 +-
 drivers/gpu/drm/tegra/drm.c                   |   4 +-
 drivers/gpu/drm/tiny/bochs.c                  |   4 +-
 drivers/gpu/drm/tiny/cirrus.c                 |   4 +-
 drivers/gpu/drm/tiny/ofdrm.c                  |   4 +-
 drivers/gpu/drm/tiny/simpledrm.c              |   8 +-
 drivers/gpu/drm/vboxvideo/vbox_drv.c          |   5 +-
 drivers/gpu/drm/vc4/vc4_drv.c                 |   4 +-
 drivers/gpu/drm/virtio/virtgpu_drv.c          |   4 +-
 drivers/gpu/drm/vmwgfx/vmwgfx_drv.c           |   5 +-
 drivers/gpu/drm/xe/xe_device.c                |   4 +-
 include/drm/drm_aperture.h                    |  38 ----
 32 files changed, 62 insertions(+), 303 deletions(-)
 delete mode 100644 drivers/gpu/drm/drm_aperture.c
 delete mode 100644 include/drm/drm_aperture.h

Comments

Javier Martinez Canillas Sept. 30, 2024, 1:26 p.m. UTC | #1 
Thomas Zimmermann <tzimmermann@suse.de> writes:

Hello Thomas,

> DRM's aperture helpers are merely wrappers around the respective
> helpers in drivers/video/. The DRM interface existed first, but was
> later generalized for use with fbdev and a few other graphics-related
> places in the kernel. It is time to remove the DRM wrappers and call
> the video helpers directly.
>
> No functional changes intended. This series should replace one interface
> with another. Individual patches could go into the various trees and the
> DRM wrappers can be removed when all driver changes have been merged.
>

Thanks for doing this cleanup. Makes a lot of sense for me to drop it.

For the whole series:

Acked-by: Javier Martinez Canillas <javierm@redhat.com>
Dmitry Baryshkov Oct. 21, 2024, 12:17 p.m. UTC | #2 
On Mon, 30 Sep 2024 15:02:58 +0200, Thomas Zimmermann wrote:
> DRM's aperture helpers are merely wrappers around the respective
> helpers in drivers/video/. The DRM interface existed first, but was
> later generalized for use with fbdev and a few other graphics-related
> places in the kernel. It is time to remove the DRM wrappers and call
> the video helpers directly.
> 
> No functional changes intended. This series should replace one interface
> with another. Individual patches could go into the various trees and the
> DRM wrappers can be removed when all driver changes have been merged.
> 
> [...]

Applied, thanks!

[11/28] drm/msm: Use video aperture helpers
        https://gitlab.freedesktop.org/lumag/msm/-/commit/42170670ee76

Best regards,