All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/8] Cleanup CRTC .enable()/.disable() cargo-cult
@ 2017-06-27 20:38 Laurent Pinchart
  2017-06-27 20:38 ` [PATCH 1/8] drm: arcpgu: Remove CRTC .commit() helper operation Laurent Pinchart
                   ` (17 more replies)
  0 siblings, 18 replies; 51+ messages in thread
From: Laurent Pinchart @ 2017-06-27 20:38 UTC (permalink / raw)
  To: dri-devel
  Cc: Neil Armstrong, Liviu Dudau, Thierry Reding, Gerd Hoffmann,
	Daniel Vetter, Boris Brezillon, Thomas Hellstrom, Alexey Brodkin,
	Xinliang Liu, Xinwei Kong, Tomi Valkeinen, Mali DP Maintainers,
	Dave Airlie, Chen Feng, Jyri Sarha, Vincent Abriou,
	VMware Graphics, Alison Wang, Seung-Woo Kim, Philippe Cornu,
	Yannick Fertre, Kyungmin Park

Hello,

The atomic helpers favour the .enable() and .atomic_disable() CRTC helper
operations, but still support the legacy .prepare(), .commit(), .disable() and
.dpms() operations. Some drivers have been updated, but most still use various
combination of new and legacy operations, leading to confusion among new
developers when they read the code.

To avoid cargo-cult use of the legacy operations, this patch series
mass-updates all atomic drivers to use the new CRTC atomic helper operations.

In addition, patch 7/8 adds an old state pointer argument to the CRTC
.enable() helper operation, and rename it to .atomic_enable() for consistency
with .atomic_disable(). These two changes could have been split in separate
patch, but as they are simple and touch the same large number of files,
keeping the two changes as one is simpler and less error-prone.

The patches are based on top of the drm-misc-next branch and have been
compile-tested only except for rcar-du-drm that has been tested on real
hardware. Given the high risk of conflicts I would like to get them merged as
soon as possible (after, of course, proper review and testing).

Laurent Pinchart (8):
  drm: arcpgu: Remove CRTC .commit() helper operation
  drm: arcpgu: Remove CRTC .prepare() helper operation
  drm: qxl: Remove unused CRTC .dpms() helper operation
  drm: qxl: Replace CRTC .commit() helper operation with .enable()
  drm: vmwgfx: Remove unneeded CRTC .prepare() helper operation
  drm: vmwgfx: Replace CRTC .commit() helper operation with .enable()
  drm: Add old state pointer to CRTC .enable() helper function
  drm: Convert atomic drivers from CRTC .disable() to .atomic_disable()

 drivers/gpu/drm/arc/arcpgu_crtc.c               | 12 ++++----
 drivers/gpu/drm/arm/hdlcd_crtc.c                | 10 ++++---
 drivers/gpu/drm/arm/malidp_crtc.c               | 10 ++++---
 drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c  | 10 ++++---
 drivers/gpu/drm/drm_atomic_helper.c             |  7 +++--
 drivers/gpu/drm/drm_simple_kms_helper.c         | 10 ++++---
 drivers/gpu/drm/exynos/exynos_drm_crtc.c        | 10 ++++---
 drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c      |  5 ++--
 drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c  | 10 ++++---
 drivers/gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 10 ++++---
 drivers/gpu/drm/imx/ipuv3-crtc.c                |  5 ++--
 drivers/gpu/drm/mediatek/mtk_drm_crtc.c         | 10 ++++---
 drivers/gpu/drm/meson/meson_crtc.c              | 10 ++++---
 drivers/gpu/drm/msm/mdp/mdp4/mdp4_crtc.c        | 10 ++++---
 drivers/gpu/drm/msm/mdp/mdp5/mdp5_crtc.c        | 10 ++++---
 drivers/gpu/drm/omapdrm/omap_crtc.c             | 10 ++++---
 drivers/gpu/drm/qxl/qxl_display.c               | 15 ++++------
 drivers/gpu/drm/rcar-du/rcar_du_crtc.c          | 10 ++++---
 drivers/gpu/drm/rockchip/rockchip_drm_vop.c     | 10 ++++---
 drivers/gpu/drm/sti/sti_crtc.c                  | 10 ++++---
 drivers/gpu/drm/stm/ltdc.c                      | 10 ++++---
 drivers/gpu/drm/sun4i/sun4i_crtc.c              | 10 ++++---
 drivers/gpu/drm/tegra/dc.c                      | 10 ++++---
 drivers/gpu/drm/tilcdc/tilcdc_crtc.c            | 16 +++++++++--
 drivers/gpu/drm/vc4/vc4_crtc.c                  | 10 ++++---
 drivers/gpu/drm/virtio/virtgpu_display.c        | 10 ++++---
 drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c             | 27 ++++++------------
 drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c            | 14 +++++----
 drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c            | 10 ++++---
 drivers/gpu/drm/zte/zx_vou.c                    | 10 ++++---
 include/drm/drm_modeset_helper_vtables.h        | 38 +++++++++++++------------
 31 files changed, 204 insertions(+), 155 deletions(-)

-- 
Regards,

Laurent Pinchart

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

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

end of thread, other threads:[~2017-07-03  8:58 UTC | newest]

Thread overview: 51+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-27 20:38 [PATCH 0/8] Cleanup CRTC .enable()/.disable() cargo-cult Laurent Pinchart
2017-06-27 20:38 ` [PATCH 1/8] drm: arcpgu: Remove CRTC .commit() helper operation Laurent Pinchart
2017-06-27 20:38 ` [PATCH 2/8] drm: arcpgu: Remove CRTC .prepare() " Laurent Pinchart
2017-06-27 20:38 ` [PATCH 3/8] drm: qxl: Remove unused CRTC .dpms() " Laurent Pinchart
2017-06-27 20:38 ` [PATCH 4/8] drm: qxl: Replace CRTC .commit() helper operation with .enable() Laurent Pinchart
2017-06-27 20:38 ` [PATCH 5/8] drm: vmwgfx: Remove unneeded CRTC .prepare() helper operation Laurent Pinchart
2017-06-27 20:38 ` [PATCH 6/8] drm: vmwgfx: Replace CRTC .commit() helper operation with .enable() Laurent Pinchart
2017-06-28  7:39   ` Daniel Vetter
2017-06-28  8:25     ` Laurent Pinchart
2017-06-27 20:38 ` [PATCH 7/8] drm: Add old state pointer to CRTC .enable() helper function Laurent Pinchart
2017-06-27 20:38 ` [PATCH 8/8] drm: Convert atomic drivers from CRTC .disable() to .atomic_disable() Laurent Pinchart
2017-06-27 21:16 ` [PATCH 1/8] drm: arcpgu: Remove CRTC .commit() helper operation Laurent Pinchart
2017-06-28  8:36   ` Alexey Brodkin
2017-06-27 21:16 ` [PATCH 2/8] drm: arcpgu: Remove CRTC .prepare() " Laurent Pinchart
2017-06-28  8:36   ` Alexey Brodkin
2017-06-27 21:16 ` [PATCH 3/8] drm: qxl: Remove unused CRTC .dpms() " Laurent Pinchart
2017-06-27 21:16 ` [PATCH 4/8] drm: qxl: Replace CRTC .commit() helper operation with .enable() Laurent Pinchart
2017-06-28  8:14   ` Daniel Vetter
2017-06-27 21:16 ` [PATCH 5/8] drm: vmwgfx: Remove unneeded CRTC .prepare() helper operation Laurent Pinchart
2017-06-29 12:56   ` Thomas Hellstrom
2017-06-27 21:16 ` [PATCH 6/8] drm: vmwgfx: Replace CRTC .commit() helper operation with .enable() Laurent Pinchart
2017-06-29 12:58   ` Thomas Hellstrom
2017-06-27 21:16 ` [PATCH 7/8] drm: Add old state pointer to CRTC .enable() helper function Laurent Pinchart
2017-06-28  6:46   ` Maxime Ripard
2017-06-28  7:42   ` Daniel Vetter
2017-06-28 11:55     ` Laurent Pinchart
2017-06-28  8:15   ` Philipp Zabel
2017-06-28  8:37   ` Alexey Brodkin
2017-06-28  8:43   ` Boris Brezillon
2017-06-28 13:41   ` Liviu Dudau
2017-06-28 13:44     ` Laurent Pinchart
2017-06-28 13:55   ` dri-devel-bounces
2017-06-29  7:48   ` Philippe CORNU
2017-06-29  8:00   ` Vincent ABRIOU
2017-06-29 12:59   ` Thomas Hellstrom
2017-07-03  8:57   ` Jyri Sarha
2017-06-27 21:16 ` [PATCH 8/8] drm: Convert atomic drivers from CRTC .disable() to .atomic_disable() Laurent Pinchart
2017-06-28  6:47   ` Maxime Ripard
2017-06-28  8:15   ` Philipp Zabel
2017-06-28  8:39   ` Alexey Brodkin
2017-06-28  8:45   ` Boris Brezillon
2017-06-29  7:49   ` Philippe CORNU
2017-06-29  8:00   ` Vincent ABRIOU
2017-06-29 13:00   ` Thomas Hellstrom
2017-07-03  8:53   ` Jyri Sarha
2017-06-28  8:16 ` [PATCH 0/8] Cleanup CRTC .enable()/.disable() cargo-cult Daniel Vetter
2017-06-28  8:20   ` Daniel Vetter
2017-06-28  8:23     ` Laurent Pinchart
2017-06-28 11:27 ` Emil Velikov
2017-06-28 13:10   ` Laurent Pinchart
2017-06-29 12:39     ` Emil Velikov

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.