Re: [PATCH v2 00/15] drm/drm_simple: remove drm_simple_encoder_init
Thomas Zimmermann <[email protected]>
| Newsgroups | org.kernel.vger.linux-samsung-soc,dev.linux.lists.imx,dev.linux.lists.virtualization,org.freedesktop.lists.dri-devel,org.infradead.lists.linux-amlogic,org.infradead.lists.linux-mediatek,org.kernel.vger.linux-doc,org.kernel.vger.linux-kernel,org.kernel.vger.linux-renesas-soc,org.kernel.vger.linux-tegra |
|---|---|
| Message-ID | <[email protected]> |
Hi Am 20.07.26 um 17:40 schrieb Diogo Silva: > The simple KMS helpers are deprecated because they only add an > intermediate layer between drivers and atomic modesetting. > > This series open-codes all remaining drm_simple_encoder_init() users by > calling drm_encoder_init() directly and providing driver-local > drm_encoder_funcs where needed. After the driver conversions, the helper > is removed and the completed DRM todo item is dropped. > > Signed-off-by: Diogo Silva <[email protected]> For the whole series: Reviewed-by: Thomas Zimmermann <[email protected]> Diogo, do you want to send out another revision? Otherwise it's time to get these patches merged. Best regards Thomas > --- > Changes in v2: > - Change logging to consistently use drm_err() > - Link to v1: https://patch.msgid.link/[email protected] > > To: Jingoo Han <[email protected]> > To: Inki Dae <[email protected]> > To: Seung-Woo Kim <[email protected]> > To: Kyungmin Park <[email protected]> > To: David Airlie <[email protected]> > To: Simona Vetter <[email protected]> > To: Krzysztof Kozlowski <[email protected]> > To: Peter Griffin <[email protected]> > To: Alim Akhtar <[email protected]> > To: Laurent Pinchart <[email protected]> > To: Tomi Valkeinen <[email protected]> > To: Maarten Lankhorst <[email protected]> > To: Maxime Ripard <[email protected]> > To: Thomas Zimmermann <[email protected]> > To: Michal Simek <[email protected]> > To: Thierry Reding <[email protected]> > To: Mikko Perttunen <[email protected]> > To: Jonathan Hunter <[email protected]> > To: Stefan Agner <[email protected]> > To: Alison Wang <[email protected]> > To: Anitha Chrisanthus <[email protected]> > To: David Airlie <[email protected]> > To: Gerd Hoffmann <[email protected]> > To: Dmitry Osipenko <[email protected]> > To: Gurchetan Singh <[email protected]> > To: Chia-I Wu <[email protected]> > To: Jyri Sarha <[email protected]> > To: Liu Ying <[email protected]> > To: Frank Li <[email protected]> > To: Sascha Hauer <[email protected]> > To: Pengutronix Kernel Team <[email protected]> > To: Fabio Estevam <[email protected]> > To: Chun-Kuang Hu <[email protected]> > To: Philipp Zabel <[email protected]> > To: Matthias Brugger <[email protected]> > To: AngeloGioacchino Del Regno <[email protected]> > To: Geert Uytterhoeven <[email protected]> > To: Xinliang Liu <[email protected]> > To: Sumit Semwal <[email protected]> > To: Yongqin Liu <[email protected]> > To: John Stultz <[email protected]> > To: Liviu Dudau <[email protected]> > To: Neil Armstrong <[email protected]> > To: Kevin Hilman <[email protected]> > To: Jerome Brunet <[email protected]> > To: Martin Blumenstingl <[email protected]> > To: Jonathan Corbet <[email protected]> > To: Shuah Khan <[email protected]> > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > Cc: [email protected] > > --- > Diogo Silva (15): > drm/exynos: remove dependency on DRM simple helpers > drm/xlnx/zynqmp_dpsub: remove dependency on DRM simple helpers > drm/tegra: remove dependency on DRM simple helpers > drm/fsl-dcu: remove dependency on DRM simple helpers > drm/kmb: remove dependency on DRM simple helpers > drm/virtio: remove dependency on DRM simple helpers > drm/tidss: remove dependency on DRM simple helpers > drm/imx: remove dependency on DRM simple helpers > drm/mediatek: remove dependency on DRM simple helpers > drm/renesas/shmobile: remove dependency on DRM simple helpers > drm/hisilicon/kirin: remove dependency on DRM simple helpers > drm/arm/komeda: remove dependency on DRM simple helpers > drm/meson: remove dependency on DRM simple helpers > drm/drm_simple: remove deprecated drm_simple_encoder_init function > Documentation/gpu: remove completed drm_simple_encoder_init() todo > > Documentation/gpu/todo.rst | 15 --------------- > drivers/gpu/drm/arm/display/komeda/komeda_crtc.c | 9 +++++++-- > drivers/gpu/drm/drm_simple_kms_helper.c | 13 ++----------- > drivers/gpu/drm/exynos/exynos_dp.c | 13 +++++++++++-- > drivers/gpu/drm/exynos/exynos_drm_dpi.c | 13 +++++++++++-- > drivers/gpu/drm/exynos/exynos_drm_dsi.c | 11 +++++++++-- > drivers/gpu/drm/exynos/exynos_drm_vidi.c | 13 +++++++++++-- > drivers/gpu/drm/exynos/exynos_hdmi.c | 14 ++++++++++++-- > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_rgb.c | 10 +++++++--- > drivers/gpu/drm/hisilicon/kirin/dw_drm_dsi.c | 9 +++++++-- > drivers/gpu/drm/imx/dc/dc-kms.c | 8 ++++++-- > drivers/gpu/drm/kmb/kmb_dsi.c | 9 +++++++-- > drivers/gpu/drm/mediatek/mtk_dsi.c | 10 +++++++--- > drivers/gpu/drm/meson/meson_encoder_cvbs.c | 11 ++++++++--- > drivers/gpu/drm/meson/meson_encoder_dsi.c | 11 ++++++++--- > drivers/gpu/drm/meson/meson_encoder_hdmi.c | 11 ++++++++--- > drivers/gpu/drm/renesas/shmobile/shmob_drm_crtc.c | 10 +++++++--- > drivers/gpu/drm/tegra/dsi.c | 16 +++++++++++++--- > drivers/gpu/drm/tegra/rgb.c | 15 +++++++++++++-- > drivers/gpu/drm/tidss/tidss_encoder.c | 10 +++++++--- > drivers/gpu/drm/virtio/virtgpu_display.c | 12 ++++++++++-- > drivers/gpu/drm/xlnx/zynqmp_kms.c | 14 ++++++++++++-- > include/drm/drm_simple_kms_helper.h | 4 ---- > 23 files changed, 183 insertions(+), 78 deletions(-) > --- > base-commit: e1113c37ba4f166e37dec74a729e6dfd8cba1687 > change-id: 20260718-drm_simple_encoder_init-d069a4cc7f8b > > Best regards, > -- > Diogo Silva <[email protected]> > -- -- Thomas Zimmermann Graphics Driver Developer SUSE Software Solutions Germany GmbH Frankenstr. 146, 90461 Nürnberg, Germany, www.suse.com GF: Jochen Jaser, Andrew McDonald, (HRB 36809, AG Nürnberg)