[PATCH v2 0/7] drm: Remove simple-display pipe

[PATCH v2 0/7] drm: Remove simple-display pipe

[Thomas Zimmermann]

There are no drivers using struct drm_simple_display_pipe left. Remove
the related helpers from DRM and clear any TODO items. Hyperv-drm needs
a minor fix to include some additional headers.

Since we're at it, also remove include statements for the simple-kms
header that are no longer needed. Most of them are likely left over from
earlier usage of the simple-display pipe.

Thomas Zimmermann (7):
  drm/hyperv: Add missing include statements
  drm/kms-helper: Remove struct drm_simple_display_pipe and helpers
  drm/bridge: Do not include <drm/drm_simple_kms_helper.h>
  drm/hisilicon/hibmc: Do not include <drm/drm_simple_kms_helper.h>
  drm/tegra: Do not include <drm/drm_simple_kms_helper.h>
  drm/verisilicon: Do not include <drm/drm_simple_kms_helper.h>
  drm/vc4: Do not include <drm/drm_simple_kms_helper.h>

 Documentation/gpu/todo.rst                    |  17 -
 drivers/gpu/drm/bridge/inno-hdmi.c            |   1 -
 drivers/gpu/drm/bridge/microchip-lvds.c       |   1 -
 drivers/gpu/drm/bridge/synopsys/dw-dp.c       |   1 -
 drivers/gpu/drm/bridge/tda998x_drv.c          |   1 -
 drivers/gpu/drm/drm_gem_atomic_helper.c       |  92 -----
 drivers/gpu/drm/drm_simple_kms_helper.c       | 328 ------------------
 .../gpu/drm/hisilicon/hibmc/hibmc_drm_dp.c    |   1 -
 .../gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c  |   1 -
 drivers/gpu/drm/hyperv/hyperv_drm.h           |   5 +
 drivers/gpu/drm/hyperv/hyperv_drm_drv.c       |   1 -
 drivers/gpu/drm/hyperv/hyperv_drm_proto.c     |   1 -
 drivers/gpu/drm/tegra/output.c                |   1 -
 drivers/gpu/drm/vc4/vc4_dpi.c                 |   1 -
 drivers/gpu/drm/vc4/vc4_dsi.c                 |   1 -
 drivers/gpu/drm/vc4/vc4_hdmi.c                |   1 -
 drivers/gpu/drm/vc4/vc4_vec.c                 |   1 -
 drivers/gpu/drm/verisilicon/vs_bridge.c       |   1 -
 drivers/staging/fbtft/TODO                    |   4 +-
 include/drm/drm_gem_atomic_helper.h           |  28 --
 include/drm/drm_simple_kms_helper.h           |  56 +--
 21 files changed, 9 insertions(+), 535 deletions(-)


base-commit: 9ecc25f4cc83ac280099d6c39e044d911774018d
-- 
2.55.0

[PATCH v2 1/7] drm/hyperv: Add missing include statements

[Thomas Zimmermann]

Hyperv-drm gets a number of headers via <drm/drm_simple_kms_helper.h>.
Include the files directly so that the simple-kms header doesn't have
to.

v2:
- remove simple-kms include statements (Javier)

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: Andy Shevchenko <andy@kernel.org>
---
 drivers/gpu/drm/hyperv/hyperv_drm.h       | 5 +++++
 drivers/gpu/drm/hyperv/hyperv_drm_drv.c   | 1 -
 drivers/gpu/drm/hyperv/hyperv_drm_proto.c | 1 -
 3 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/hyperv/hyperv_drm.h b/drivers/gpu/drm/hyperv/hyperv_drm.h
index 29e6f54af15f..f4262a37af36 100644
--- a/drivers/gpu/drm/hyperv/hyperv_drm.h
+++ b/drivers/gpu/drm/hyperv/hyperv_drm.h
@@ -6,6 +6,11 @@
 #ifndef _HYPERV_DRM_H_
 #define _HYPERV_DRM_H_
 
+#include <drm/drm_connector.h>
+#include <drm/drm_crtc.h>
+#include <drm/drm_encoder.h>
+#include <drm/drm_plane.h>
+
 #define VMBUS_MAX_PACKET_SIZE 0x4000
 
 struct hv_drm_device {
diff --git a/drivers/gpu/drm/hyperv/hyperv_drm_drv.c b/drivers/gpu/drm/hyperv/hyperv_drm_drv.c
index 0efd0593d031..031206a5b3d7 100644
--- a/drivers/gpu/drm/hyperv/hyperv_drm_drv.c
+++ b/drivers/gpu/drm/hyperv/hyperv_drm_drv.c
@@ -15,7 +15,6 @@
 #include <drm/drm_fbdev_shmem.h>
 #include <drm/drm_gem_shmem_helper.h>
 #include <drm/drm_print.h>
-#include <drm/drm_simple_kms_helper.h>
 
 #include "hyperv_drm.h"
 
diff --git a/drivers/gpu/drm/hyperv/hyperv_drm_proto.c b/drivers/gpu/drm/hyperv/hyperv_drm_proto.c
index 1d09bba1efdd..8ddd34833941 100644
--- a/drivers/gpu/drm/hyperv/hyperv_drm_proto.c
+++ b/drivers/gpu/drm/hyperv/hyperv_drm_proto.c
@@ -6,7 +6,6 @@
 #include <linux/hyperv.h>
 
 #include <drm/drm_print.h>
-#include <drm/drm_simple_kms_helper.h>
 
 #include "hyperv_drm.h"
 
-- 
2.55.0

Re: [PATCH v2 1/7] drm/hyperv: Add missing include statements

[Javier Martinez Canillas]

Thomas Zimmermann <tzimmermann@suse.de> writes:

Hello Thomas,

> Hyperv-drm gets a number of headers via <drm/drm_simple_kms_helper.h>.
> Include the files directly so that the simple-kms header doesn't have
> to.
>
> v2:
> - remove simple-kms include statements (Javier)
>
> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
> Reviewed-by: Andy Shevchenko <andy@kernel.org>
> ---

Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat

[PATCH v2 2/7] drm/kms-helper: Remove struct drm_simple_display_pipe and helpers

[Thomas Zimmermann]

There are no drivers using struct drm_simple_display_pipe. Remove
the data type and all related helpers.

The idea behind struct drm_simple_display_pipe was to provide an
easier way for writing DRM drivers besides regular atomic helpers.
It turned out to be more complicated. With all drivers being updated
to use atomic helpers, it's time to remove simple-pipe. Also clean
up the related TODO items.

v2:
- fix typo in fbtft TODO (Andy)

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: Andy Shevchenko <andy@kernel.org>
Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>
---
 Documentation/gpu/todo.rst              |  17 --
 drivers/gpu/drm/drm_gem_atomic_helper.c |  92 -------
 drivers/gpu/drm/drm_simple_kms_helper.c | 328 ------------------------
 drivers/staging/fbtft/TODO              |   4 +-
 include/drm/drm_gem_atomic_helper.h     |  28 --
 include/drm/drm_simple_kms_helper.h     |  56 +---
 6 files changed, 4 insertions(+), 521 deletions(-)

diff --git a/Documentation/gpu/todo.rst b/Documentation/gpu/todo.rst
index a437ae5c71f4..e6d36f8da086 100644
--- a/Documentation/gpu/todo.rst
+++ b/Documentation/gpu/todo.rst
@@ -29,23 +29,6 @@ refactorings already and are an expert in the specific area
 Subsystem-wide refactorings
 ===========================
 
-Replace struct drm_simple_display_pipe with regular atomic helpers
-------------------------------------------------------------------
-
-The data type struct drm_simple_display_pipe and its helpers were supposed
-to simplify driver development. Instead they only added an intermediate layer
-between atomic modesetting and the DRM driver.
-
-There are still drivers that use drm_simple_display_pipe. The task here is to
-convert them to use regular atomic helpers. Search for a driver that calls
-drm_simple_display_pipe_init() and inline all helpers from drm_simple_kms_helper.c
-into the driver, such that no simple-KMS interfaces are required. Please also
-rename all inlined functions according to driver conventions.
-
-Contact: Thomas Zimmermann, respective driver maintainer
-
-Level: Easy
-
 Remove custom dumb_map_offset implementations
 ---------------------------------------------
 
diff --git a/drivers/gpu/drm/drm_gem_atomic_helper.c b/drivers/gpu/drm/drm_gem_atomic_helper.c
index dd9efa7e3216..c952cfed84c1 100644
--- a/drivers/gpu/drm/drm_gem_atomic_helper.c
+++ b/drivers/gpu/drm/drm_gem_atomic_helper.c
@@ -10,7 +10,6 @@
 #include <drm/drm_gem.h>
 #include <drm/drm_gem_atomic_helper.h>
 #include <drm/drm_gem_framebuffer_helper.h>
-#include <drm/drm_simple_kms_helper.h>
 
 #include "drm_internal.h"
 
@@ -367,94 +366,3 @@ void drm_gem_end_shadow_fb_access(struct drm_plane *plane, struct drm_plane_stat
 	drm_gem_fb_vunmap(fb, shadow_plane_state->map);
 }
 EXPORT_SYMBOL(drm_gem_end_shadow_fb_access);
-
-/**
- * drm_gem_simple_kms_begin_shadow_fb_access - prepares shadow framebuffers for CPU access
- * @pipe: the simple display pipe
- * @plane_state: the plane state of type struct drm_shadow_plane_state
- *
- * This function implements struct drm_simple_display_funcs.begin_fb_access.
- *
- * See drm_gem_begin_shadow_fb_access() for details and
- * drm_gem_simple_kms_cleanup_shadow_fb() for cleanup.
- *
- * Returns:
- * 0 on success, or a negative errno code otherwise.
- */
-int drm_gem_simple_kms_begin_shadow_fb_access(struct drm_simple_display_pipe *pipe,
-					      struct drm_plane_state *plane_state)
-{
-	return drm_gem_begin_shadow_fb_access(&pipe->plane, plane_state);
-}
-EXPORT_SYMBOL(drm_gem_simple_kms_begin_shadow_fb_access);
-
-/**
- * drm_gem_simple_kms_end_shadow_fb_access - releases shadow framebuffers from CPU access
- * @pipe: the simple display pipe
- * @plane_state: the plane state of type struct drm_shadow_plane_state
- *
- * This function implements struct drm_simple_display_funcs.end_fb_access.
- * It undoes all effects of drm_gem_simple_kms_begin_shadow_fb_access() in
- * reverse order.
- *
- * See drm_gem_simple_kms_begin_shadow_fb_access().
- */
-void drm_gem_simple_kms_end_shadow_fb_access(struct drm_simple_display_pipe *pipe,
-					     struct drm_plane_state *plane_state)
-{
-	drm_gem_end_shadow_fb_access(&pipe->plane, plane_state);
-}
-EXPORT_SYMBOL(drm_gem_simple_kms_end_shadow_fb_access);
-
-/**
- * drm_gem_simple_kms_create_shadow_plane_state - creates shadow-buffered plane state
- * @pipe: the simple display pipe
- *
- * This function implements struct drm_simple_display_pipe_funcs.create_plane_state
- * for shadow-buffered planes.
- *
- * Returns:
- * A pointer to a new plane state on success, or an ERR_PTR()-encoded
- * error code otherwise.
- */
-struct drm_plane_state *
-drm_gem_simple_kms_create_shadow_plane_state(struct drm_simple_display_pipe *pipe)
-{
-	return drm_gem_create_shadow_plane_state(&pipe->plane);
-}
-EXPORT_SYMBOL(drm_gem_simple_kms_create_shadow_plane_state);
-
-/**
- * drm_gem_simple_kms_duplicate_shadow_plane_state - duplicates shadow-buffered plane state
- * @pipe: the simple display pipe
- *
- * This function implements struct drm_simple_display_funcs.duplicate_plane_state
- * for shadow-buffered planes. It does not duplicate existing mappings of the shadow
- * buffers. Mappings are maintained during the atomic commit by the plane's prepare_fb
- * and cleanup_fb helpers.
- *
- * Returns:
- * A pointer to a new plane state on success, or NULL otherwise.
- */
-struct drm_plane_state *
-drm_gem_simple_kms_duplicate_shadow_plane_state(struct drm_simple_display_pipe *pipe)
-{
-	return drm_gem_duplicate_shadow_plane_state(&pipe->plane);
-}
-EXPORT_SYMBOL(drm_gem_simple_kms_duplicate_shadow_plane_state);
-
-/**
- * drm_gem_simple_kms_destroy_shadow_plane_state - resets shadow-buffered plane state
- * @pipe: the simple display pipe
- * @plane_state: the plane state of type struct drm_shadow_plane_state
- *
- * This function implements struct drm_simple_display_funcs.destroy_plane_state
- * for shadow-buffered planes. It expects that mappings of shadow buffers
- * have been released already.
- */
-void drm_gem_simple_kms_destroy_shadow_plane_state(struct drm_simple_display_pipe *pipe,
-						   struct drm_plane_state *plane_state)
-{
-	drm_gem_destroy_shadow_plane_state(&pipe->plane, plane_state);
-}
-EXPORT_SYMBOL(drm_gem_simple_kms_destroy_shadow_plane_state);
diff --git a/drivers/gpu/drm/drm_simple_kms_helper.c b/drivers/gpu/drm/drm_simple_kms_helper.c
index 5e324c77ad27..1d048282ed50 100644
--- a/drivers/gpu/drm/drm_simple_kms_helper.c
+++ b/drivers/gpu/drm/drm_simple_kms_helper.c
@@ -16,10 +16,6 @@
 #include <drm/drm_probe_helper.h>
 #include <drm/drm_simple_kms_helper.h>
 
-static const struct drm_encoder_funcs drm_simple_encoder_funcs_cleanup = {
-	.destroy = drm_encoder_cleanup,
-};
-
 void *__drmm_simple_encoder_alloc(struct drm_device *dev, size_t size,
 				  size_t offset, int encoder_type)
 {
@@ -28,329 +24,5 @@ void *__drmm_simple_encoder_alloc(struct drm_device *dev, size_t size,
 }
 EXPORT_SYMBOL(__drmm_simple_encoder_alloc);
 
-static enum drm_mode_status
-drm_simple_kms_crtc_mode_valid(struct drm_crtc *crtc,
-			       const struct drm_display_mode *mode)
-{
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(crtc, struct drm_simple_display_pipe, crtc);
-	if (!pipe->funcs || !pipe->funcs->mode_valid)
-		/* Anything goes */
-		return MODE_OK;
-
-	return pipe->funcs->mode_valid(pipe, mode);
-}
-
-static int drm_simple_kms_crtc_check(struct drm_crtc *crtc,
-				     struct drm_atomic_commit *state)
-{
-	struct drm_crtc_state *crtc_state = drm_atomic_get_new_crtc_state(state, crtc);
-	int ret;
-
-	if (!crtc_state->enable)
-		goto out;
-
-	ret = drm_atomic_helper_check_crtc_primary_plane(crtc_state);
-	if (ret)
-		return ret;
-
-out:
-	return drm_atomic_add_affected_planes(state, crtc);
-}
-
-static void drm_simple_kms_crtc_enable(struct drm_crtc *crtc,
-				       struct drm_atomic_commit *state)
-{
-	struct drm_plane *plane;
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(crtc, struct drm_simple_display_pipe, crtc);
-	if (!pipe->funcs || !pipe->funcs->enable)
-		return;
-
-	plane = &pipe->plane;
-	pipe->funcs->enable(pipe, crtc->state, plane->state);
-}
-
-static void drm_simple_kms_crtc_disable(struct drm_crtc *crtc,
-					struct drm_atomic_commit *state)
-{
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(crtc, struct drm_simple_display_pipe, crtc);
-	if (!pipe->funcs || !pipe->funcs->disable)
-		return;
-
-	pipe->funcs->disable(pipe);
-}
-
-static const struct drm_crtc_helper_funcs drm_simple_kms_crtc_helper_funcs = {
-	.mode_valid = drm_simple_kms_crtc_mode_valid,
-	.atomic_check = drm_simple_kms_crtc_check,
-	.atomic_enable = drm_simple_kms_crtc_enable,
-	.atomic_disable = drm_simple_kms_crtc_disable,
-};
-
-static struct drm_crtc_state *drm_simple_kms_crtc_duplicate_state(struct drm_crtc *crtc)
-{
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(crtc, struct drm_simple_display_pipe, crtc);
-	if (!pipe->funcs || !pipe->funcs->duplicate_crtc_state)
-		return drm_atomic_helper_crtc_duplicate_state(crtc);
-
-	return pipe->funcs->duplicate_crtc_state(pipe);
-}
-
-static void drm_simple_kms_crtc_destroy_state(struct drm_crtc *crtc, struct drm_crtc_state *state)
-{
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(crtc, struct drm_simple_display_pipe, crtc);
-	if (!pipe->funcs || !pipe->funcs->destroy_crtc_state)
-		drm_atomic_helper_crtc_destroy_state(crtc, state);
-	else
-		pipe->funcs->destroy_crtc_state(pipe, state);
-}
-
-static int drm_simple_kms_crtc_enable_vblank(struct drm_crtc *crtc)
-{
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(crtc, struct drm_simple_display_pipe, crtc);
-	if (!pipe->funcs || !pipe->funcs->enable_vblank)
-		return 0;
-
-	return pipe->funcs->enable_vblank(pipe);
-}
-
-static void drm_simple_kms_crtc_disable_vblank(struct drm_crtc *crtc)
-{
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(crtc, struct drm_simple_display_pipe, crtc);
-	if (!pipe->funcs || !pipe->funcs->disable_vblank)
-		return;
-
-	pipe->funcs->disable_vblank(pipe);
-}
-
-static const struct drm_crtc_funcs drm_simple_kms_crtc_funcs = {
-	.atomic_create_state = drm_atomic_helper_crtc_create_state,
-	.destroy = drm_crtc_cleanup,
-	.set_config = drm_atomic_helper_set_config,
-	.page_flip = drm_atomic_helper_page_flip,
-	.atomic_duplicate_state = drm_simple_kms_crtc_duplicate_state,
-	.atomic_destroy_state = drm_simple_kms_crtc_destroy_state,
-	.enable_vblank = drm_simple_kms_crtc_enable_vblank,
-	.disable_vblank = drm_simple_kms_crtc_disable_vblank,
-};
-
-static int drm_simple_kms_plane_atomic_check(struct drm_plane *plane,
-					struct drm_atomic_commit *state)
-{
-	struct drm_plane_state *plane_state = drm_atomic_get_new_plane_state(state,
-									     plane);
-	struct drm_simple_display_pipe *pipe;
-	struct drm_crtc_state *crtc_state;
-	int ret;
-
-	pipe = container_of(plane, struct drm_simple_display_pipe, plane);
-	crtc_state = drm_atomic_get_new_crtc_state(state,
-						   &pipe->crtc);
-
-	ret = drm_atomic_helper_check_plane_state(plane_state, crtc_state,
-						  DRM_PLANE_NO_SCALING,
-						  DRM_PLANE_NO_SCALING,
-						  false, false);
-	if (ret)
-		return ret;
-
-	if (!plane_state->visible)
-		return 0;
-
-	if (!pipe->funcs || !pipe->funcs->check)
-		return 0;
-
-	return pipe->funcs->check(pipe, plane_state, crtc_state);
-}
-
-static void drm_simple_kms_plane_atomic_update(struct drm_plane *plane,
-					struct drm_atomic_commit *state)
-{
-	struct drm_plane_state *old_pstate = drm_atomic_get_old_plane_state(state,
-									    plane);
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(plane, struct drm_simple_display_pipe, plane);
-	if (!pipe->funcs || !pipe->funcs->update)
-		return;
-
-	pipe->funcs->update(pipe, old_pstate);
-}
-
-static int drm_simple_kms_plane_prepare_fb(struct drm_plane *plane,
-					   struct drm_plane_state *state)
-{
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(plane, struct drm_simple_display_pipe, plane);
-	if (!pipe->funcs || !pipe->funcs->prepare_fb) {
-		if (WARN_ON_ONCE(!drm_core_check_feature(plane->dev, DRIVER_GEM)))
-			return 0;
-
-		WARN_ON_ONCE(pipe->funcs && pipe->funcs->cleanup_fb);
-
-		return drm_gem_plane_helper_prepare_fb(plane, state);
-	}
-
-	return pipe->funcs->prepare_fb(pipe, state);
-}
-
-static void drm_simple_kms_plane_cleanup_fb(struct drm_plane *plane,
-					    struct drm_plane_state *state)
-{
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(plane, struct drm_simple_display_pipe, plane);
-	if (!pipe->funcs || !pipe->funcs->cleanup_fb)
-		return;
-
-	pipe->funcs->cleanup_fb(pipe, state);
-}
-
-static int drm_simple_kms_plane_begin_fb_access(struct drm_plane *plane,
-						struct drm_plane_state *new_plane_state)
-{
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(plane, struct drm_simple_display_pipe, plane);
-	if (!pipe->funcs || !pipe->funcs->begin_fb_access)
-		return 0;
-
-	return pipe->funcs->begin_fb_access(pipe, new_plane_state);
-}
-
-static void drm_simple_kms_plane_end_fb_access(struct drm_plane *plane,
-					       struct drm_plane_state *new_plane_state)
-{
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(plane, struct drm_simple_display_pipe, plane);
-	if (!pipe->funcs || !pipe->funcs->end_fb_access)
-		return;
-
-	pipe->funcs->end_fb_access(pipe, new_plane_state);
-}
-
-static bool drm_simple_kms_format_mod_supported(struct drm_plane *plane,
-						uint32_t format,
-						uint64_t modifier)
-{
-	return modifier == DRM_FORMAT_MOD_LINEAR;
-}
-
-static const struct drm_plane_helper_funcs drm_simple_kms_plane_helper_funcs = {
-	.prepare_fb = drm_simple_kms_plane_prepare_fb,
-	.cleanup_fb = drm_simple_kms_plane_cleanup_fb,
-	.begin_fb_access = drm_simple_kms_plane_begin_fb_access,
-	.end_fb_access = drm_simple_kms_plane_end_fb_access,
-	.atomic_check = drm_simple_kms_plane_atomic_check,
-	.atomic_update = drm_simple_kms_plane_atomic_update,
-};
-
-static struct drm_plane_state *
-drm_simple_kms_plane_create_state(struct drm_plane *plane)
-{
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(plane, struct drm_simple_display_pipe, plane);
-	if (!pipe->funcs || !pipe->funcs->create_plane_state)
-		return drm_atomic_helper_plane_create_state(plane);
-
-	return pipe->funcs->create_plane_state(pipe);
-}
-
-static struct drm_plane_state *drm_simple_kms_plane_duplicate_state(struct drm_plane *plane)
-{
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(plane, struct drm_simple_display_pipe, plane);
-	if (!pipe->funcs || !pipe->funcs->duplicate_plane_state)
-		return drm_atomic_helper_plane_duplicate_state(plane);
-
-	return pipe->funcs->duplicate_plane_state(pipe);
-}
-
-static void drm_simple_kms_plane_destroy_state(struct drm_plane *plane,
-					       struct drm_plane_state *state)
-{
-	struct drm_simple_display_pipe *pipe;
-
-	pipe = container_of(plane, struct drm_simple_display_pipe, plane);
-	if (!pipe->funcs || !pipe->funcs->destroy_plane_state)
-		drm_atomic_helper_plane_destroy_state(plane, state);
-	else
-		pipe->funcs->destroy_plane_state(pipe, state);
-}
-
-static const struct drm_plane_funcs drm_simple_kms_plane_funcs = {
-	.update_plane		= drm_atomic_helper_update_plane,
-	.disable_plane		= drm_atomic_helper_disable_plane,
-	.destroy		= drm_plane_cleanup,
-	.atomic_create_state	= drm_simple_kms_plane_create_state,
-	.atomic_duplicate_state	= drm_simple_kms_plane_duplicate_state,
-	.atomic_destroy_state	= drm_simple_kms_plane_destroy_state,
-	.format_mod_supported   = drm_simple_kms_format_mod_supported,
-};
-
-int drm_simple_display_pipe_attach_bridge(struct drm_simple_display_pipe *pipe,
-					  struct drm_bridge *bridge)
-{
-	return drm_bridge_attach(&pipe->encoder, bridge, NULL, 0);
-}
-EXPORT_SYMBOL(drm_simple_display_pipe_attach_bridge);
-
-int drm_simple_display_pipe_init(struct drm_device *dev,
-			struct drm_simple_display_pipe *pipe,
-			const struct drm_simple_display_pipe_funcs *funcs,
-			const uint32_t *formats, unsigned int format_count,
-			const uint64_t *format_modifiers,
-			struct drm_connector *connector)
-{
-	struct drm_encoder *encoder = &pipe->encoder;
-	struct drm_plane *plane = &pipe->plane;
-	struct drm_crtc *crtc = &pipe->crtc;
-	int ret;
-
-	pipe->connector = connector;
-	pipe->funcs = funcs;
-
-	drm_plane_helper_add(plane, &drm_simple_kms_plane_helper_funcs);
-	ret = drm_universal_plane_init(dev, plane, 0,
-				       &drm_simple_kms_plane_funcs,
-				       formats, format_count,
-				       format_modifiers,
-				       DRM_PLANE_TYPE_PRIMARY, NULL);
-	if (ret)
-		return ret;
-
-	drm_crtc_helper_add(crtc, &drm_simple_kms_crtc_helper_funcs);
-	ret = drm_crtc_init_with_planes(dev, crtc, plane, NULL,
-					&drm_simple_kms_crtc_funcs, NULL);
-	if (ret)
-		return ret;
-
-	encoder->possible_crtcs = drm_crtc_mask(crtc);
-	ret = drm_encoder_init(dev, encoder, &drm_simple_encoder_funcs_cleanup,
-			       DRM_MODE_ENCODER_NONE, NULL);
-	if (ret || !connector)
-		return ret;
-
-	return drm_connector_attach_encoder(connector, encoder);
-}
-EXPORT_SYMBOL(drm_simple_display_pipe_init);
-
 MODULE_DESCRIPTION("Helpers for drivers for simple display hardware");
 MODULE_LICENSE("GPL");
diff --git a/drivers/staging/fbtft/TODO b/drivers/staging/fbtft/TODO
index e72a08bf221c..2222998fd3ef 100644
--- a/drivers/staging/fbtft/TODO
+++ b/drivers/staging/fbtft/TODO
@@ -1,3 +1,3 @@
-* convert all these over to drm_simple_display_pipe and submit for inclusion
-  into the DRM subsystem under drivers/gpu/drm - fbdev doesn't take any new
+* convert all drivers over to DRM and submit them for inclusion into
+  the DRM subsystem under drivers/gpu/drm - fbdev doesn't take new
   drivers anymore.
diff --git a/include/drm/drm_gem_atomic_helper.h b/include/drm/drm_gem_atomic_helper.h
index d41877f77b77..f605a8a7b017 100644
--- a/include/drm/drm_gem_atomic_helper.h
+++ b/include/drm/drm_gem_atomic_helper.h
@@ -9,8 +9,6 @@
 #include <drm/drm_fourcc.h>
 #include <drm/drm_plane.h>
 
-struct drm_simple_display_pipe;
-
 /*
  * Plane Helpers
  */
@@ -126,30 +124,4 @@ void drm_gem_end_shadow_fb_access(struct drm_plane *plane, struct drm_plane_stat
 	.begin_fb_access = drm_gem_begin_shadow_fb_access, \
 	.end_fb_access = drm_gem_end_shadow_fb_access
 
-int drm_gem_simple_kms_begin_shadow_fb_access(struct drm_simple_display_pipe *pipe,
-					      struct drm_plane_state *plane_state);
-void drm_gem_simple_kms_end_shadow_fb_access(struct drm_simple_display_pipe *pipe,
-					     struct drm_plane_state *plane_state);
-struct drm_plane_state *
-drm_gem_simple_kms_create_shadow_plane_state(struct drm_simple_display_pipe *pipe);
-struct drm_plane_state *
-drm_gem_simple_kms_duplicate_shadow_plane_state(struct drm_simple_display_pipe *pipe);
-void drm_gem_simple_kms_destroy_shadow_plane_state(struct drm_simple_display_pipe *pipe,
-						   struct drm_plane_state *plane_state);
-
-/**
- * DRM_GEM_SIMPLE_DISPLAY_PIPE_SHADOW_PLANE_FUNCS -
- *	Initializes struct drm_simple_display_pipe_funcs for shadow-buffered planes
- *
- * Drivers may use GEM BOs as shadow buffers over the framebuffer memory. This
- * macro initializes struct drm_simple_display_pipe_funcs to use the rsp helper
- * functions.
- */
-#define DRM_GEM_SIMPLE_DISPLAY_PIPE_SHADOW_PLANE_FUNCS \
-	.begin_fb_access = drm_gem_simple_kms_begin_shadow_fb_access, \
-	.end_fb_access = drm_gem_simple_kms_end_shadow_fb_access, \
-	.create_plane_state = drm_gem_simple_kms_create_shadow_plane_state, \
-	.duplicate_plane_state = drm_gem_simple_kms_duplicate_shadow_plane_state, \
-	.destroy_plane_state = drm_gem_simple_kms_destroy_shadow_plane_state
-
 #endif /* __DRM_GEM_ATOMIC_HELPER_H__ */
diff --git a/include/drm/drm_simple_kms_helper.h b/include/drm/drm_simple_kms_helper.h
index 2b31c25c57a9..beb5c8a19bb7 100644
--- a/include/drm/drm_simple_kms_helper.h
+++ b/include/drm/drm_simple_kms_helper.h
@@ -11,61 +11,9 @@
 #ifndef __LINUX_DRM_SIMPLE_KMS_HELPER_H
 #define __LINUX_DRM_SIMPLE_KMS_HELPER_H
 
-#include <drm/drm_crtc.h>
-#include <drm/drm_encoder.h>
-#include <drm/drm_plane.h>
+#include <linux/types.h>
 
-struct drm_simple_display_pipe;
-
-struct drm_simple_display_pipe_funcs {
-	enum drm_mode_status (*mode_valid)(struct drm_simple_display_pipe *pipe,
-					   const struct drm_display_mode *mode);
-	void (*enable)(struct drm_simple_display_pipe *pipe,
-		       struct drm_crtc_state *crtc_state,
-		       struct drm_plane_state *plane_state);
-	void (*disable)(struct drm_simple_display_pipe *pipe);
-	int (*check)(struct drm_simple_display_pipe *pipe,
-		     struct drm_plane_state *plane_state,
-		     struct drm_crtc_state *crtc_state);
-	void (*update)(struct drm_simple_display_pipe *pipe,
-		       struct drm_plane_state *old_plane_state);
-	int (*prepare_fb)(struct drm_simple_display_pipe *pipe,
-			  struct drm_plane_state *plane_state);
-	void (*cleanup_fb)(struct drm_simple_display_pipe *pipe,
-			   struct drm_plane_state *plane_state);
-	int (*begin_fb_access)(struct drm_simple_display_pipe *pipe,
-			       struct drm_plane_state *new_plane_state);
-	void (*end_fb_access)(struct drm_simple_display_pipe *pipe,
-			      struct drm_plane_state *plane_state);
-	int (*enable_vblank)(struct drm_simple_display_pipe *pipe);
-	void (*disable_vblank)(struct drm_simple_display_pipe *pipe);
-	struct drm_crtc_state * (*duplicate_crtc_state)(struct drm_simple_display_pipe *pipe);
-	void (*destroy_crtc_state)(struct drm_simple_display_pipe *pipe,
-				   struct drm_crtc_state *crtc_state);
-	struct drm_plane_state * (*create_plane_state)(struct drm_simple_display_pipe *pipe);
-	struct drm_plane_state * (*duplicate_plane_state)(struct drm_simple_display_pipe *pipe);
-	void (*destroy_plane_state)(struct drm_simple_display_pipe *pipe,
-				    struct drm_plane_state *plane_state);
-};
-
-struct drm_simple_display_pipe {
-	struct drm_crtc crtc;
-	struct drm_plane plane;
-	struct drm_encoder encoder;
-	struct drm_connector *connector;
-
-	const struct drm_simple_display_pipe_funcs *funcs;
-};
-
-int drm_simple_display_pipe_attach_bridge(struct drm_simple_display_pipe *pipe,
-					  struct drm_bridge *bridge);
-
-int drm_simple_display_pipe_init(struct drm_device *dev,
-			struct drm_simple_display_pipe *pipe,
-			const struct drm_simple_display_pipe_funcs *funcs,
-			const uint32_t *formats, unsigned int format_count,
-			const uint64_t *format_modifiers,
-			struct drm_connector *connector);
+struct drm_device;
 
 void *__drmm_simple_encoder_alloc(struct drm_device *dev, size_t size,
 				  size_t offset, int encoder_type);
-- 
2.55.0

[PATCH v2 3/7] drm/bridge: Do not include <drm/drm_simple_kms_helper.h>

[Thomas Zimmermann]

The bridge code does not require anything from the simple-kms
header. Do not include it.

Suggested-by: Javier Martinez Canillas <javierm@redhat.com>
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
---
 drivers/gpu/drm/bridge/inno-hdmi.c      | 1 -
 drivers/gpu/drm/bridge/microchip-lvds.c | 1 -
 drivers/gpu/drm/bridge/synopsys/dw-dp.c | 1 -
 drivers/gpu/drm/bridge/tda998x_drv.c    | 1 -
 4 files changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/bridge/inno-hdmi.c b/drivers/gpu/drm/bridge/inno-hdmi.c
index dd35f5a875d3..ddf26ef08151 100644
--- a/drivers/gpu/drm/bridge/inno-hdmi.c
+++ b/drivers/gpu/drm/bridge/inno-hdmi.c
@@ -25,7 +25,6 @@
 #include <drm/drm_of.h>
 #include <drm/drm_print.h>
 #include <drm/drm_probe_helper.h>
-#include <drm/drm_simple_kms_helper.h>
 
 #include <drm/display/drm_hdmi_helper.h>
 #include <drm/display/drm_hdmi_state_helper.h>
diff --git a/drivers/gpu/drm/bridge/microchip-lvds.c b/drivers/gpu/drm/bridge/microchip-lvds.c
index dd4ffc9f4df3..39d5ec051627 100644
--- a/drivers/gpu/drm/bridge/microchip-lvds.c
+++ b/drivers/gpu/drm/bridge/microchip-lvds.c
@@ -26,7 +26,6 @@
 #include <drm/drm_of.h>
 #include <drm/drm_print.h>
 #include <drm/drm_probe_helper.h>
-#include <drm/drm_simple_kms_helper.h>
 
 #define LVDS_POLL_TIMEOUT_MS 1000
 
diff --git a/drivers/gpu/drm/bridge/synopsys/dw-dp.c b/drivers/gpu/drm/bridge/synopsys/dw-dp.c
index 86a20a0b9c01..b87bd91862af 100644
--- a/drivers/gpu/drm/bridge/synopsys/dw-dp.c
+++ b/drivers/gpu/drm/bridge/synopsys/dw-dp.c
@@ -27,7 +27,6 @@
 #include <drm/drm_of.h>
 #include <drm/drm_print.h>
 #include <drm/drm_probe_helper.h>
-#include <drm/drm_simple_kms_helper.h>
 
 #define DW_DP_VERSION_NUMBER			0x0000
 #define DW_DP_VERSION_TYPE			0x0004
diff --git a/drivers/gpu/drm/bridge/tda998x_drv.c b/drivers/gpu/drm/bridge/tda998x_drv.c
index 0b966c9844e7..9a4c47134fc4 100644
--- a/drivers/gpu/drm/bridge/tda998x_drv.c
+++ b/drivers/gpu/drm/bridge/tda998x_drv.c
@@ -19,7 +19,6 @@
 #include <drm/drm_of.h>
 #include <drm/drm_print.h>
 #include <drm/drm_probe_helper.h>
-#include <drm/drm_simple_kms_helper.h>
 
 #include <media/cec-notifier.h>
 
-- 
2.55.0

Re: [PATCH v2 3/7] drm/bridge: Do not include <drm/drm_simple_kms_helper.h>

[Javier Martinez Canillas]

Thomas Zimmermann <tzimmermann@suse.de> writes:

> The bridge code does not require anything from the simple-kms
> header. Do not include it.
>
> Suggested-by: Javier Martinez Canillas <javierm@redhat.com>
> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
> ---

Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat

[PATCH v2 4/7] drm/hisilicon/hibmc: Do not include <drm/drm_simple_kms_helper.h>

[Thomas Zimmermann]

The hibmc driver does not require anything from the simple-kms
header. Do not include it.

Suggested-by: Javier Martinez Canillas <javierm@redhat.com>
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
---
 drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_dp.c   | 1 -
 drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c | 1 -
 2 files changed, 2 deletions(-)

diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_dp.c b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_dp.c
index 2e9403b8bf3c..7bf6befa0440 100644
--- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_dp.c
+++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_dp.c
@@ -4,7 +4,6 @@
 #include <linux/io.h>
 
 #include <drm/drm_probe_helper.h>
-#include <drm/drm_simple_kms_helper.h>
 #include <drm/drm_atomic_helper.h>
 #include <drm/drm_modes.h>
 #include <drm/drm_drv.h>
diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c
index b9bd6d33fb0f..0b72509f7a69 100644
--- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c
+++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c
@@ -17,7 +17,6 @@
 #include <drm/drm_edid.h>
 #include <drm/drm_probe_helper.h>
 #include <drm/drm_print.h>
-#include <drm/drm_simple_kms_helper.h>
 
 #include "hibmc_drm_drv.h"
 #include "hibmc_drm_regs.h"
-- 
2.55.0

Re: [PATCH v2 4/7] drm/hisilicon/hibmc: Do not include <drm/drm_simple_kms_helper.h>

[Javier Martinez Canillas]

Thomas Zimmermann <tzimmermann@suse.de> writes:

> The hibmc driver does not require anything from the simple-kms
> header. Do not include it.
>
> Suggested-by: Javier Martinez Canillas <javierm@redhat.com>
> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
> ---

Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat

[PATCH v2 5/7] drm/tegra: Do not include <drm/drm_simple_kms_helper.h>

[Thomas Zimmermann]

The tegra driver does not require anything from the simple-kms
header. Do not include it.

Suggested-by: Javier Martinez Canillas <javierm@redhat.com>
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
---
 drivers/gpu/drm/tegra/output.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/tegra/output.c b/drivers/gpu/drm/tegra/output.c
index d0ba3bd0aab8..eebf0ec0eea0 100644
--- a/drivers/gpu/drm/tegra/output.c
+++ b/drivers/gpu/drm/tegra/output.c
@@ -11,7 +11,6 @@
 #include <drm/drm_edid.h>
 #include <drm/drm_of.h>
 #include <drm/drm_panel.h>
-#include <drm/drm_simple_kms_helper.h>
 
 #include "drm.h"
 #include "dc.h"
-- 
2.55.0

Re: [PATCH v2 5/7] drm/tegra: Do not include <drm/drm_simple_kms_helper.h>

[Javier Martinez Canillas]

Thomas Zimmermann <tzimmermann@suse.de> writes:

> The tegra driver does not require anything from the simple-kms
> header. Do not include it.
>
> Suggested-by: Javier Martinez Canillas <javierm@redhat.com>
> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
> ---

Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat

[PATCH v2 6/7] drm/verisilicon: Do not include <drm/drm_simple_kms_helper.h>

[Thomas Zimmermann]

The verisilicon driver does not require anything from the simple-kms
header. Do not include it.

Suggested-by: Javier Martinez Canillas <javierm@redhat.com>
Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
---
 drivers/gpu/drm/verisilicon/vs_bridge.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/verisilicon/vs_bridge.c b/drivers/gpu/drm/verisilicon/vs_bridge.c
index dc7c85b07fe3..b0c221f5bfa7 100644
--- a/drivers/gpu/drm/verisilicon/vs_bridge.c
+++ b/drivers/gpu/drm/verisilicon/vs_bridge.c
@@ -16,7 +16,6 @@
 #include <drm/drm_encoder.h>
 #include <drm/drm_of.h>
 #include <drm/drm_print.h>
-#include <drm/drm_simple_kms_helper.h>
 
 #include "vs_bridge.h"
 #include "vs_bridge_regs.h"
-- 
2.55.0

Re: [PATCH v2 6/7] drm/verisilicon: Do not include <drm/drm_simple_kms_helper.h>

[Javier Martinez Canillas]

Thomas Zimmermann <tzimmermann@suse.de> writes:

> The verisilicon driver does not require anything from the simple-kms
> header. Do not include it.
>
> Suggested-by: Javier Martinez Canillas <javierm@redhat.com>
> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
> ---

Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat

[PATCH v2 7/7] drm/vc4: Do not include <drm/drm_simple_kms_helper.h>

[Thomas Zimmermann]

The vc4 driver does not require anything from the simple-kms
header. Do not include it.

Suggested-by: Javier Martinez Canillas <javierm@redhat.com>

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
---
 drivers/gpu/drm/vc4/vc4_dpi.c  | 1 -
 drivers/gpu/drm/vc4/vc4_dsi.c  | 1 -
 drivers/gpu/drm/vc4/vc4_hdmi.c | 1 -
 drivers/gpu/drm/vc4/vc4_vec.c  | 1 -
 4 files changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/vc4/vc4_dpi.c b/drivers/gpu/drm/vc4/vc4_dpi.c
index 53f36626a50d..686903b8864b 100644
--- a/drivers/gpu/drm/vc4/vc4_dpi.c
+++ b/drivers/gpu/drm/vc4/vc4_dpi.c
@@ -19,7 +19,6 @@
 #include <drm/drm_panel.h>
 #include <drm/drm_print.h>
 #include <drm/drm_probe_helper.h>
-#include <drm/drm_simple_kms_helper.h>
 #include <linux/clk.h>
 #include <linux/component.h>
 #include <linux/media-bus-format.h>
diff --git a/drivers/gpu/drm/vc4/vc4_dsi.c b/drivers/gpu/drm/vc4/vc4_dsi.c
index 0148f2befed1..5ed3c35ee9c7 100644
--- a/drivers/gpu/drm/vc4/vc4_dsi.c
+++ b/drivers/gpu/drm/vc4/vc4_dsi.c
@@ -38,7 +38,6 @@
 #include <drm/drm_panel.h>
 #include <drm/drm_print.h>
 #include <drm/drm_probe_helper.h>
-#include <drm/drm_simple_kms_helper.h>
 
 #include "vc4_drv.h"
 #include "vc4_regs.h"
diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4/vc4_hdmi.c
index 3a90dbb61f4e..6b00581f304f 100644
--- a/drivers/gpu/drm/vc4/vc4_hdmi.c
+++ b/drivers/gpu/drm/vc4/vc4_hdmi.c
@@ -41,7 +41,6 @@
 #include <drm/drm_edid.h>
 #include <drm/drm_print.h>
 #include <drm/drm_probe_helper.h>
-#include <drm/drm_simple_kms_helper.h>
 #include <linux/clk.h>
 #include <linux/component.h>
 #include <linux/gpio/consumer.h>
diff --git a/drivers/gpu/drm/vc4/vc4_vec.c b/drivers/gpu/drm/vc4/vc4_vec.c
index 792b7b39be59..215b5d3dc5c5 100644
--- a/drivers/gpu/drm/vc4/vc4_vec.c
+++ b/drivers/gpu/drm/vc4/vc4_vec.c
@@ -19,7 +19,6 @@
 #include <drm/drm_panel.h>
 #include <drm/drm_print.h>
 #include <drm/drm_probe_helper.h>
-#include <drm/drm_simple_kms_helper.h>
 #include <linux/clk.h>
 #include <linux/component.h>
 #include <linux/of.h>
-- 
2.55.0

Re: [PATCH v2 7/7] drm/vc4: Do not include <drm/drm_simple_kms_helper.h>

[Javier Martinez Canillas]

Thomas Zimmermann <tzimmermann@suse.de> writes:

> The vc4 driver does not require anything from the simple-kms
> header. Do not include it.
>
> Suggested-by: Javier Martinez Canillas <javierm@redhat.com>
>
> Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>

Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat

Re: [PATCH v2 0/7] drm: Remove simple-display pipe

[Andy Shevchenko]

On Fri, Oct 02, 2026 at 02:41:50PM +0200, Thomas Zimmermann wrote:
> There are no drivers using struct drm_simple_display_pipe left. Remove
> the related helpers from DRM and clear any TODO items. Hyperv-drm needs
> a minor fix to include some additional headers.
> 
> Since we're at it, also remove include statements for the simple-kms
> header that are no longer needed. Most of them are likely left over from
> earlier usage of the simple-display pipe.

Reviewed-by: Andy Shevchenko <andriy.shevchenko@intel.com>
for the patches 3-7.

-- 
With Best Regards,
Andy Shevchenko