Re: [PATCH RFC v1 00/52] drm/crtc: Rename struct drm_crtc::dev to drm_dev

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi

Am 12.07.23 um 11:46 schrieb Uwe Kleine-König:
Hello,

while I debugged an issue in the imx-lcdc driver I was constantly
irritated about struct drm_device pointer variables being named "dev"
because with that name I usually expect a struct device pointer.

Rather than renaming dev in all the DRM structs, did you consider renaming struct drm_device.dev instead?

Everyone in DRM-land knows that 'dev' is the DRM device. But for struct drm_device.dev a more expressive name would be helpful. Maybe 'parent'. (It's also much less churn.)

Best regards
Thomas


I think there is a big benefit when these are all renamed to "drm_dev".
I have no strong preference here though, so "drmdev" or "drm" are fine
for me, too. Let the bikesheding begin!

Some statistics:

$ git grep -ohE 'struct drm_device *\* *[^ (),;]*' v6.5-rc1 | sort | uniq -c | sort -n
       1 struct drm_device *adev_to_drm
       1 struct drm_device *drm_
       1 struct drm_device          *drm_dev
       1 struct drm_device        *drm_dev
       1 struct drm_device *pdev
       1 struct drm_device *rdev
       1 struct drm_device *vdev
       2 struct drm_device *dcss_drv_dev_to_drm
       2 struct drm_device **ddev
       2 struct drm_device *drm_dev_alloc
       2 struct drm_device *mock
       2 struct drm_device *p_ddev
       5 struct drm_device *device
       9 struct drm_device * dev
      25 struct drm_device *d
      95 struct drm_device *
     216 struct drm_device *ddev
     234 struct drm_device *drm_dev
     611 struct drm_device *drm
    4190 struct drm_device *dev

This series starts with renaming struct drm_crtc::dev to drm_dev. If
it's not only me and others like the result of this effort it should be
followed up by adapting the other structs and the individual usages in
the different drivers.

To make this series a bit easier handleable, I first added an alias for
drm_crtc::dev, then converted the drivers one after another and the last
patch drops the "dev" name. This has the advantage of being easier to
review, and if I should have missed an instance only the last patch must
be dropped/reverted. Also this series might conflict with other patches,
in this case the remaining patches can still go in (apart from the last
one of course). Maybe it also makes sense to delay applying the last
patch by one development cycle?

The series was compile tested for arm, arm64, powerpc and amd64 using an
allmodconfig (though I only build drivers/gpu/).

Best regards
Uwe

Uwe Kleine-König (52):
   drm/crtc: Start renaming struct drm_crtc::dev to drm_dev
   drm/core: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/amd: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/armada: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/arm: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/aspeed: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/ast: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/atmel-hlcdc: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/exynos: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/fsl-dcu: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/gma500: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/gud: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/hisilicon: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/hyperv: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/i915: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/imx: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/ingenic: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/kmb: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/logicvc: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/mcde: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/mediatek: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/meson: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/mgag200: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/msm: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/mxsfb: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/nouveau: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/omapdrm: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/panel-ili9341: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/pl111: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/qxl: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/radeon: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/renesas: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/rockchip: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/solomon: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/sprd: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/sti: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/stm: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/sun4i: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/tegra: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/tidss: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/tilcdc: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/tiny: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/tve200: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/udl: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/vboxvideo: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/vc4: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/virtio: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/vkms: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/vmwgfx: Use struct drm_crtc::drm_dev instead of struct
     drm_crtc::dev
   drm/xen: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/xlnx: Use struct drm_crtc::drm_dev instead of struct drm_crtc::dev
   drm/crtc: Complete renaming struct drm_crtc::dev to drm_dev

  drivers/gpu/drm/amd/amdgpu/amdgpu_display.c   |  18 +-
  drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c       |   6 +-
  drivers/gpu/drm/amd/amdgpu/amdgpu_pll.c       |   6 +-
  drivers/gpu/drm/amd/amdgpu/amdgpu_vkms.c      |   8 +-
  drivers/gpu/drm/amd/amdgpu/atombios_crtc.c    |  22 +--
  drivers/gpu/drm/amd/amdgpu/dce_v10_0.c        |  26 +--
  drivers/gpu/drm/amd/amdgpu/dce_v11_0.c        |  28 ++--
  drivers/gpu/drm/amd/amdgpu/dce_v6_0.c         |  26 +--
  drivers/gpu/drm/amd/amdgpu/dce_v8_0.c         |  26 +--
  .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c |  29 ++--
  .../drm/amd/display/amdgpu_dm/amdgpu_dm_crc.c |  20 +--
  .../amd/display/amdgpu_dm/amdgpu_dm_crtc.c    |   8 +-
  .../amd/display/amdgpu_dm/amdgpu_dm_debugfs.c |  22 +--
  .../amd/display/amdgpu_dm/amdgpu_dm_plane.c   |   2 +-
  .../gpu/drm/arm/display/komeda/komeda_crtc.c  |  24 +--
  .../gpu/drm/arm/display/komeda/komeda_kms.c   |   2 +-
  drivers/gpu/drm/arm/hdlcd_crtc.c              |   4 +-
  drivers/gpu/drm/arm/malidp_crtc.c             |   7 +-
  drivers/gpu/drm/armada/armada_crtc.c          |  10 +-
  drivers/gpu/drm/aspeed/aspeed_gfx_crtc.c      |   6 +-
  drivers/gpu/drm/ast/ast_dp.c                  |   2 +-
  drivers/gpu/drm/ast/ast_mode.c                |  26 +--
  .../gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c    |  10 +-
  drivers/gpu/drm/drm_atomic.c                  |  22 +--
  drivers/gpu/drm/drm_atomic_helper.c           |  20 ++-
  drivers/gpu/drm/drm_atomic_state_helper.c     |   2 +-
  drivers/gpu/drm/drm_atomic_uapi.c             |  22 +--
  drivers/gpu/drm/drm_blend.c                   |   2 +-
  drivers/gpu/drm/drm_color_mgmt.c              |  10 +-
  drivers/gpu/drm/drm_crtc.c                    |  19 ++-
  drivers/gpu/drm/drm_crtc_helper.c             |  10 +-
  drivers/gpu/drm/drm_debugfs.c                 |   2 +-
  drivers/gpu/drm/drm_debugfs_crc.c             |   2 +-
  drivers/gpu/drm/drm_fb_helper.c               |   6 +-
  drivers/gpu/drm/drm_mipi_dbi.c                |   4 +-
  drivers/gpu/drm/drm_plane.c                   |   2 +-
  drivers/gpu/drm/drm_plane_helper.c            |   2 +-
  drivers/gpu/drm/drm_self_refresh_helper.c     |   2 +-
  drivers/gpu/drm/drm_vblank.c                  |  40 ++---
  drivers/gpu/drm/drm_vblank_work.c             |   2 +-
  drivers/gpu/drm/exynos/exynos_drm_crtc.c      |   8 +-
  drivers/gpu/drm/exynos/exynos_drm_plane.c     |   4 +-
  drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c    |  16 +-
  drivers/gpu/drm/gma500/cdv_intel_display.c    |   2 +-
  drivers/gpu/drm/gma500/cdv_intel_dp.c         |   2 +-
  drivers/gpu/drm/gma500/gma_display.c          |  20 +--
  drivers/gpu/drm/gma500/oaktrail_crtc.c        |   8 +-
  drivers/gpu/drm/gma500/oaktrail_hdmi.c        |   4 +-
  drivers/gpu/drm/gma500/psb_intel_display.c    |   2 +-
  drivers/gpu/drm/gma500/psb_irq.c              |   6 +-
  drivers/gpu/drm/gud/gud_pipe.c                |   6 +-
  .../gpu/drm/hisilicon/hibmc/hibmc_drm_de.c    |  20 +--
  .../gpu/drm/hisilicon/kirin/kirin_drm_ade.c   |   4 +-
  drivers/gpu/drm/hyperv/hyperv_drm_modeset.c   |   6 +-
  drivers/gpu/drm/i915/display/g4x_dp.c         |   4 +-
  drivers/gpu/drm/i915/display/hsw_ips.c        |  16 +-
  drivers/gpu/drm/i915/display/i9xx_plane.c     |   4 +-
  drivers/gpu/drm/i915/display/i9xx_wm.c        |  40 ++---
  drivers/gpu/drm/i915/display/icl_dsi.c        |   2 +-
  drivers/gpu/drm/i915/display/intel_atomic.c   |   2 +-
  .../gpu/drm/i915/display/intel_atomic_plane.c |   4 +-
  drivers/gpu/drm/i915/display/intel_audio.c    |   2 +-
  drivers/gpu/drm/i915/display/intel_bw.c       |  10 +-
  drivers/gpu/drm/i915/display/intel_cdclk.c    |   6 +-
  drivers/gpu/drm/i915/display/intel_color.c    | 124 +++++++-------
  drivers/gpu/drm/i915/display/intel_crtc.c     |  20 +--
  .../drm/i915/display/intel_crtc_state_dump.c  |   4 +-
  drivers/gpu/drm/i915/display/intel_cursor.c   |   2 +-
  drivers/gpu/drm/i915/display/intel_ddi.c      |  28 ++--
  drivers/gpu/drm/i915/display/intel_display.c  | 154 +++++++++---------
  .../gpu/drm/i915/display/intel_display_irq.c  |  22 +--
  .../gpu/drm/i915/display/intel_display_rps.c  |   2 +-
  .../drm/i915/display/intel_display_trace.h    |  12 +-
  drivers/gpu/drm/i915/display/intel_dp.c       |   2 +-
  drivers/gpu/drm/i915/display/intel_dpll.c     |  38 ++---
  drivers/gpu/drm/i915/display/intel_dpll_mgr.c |  44 ++---
  drivers/gpu/drm/i915/display/intel_dpt.c      |   2 +-
  drivers/gpu/drm/i915/display/intel_drrs.c     |  10 +-
  drivers/gpu/drm/i915/display/intel_dsb.c      |   8 +-
  drivers/gpu/drm/i915/display/intel_fbc.c      |   2 +-
  drivers/gpu/drm/i915/display/intel_fdi.c      |  22 +--
  .../drm/i915/display/intel_fifo_underrun.c    |   6 +-
  drivers/gpu/drm/i915/display/intel_hdmi.c     |   2 +-
  .../drm/i915/display/intel_modeset_setup.c    |  22 +--
  .../drm/i915/display/intel_modeset_verify.c   |   2 +-
  drivers/gpu/drm/i915/display/intel_panel.c    |   4 +-
  .../gpu/drm/i915/display/intel_pch_display.c  |  32 ++--
  .../gpu/drm/i915/display/intel_pch_refclk.c   |   2 +-
  drivers/gpu/drm/i915/display/intel_pipe_crc.c |  10 +-
  .../drm/i915/display/intel_plane_initial.c    |   6 +-
  drivers/gpu/drm/i915/display/intel_psr.c      |  14 +-
  drivers/gpu/drm/i915/display/intel_sdvo.c     |   2 +-
  drivers/gpu/drm/i915/display/intel_vblank.c   |  24 +--
  drivers/gpu/drm/i915/display/intel_vdsc.c     |  18 +-
  drivers/gpu/drm/i915/display/intel_vrr.c      |  18 +-
  drivers/gpu/drm/i915/display/skl_scaler.c     |  10 +-
  .../drm/i915/display/skl_universal_plane.c    |   6 +-
  drivers/gpu/drm/i915/display/skl_watermark.c  |  42 ++---
  drivers/gpu/drm/i915/display/vlv_dsi.c        |   2 +-
  drivers/gpu/drm/imx/dcss/dcss-crtc.c          |  20 +--
  drivers/gpu/drm/imx/ipuv3/ipuv3-crtc.c        |  15 +-
  drivers/gpu/drm/imx/lcdc/imx-lcdc.c           |  16 +-
  drivers/gpu/drm/ingenic/ingenic-drm-drv.c     |   4 +-
  drivers/gpu/drm/kmb/kmb_crtc.c                |  16 +-
  drivers/gpu/drm/logicvc/logicvc_crtc.c        |  14 +-
  drivers/gpu/drm/mcde/mcde_display.c           |  18 +-
  drivers/gpu/drm/mediatek/mtk_drm_crtc.c       |  22 +--
  drivers/gpu/drm/meson/meson_crtc.c            |  12 +-
  drivers/gpu/drm/mgag200/mgag200_g200.c        |   4 +-
  drivers/gpu/drm/mgag200/mgag200_g200eh.c      |   2 +-
  drivers/gpu/drm/mgag200/mgag200_g200er.c      |   4 +-
  drivers/gpu/drm/mgag200/mgag200_g200ev.c      |   4 +-
  drivers/gpu/drm/mgag200/mgag200_g200se.c      |   6 +-
  drivers/gpu/drm/mgag200/mgag200_g200wb.c      |   2 +-
  drivers/gpu/drm/mgag200/mgag200_mode.c        |  10 +-
  drivers/gpu/drm/msm/disp/dpu1/dpu_core_perf.c |   6 +-
  drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c      |  70 ++++----
  drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c       |   2 +-
  drivers/gpu/drm/msm/disp/mdp4/mdp4_crtc.c     |  12 +-
  drivers/gpu/drm/msm/disp/mdp5/mdp5_crtc.c     |  20 +--
  drivers/gpu/drm/msm/msm_drv.c                 |   4 +-
  drivers/gpu/drm/mxsfb/lcdif_kms.c             |  18 +-
  drivers/gpu/drm/mxsfb/mxsfb_kms.c             |  16 +-
  drivers/gpu/drm/nouveau/dispnv04/crtc.c       |  58 +++----
  drivers/gpu/drm/nouveau/dispnv04/cursor.c     |  10 +-
  drivers/gpu/drm/nouveau/dispnv50/atom.h       |   2 +-
  drivers/gpu/drm/nouveau/dispnv50/crc.c        |  30 ++--
  drivers/gpu/drm/nouveau/dispnv50/crc907d.c    |   6 +-
  drivers/gpu/drm/nouveau/dispnv50/crcc37d.c    |   6 +-
  drivers/gpu/drm/nouveau/dispnv50/crcc57d.c    |   2 +-
  drivers/gpu/drm/nouveau/dispnv50/disp.c       |   5 +-
  drivers/gpu/drm/nouveau/dispnv50/head.c       |   4 +-
  drivers/gpu/drm/nouveau/dispnv50/head507d.c   |  26 +--
  drivers/gpu/drm/nouveau/dispnv50/head827d.c   |  10 +-
  drivers/gpu/drm/nouveau/dispnv50/head907d.c   |  26 +--
  drivers/gpu/drm/nouveau/dispnv50/head917d.c   |   6 +-
  drivers/gpu/drm/nouveau/dispnv50/headc37d.c   |  18 +-
  drivers/gpu/drm/nouveau/dispnv50/headc57d.c   |  10 +-
  drivers/gpu/drm/nouveau/nouveau_connector.h   |   2 +-
  drivers/gpu/drm/nouveau/nouveau_display.c     |   2 +-
  drivers/gpu/drm/omapdrm/omap_crtc.c           |  56 +++----
  drivers/gpu/drm/omapdrm/omap_irq.c            |   6 +-
  drivers/gpu/drm/panel/panel-ilitek-ili9341.c  |   4 +-
  drivers/gpu/drm/pl111/pl111_display.c         |  16 +-
  drivers/gpu/drm/qxl/qxl_display.c             |   2 +-
  drivers/gpu/drm/radeon/atombios_crtc.c        |  54 +++---
  drivers/gpu/drm/radeon/radeon_cursor.c        |  14 +-
  drivers/gpu/drm/radeon/radeon_display.c       |  28 ++--
  drivers/gpu/drm/radeon/radeon_kms.c           |   6 +-
  drivers/gpu/drm/radeon/radeon_legacy_crtc.c   |  16 +-
  .../gpu/drm/renesas/rcar-du/rcar_du_crtc.c    |  14 +-
  .../gpu/drm/renesas/shmobile/shmob_drm_crtc.c |  20 +--
  drivers/gpu/drm/rockchip/rockchip_drm_vop.c   |   8 +-
  drivers/gpu/drm/rockchip/rockchip_drm_vop2.c  |  15 +-
  drivers/gpu/drm/solomon/ssd130x.c             |   2 +-
  drivers/gpu/drm/sprd/sprd_dpu.c               |   6 +-
  drivers/gpu/drm/sti/sti_crtc.c                |  14 +-
  drivers/gpu/drm/stm/ltdc.c                    |  12 +-
  drivers/gpu/drm/sun4i/sun4i_crtc.c            |  12 +-
  drivers/gpu/drm/tegra/dc.c                    |  12 +-
  drivers/gpu/drm/tidss/tidss_crtc.c            |  19 ++-
  drivers/gpu/drm/tidss/tidss_irq.c             |   4 +-
  drivers/gpu/drm/tilcdc/tilcdc_crtc.c          |  43 ++---
  drivers/gpu/drm/tiny/bochs.c                  |   6 +-
  drivers/gpu/drm/tiny/cirrus.c                 |   2 +-
  drivers/gpu/drm/tiny/gm12u320.c               |   4 +-
  drivers/gpu/drm/tiny/hx8357d.c                |   4 +-
  drivers/gpu/drm/tiny/ili9163.c                |   4 +-
  drivers/gpu/drm/tiny/ili9225.c                |   8 +-
  drivers/gpu/drm/tiny/ili9341.c                |   4 +-
  drivers/gpu/drm/tiny/ili9486.c                |   4 +-
  drivers/gpu/drm/tiny/mi0283qt.c               |   4 +-
  drivers/gpu/drm/tiny/ofdrm.c                  |   8 +-
  drivers/gpu/drm/tiny/panel-mipi-dbi.c         |   6 +-
  drivers/gpu/drm/tiny/repaper.c                |   8 +-
  drivers/gpu/drm/tiny/simpledrm.c              |   2 +-
  drivers/gpu/drm/tiny/st7586.c                 |   6 +-
  drivers/gpu/drm/tiny/st7735r.c                |   4 +-
  drivers/gpu/drm/tve200/tve200_display.c       |  14 +-
  drivers/gpu/drm/udl/udl_modeset.c             |   4 +-
  drivers/gpu/drm/vboxvideo/vbox_mode.c         |   6 +-
  drivers/gpu/drm/vc4/vc4_crtc.c                |  38 ++---
  drivers/gpu/drm/vc4/vc4_hdmi.c                |   2 +-
  drivers/gpu/drm/vc4/vc4_hvs.c                 |  12 +-
  drivers/gpu/drm/vc4/vc4_txp.c                 |   2 +-
  drivers/gpu/drm/virtio/virtgpu_display.c      |   4 +-
  drivers/gpu/drm/vkms/vkms_crtc.c              |  12 +-
  drivers/gpu/drm/vmwgfx/vmwgfx_kms.c           |   4 +-
  drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c          |  10 +-
  drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c          |   8 +-
  drivers/gpu/drm/xen/xen_drm_front_kms.c       |  10 +-
  drivers/gpu/drm/xlnx/zynqmp_kms.c             |   8 +-
  include/drm/drm_atomic_helper.h               |   2 +-
  include/drm/drm_crtc.h                        |   4 +-
  194 files changed, 1296 insertions(+), 1264 deletions(-)

base-commit: 06c2afb862f9da8dc5efa4b6076a0e48c3fbaaa5

--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)

Attachment: OpenPGP_signature
Description: OpenPGP digital signature


[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux