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

Thomas Zimmermann [off-list ref] writes:

Hello Thomas,

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.

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
---
 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              |   6 +-
 include/drm/drm_gem_atomic_helper.h     |  28 --
 include/drm/drm_simple_kms_helper.h     |  56 +---

There are no more users of struct drm_simple_display_pipe but still some
drivers include the drm_simple_kms_helper.h header. Latest drm-misc-next: 

git grep "drm_simple_kms_helper.h" -- drivers/gpu/drm/*/ | wc -l
21

So it seems you need more  preparatory patches. For this patch though:

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

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat