From: Thomas Zimmermann <tzimmermann@suse.de>
To: linux-aspeed@lists.ozlabs.org
Subject: [PATCH 00/53] drm: Convert to platform remove callback returning void
Date: Mon, 8 May 2023 09:06:27 +0200 [thread overview]
Message-ID: <935faac5-280b-b2e0-3fdb-d0424990e43a@suse.de> (raw)
In-Reply-To: <20230507162616.1368908-1-u.kleine-koenig@pengutronix.de>
Hi,
for the whole series:
Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
Please see my comment on the patches to tiny/.
Let me know if you want me to merge this patchset into drm-misc-next.
Best regards
Thomas
Am 07.05.23 um 18:25 schrieb Uwe Kleine-K?nig:
> Hello,
>
> this patch series adapts the platform drivers below drivers/gpu/drm
> to use the .remove_new() callback. Compared to the traditional .remove()
> callback .remove_new() returns no value. This is a good thing because
> the driver core doesn't (and cannot) cope for errors during remove. The
> only effect of a non-zero return value in .remove() is that the driver
> core emits a warning. The device is removed anyhow and an early return
> from .remove() usually yields a resource leak.
>
> By changing the remove callback to return void driver authors cannot
> reasonably (but wrongly) assume any more that there happens some kind of
> cleanup later.
>
> Best regards
> Uwe
>
> Uwe Kleine-K?nig (53):
> drm/komeda: Convert to platform remove callback returning void
> drm/arm/hdlcd: Convert to platform remove callback returning void
> drm/arm/malidp: Convert to platform remove callback returning void
> drm/armada: Convert to platform remove callback returning void
> drm/aspeed: Convert to platform remove callback returning void
> drm/atmel-hlcdc: Convert to platform remove callback returning void
> drm/bridge: cdns-dsi: Convert to platform remove callback returning
> void
> drm/bridge: display-connector: Convert to platform remove callback
> returning void
> drm/bridge: fsl-ldb: Convert to platform remove callback returning
> void
> drm/imx/imx8*: Convert to platform remove callback returning void
> drm/bridge: lvds-codec: Convert to platform remove callback returning
> void
> drm/bridge: nwl-dsi: Convert to platform remove callback returning
> void
> drm/bridge: simple-bridge: Convert to platform remove callback
> returning void
> drm/bridge: synopsys: Convert to platform remove callback returning
> void
> drm/bridge: thc63lvd1024: Convert to platform remove callback
> returning void
> drm/bridge: tfp410: Convert to platform remove callback returning void
> drm/etnaviv: Convert to platform remove callback returning void
> drm/exynos: Convert to platform remove callback returning void
> drm/fsl-dcu: Convert to platform remove callback returning void
> drm/hisilicon: Convert to platform remove callback returning void
> drm/imx/dcss: Convert to platform remove callback returning void
> drm/imx/ipuv3: Convert to platform remove callback returning void
> drm/ingenic: Convert to platform remove callback returning void
> drm/kmb: Convert to platform remove callback returning void
> drm/lima: Convert to platform remove callback returning void
> drm/logicvc: Convert to platform remove callback returning void
> drm/mcde: Convert to platform remove callback returning void
> drm/mediatek: Convert to platform remove callback returning void
> drm/mediatek: Convert to platform remove callback returning void
> drm/meson: Convert to platform remove callback returning void
> drm/msm: Convert to platform remove callback returning void
> drm/mxsfb: Convert to platform remove callback returning void
> drm/nouveau: Convert to platform remove callback returning void
> drm/omap: Convert to platform remove callback returning void
> drm/panel: Convert to platform remove callback returning void
> drm/panfrost: Convert to platform remove callback returning void
> drm/rcar-du: Convert to platform remove callback returning void
> drm/rockchip: Convert to platform remove callback returning void
> drm/shmobile: Convert to platform remove callback returning void
> drm/sprd: Convert to platform remove callback returning void
> drm/sti: Convert to platform remove callback returning void
> drm/stm: Convert to platform remove callback returning void
> drm/sun4i: Convert to platform remove callback returning void
> drm/tegra: Convert to platform remove callback returning void
> drm/tests: helpers: Convert to platform remove callback returning void
> drm/tidss: Convert to platform remove callback returning void
> drm/tilcdc: Convert to platform remove callback returning void
> drm/tiny: Convert to platform remove callback returning void
> drm/tiny: Convert to platform remove callback returning void
> drm/tve200: Convert to platform remove callback returning void
> drm/v3d: Convert to platform remove callback returning void
> drm/vc4: Convert to platform remove callback returning void
> drm/xlnx/zynqmp_dpsub: Convert to platform remove callback returning
> void
>
> drivers/gpu/drm/arm/display/komeda/komeda_drv.c | 5 ++---
> drivers/gpu/drm/arm/hdlcd_drv.c | 5 ++---
> drivers/gpu/drm/arm/malidp_drv.c | 5 ++---
> drivers/gpu/drm/armada/armada_crtc.c | 5 ++---
> drivers/gpu/drm/armada/armada_drv.c | 5 ++---
> drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | 6 ++----
> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 6 ++----
> drivers/gpu/drm/bridge/cadence/cdns-dsi-core.c | 6 ++----
> drivers/gpu/drm/bridge/display-connector.c | 6 ++----
> drivers/gpu/drm/bridge/fsl-ldb.c | 6 ++----
> drivers/gpu/drm/bridge/imx/imx8qm-ldb-drv.c | 6 ++----
> drivers/gpu/drm/bridge/imx/imx8qxp-ldb-drv.c | 6 ++----
> drivers/gpu/drm/bridge/imx/imx8qxp-pixel-combiner.c | 6 ++----
> drivers/gpu/drm/bridge/imx/imx8qxp-pixel-link.c | 6 ++----
> drivers/gpu/drm/bridge/imx/imx8qxp-pxl2dpi.c | 6 ++----
> drivers/gpu/drm/bridge/lvds-codec.c | 6 ++----
> drivers/gpu/drm/bridge/nwl-dsi.c | 5 ++---
> drivers/gpu/drm/bridge/simple-bridge.c | 6 ++----
> drivers/gpu/drm/bridge/synopsys/dw-hdmi-ahb-audio.c | 6 ++----
> drivers/gpu/drm/bridge/synopsys/dw-hdmi-cec.c | 6 ++----
> drivers/gpu/drm/bridge/synopsys/dw-hdmi-gp-audio.c | 6 ++----
> drivers/gpu/drm/bridge/synopsys/dw-hdmi-i2s-audio.c | 6 ++----
> drivers/gpu/drm/bridge/thc63lvd1024.c | 6 ++----
> drivers/gpu/drm/bridge/ti-tfp410.c | 6 ++----
> drivers/gpu/drm/etnaviv/etnaviv_drv.c | 6 ++----
> drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 5 ++---
> drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 6 ++----
> drivers/gpu/drm/exynos/exynos7_drm_decon.c | 6 ++----
> drivers/gpu/drm/exynos/exynos_dp.c | 6 ++----
> drivers/gpu/drm/exynos/exynos_drm_drv.c | 5 ++---
> drivers/gpu/drm/exynos/exynos_drm_dsi.c | 6 ++----
> drivers/gpu/drm/exynos/exynos_drm_fimc.c | 6 ++----
> drivers/gpu/drm/exynos/exynos_drm_fimd.c | 6 ++----
> drivers/gpu/drm/exynos/exynos_drm_g2d.c | 6 ++----
> drivers/gpu/drm/exynos/exynos_drm_gsc.c | 6 ++----
> drivers/gpu/drm/exynos/exynos_drm_mic.c | 6 ++----
> drivers/gpu/drm/exynos/exynos_drm_rotator.c | 6 ++----
> drivers/gpu/drm/exynos/exynos_drm_scaler.c | 6 ++----
> drivers/gpu/drm/exynos/exynos_hdmi.c | 6 ++----
> drivers/gpu/drm/exynos/exynos_mixer.c | 6 ++----
> drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c | 6 ++----
> drivers/gpu/drm/hisilicon/kirin/dw_drm_dsi.c | 6 ++----
> drivers/gpu/drm/hisilicon/kirin/kirin_drm_drv.c | 5 ++---
> drivers/gpu/drm/imx/dcss/dcss-drv.c | 6 ++----
> drivers/gpu/drm/imx/ipuv3/dw_hdmi-imx.c | 6 ++----
> drivers/gpu/drm/imx/ipuv3/imx-drm-core.c | 5 ++---
> drivers/gpu/drm/imx/ipuv3/imx-ldb.c | 5 ++---
> drivers/gpu/drm/imx/ipuv3/imx-tve.c | 5 ++---
> drivers/gpu/drm/imx/ipuv3/ipuv3-crtc.c | 5 ++---
> drivers/gpu/drm/imx/ipuv3/parallel-display.c | 6 ++----
> drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 6 ++----
> drivers/gpu/drm/ingenic/ingenic-ipu.c | 5 ++---
> drivers/gpu/drm/kmb/kmb_drv.c | 5 ++---
> drivers/gpu/drm/lima/lima_drv.c | 5 ++---
> drivers/gpu/drm/logicvc/logicvc_drm.c | 6 ++----
> drivers/gpu/drm/mcde/mcde_drv.c | 6 ++----
> drivers/gpu/drm/mcde/mcde_dsi.c | 6 ++----
> drivers/gpu/drm/mediatek/mtk_cec.c | 5 ++---
> drivers/gpu/drm/mediatek/mtk_disp_aal.c | 6 ++----
> drivers/gpu/drm/mediatek/mtk_disp_ccorr.c | 6 ++----
> drivers/gpu/drm/mediatek/mtk_disp_color.c | 6 ++----
> drivers/gpu/drm/mediatek/mtk_disp_gamma.c | 6 ++----
> drivers/gpu/drm/mediatek/mtk_disp_merge.c | 6 ++----
> drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 6 ++----
> drivers/gpu/drm/mediatek/mtk_disp_rdma.c | 6 ++----
> drivers/gpu/drm/mediatek/mtk_dp.c | 6 ++----
> drivers/gpu/drm/mediatek/mtk_dpi.c | 6 ++----
> drivers/gpu/drm/mediatek/mtk_drm_drv.c | 6 ++----
> drivers/gpu/drm/mediatek/mtk_dsi.c | 6 ++----
> drivers/gpu/drm/mediatek/mtk_hdmi.c | 5 ++---
> drivers/gpu/drm/mediatek/mtk_hdmi_ddc.c | 6 ++----
> drivers/gpu/drm/mediatek/mtk_mdp_rdma.c | 5 ++---
> drivers/gpu/drm/meson/meson_drv.c | 6 ++----
> drivers/gpu/drm/meson/meson_dw_hdmi.c | 6 ++----
> drivers/gpu/drm/msm/adreno/adreno_device.c | 5 ++---
> drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 6 ++----
> drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c | 6 ++----
> drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c | 5 ++---
> drivers/gpu/drm/msm/dp/dp_display.c | 6 ++----
> drivers/gpu/drm/msm/dsi/dsi.c | 6 ++----
> drivers/gpu/drm/msm/hdmi/hdmi.c | 6 ++----
> drivers/gpu/drm/msm/hdmi/hdmi_phy.c | 6 ++----
> drivers/gpu/drm/msm/msm_drv.c | 6 ++----
> drivers/gpu/drm/msm/msm_mdss.c | 6 ++----
> drivers/gpu/drm/mxsfb/lcdif_drv.c | 6 ++----
> drivers/gpu/drm/mxsfb/mxsfb_drv.c | 6 ++----
> drivers/gpu/drm/nouveau/nouveau_platform.c | 5 ++---
> drivers/gpu/drm/omapdrm/dss/dispc.c | 5 ++---
> drivers/gpu/drm/omapdrm/dss/dsi.c | 6 ++----
> drivers/gpu/drm/omapdrm/dss/dss.c | 6 ++----
> drivers/gpu/drm/omapdrm/dss/hdmi4.c | 5 ++---
> drivers/gpu/drm/omapdrm/dss/hdmi5.c | 5 ++---
> drivers/gpu/drm/omapdrm/dss/venc.c | 5 ++---
> drivers/gpu/drm/omapdrm/omap_dmm_tiler.c | 9 +++------
> drivers/gpu/drm/omapdrm/omap_drv.c | 6 ++----
> drivers/gpu/drm/panel/panel-lvds.c | 6 ++----
> drivers/gpu/drm/panel/panel-seiko-43wvf1g.c | 6 ++----
> drivers/gpu/drm/panel/panel-sharp-ls037v7dw01.c | 6 ++----
> drivers/gpu/drm/panel/panel-simple.c | 6 ++----
> drivers/gpu/drm/panfrost/panfrost_drv.c | 5 ++---
> drivers/gpu/drm/rcar-du/rcar_cmm.c | 6 ++----
> drivers/gpu/drm/rcar-du/rcar_du_drv.c | 6 ++----
> drivers/gpu/drm/rcar-du/rcar_dw_hdmi.c | 6 ++----
> drivers/gpu/drm/rcar-du/rcar_lvds.c | 6 ++----
> drivers/gpu/drm/rcar-du/rcar_mipi_dsi.c | 6 ++----
> drivers/gpu/drm/rcar-du/rzg2l_mipi_dsi.c | 6 ++----
> drivers/gpu/drm/rockchip/analogix_dp-rockchip.c | 6 ++----
> drivers/gpu/drm/rockchip/cdn-dp-core.c | 6 ++----
> drivers/gpu/drm/rockchip/dw-mipi-dsi-rockchip.c | 6 ++----
> drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c | 6 ++----
> drivers/gpu/drm/rockchip/inno_hdmi.c | 6 ++----
> drivers/gpu/drm/rockchip/rk3066_hdmi.c | 6 ++----
> drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 6 ++----
> drivers/gpu/drm/rockchip/rockchip_lvds.c | 6 ++----
> drivers/gpu/drm/rockchip/rockchip_vop2_reg.c | 6 ++----
> drivers/gpu/drm/rockchip/rockchip_vop_reg.c | 6 ++----
> drivers/gpu/drm/shmobile/shmob_drm_drv.c | 6 ++----
> drivers/gpu/drm/sprd/sprd_dpu.c | 6 ++----
> drivers/gpu/drm/sprd/sprd_drm.c | 5 ++---
> drivers/gpu/drm/sprd/sprd_dsi.c | 6 ++----
> drivers/gpu/drm/sti/sti_compositor.c | 5 ++---
> drivers/gpu/drm/sti/sti_drv.c | 6 ++----
> drivers/gpu/drm/sti/sti_dvo.c | 5 ++---
> drivers/gpu/drm/sti/sti_hda.c | 5 ++---
> drivers/gpu/drm/sti/sti_hdmi.c | 6 ++----
> drivers/gpu/drm/sti/sti_hqvdp.c | 5 ++---
> drivers/gpu/drm/sti/sti_tvout.c | 5 ++---
> drivers/gpu/drm/stm/drv.c | 6 ++----
> drivers/gpu/drm/stm/dw_mipi_dsi-stm.c | 6 ++----
> drivers/gpu/drm/sun4i/sun4i_backend.c | 6 ++----
> drivers/gpu/drm/sun4i/sun4i_drv.c | 6 ++----
> drivers/gpu/drm/sun4i/sun4i_frontend.c | 6 ++----
> drivers/gpu/drm/sun4i/sun4i_hdmi_enc.c | 6 ++----
> drivers/gpu/drm/sun4i/sun4i_tcon.c | 6 ++----
> drivers/gpu/drm/sun4i/sun4i_tv.c | 6 ++----
> drivers/gpu/drm/sun4i/sun6i_drc.c | 6 ++----
> drivers/gpu/drm/sun4i/sun6i_mipi_dsi.c | 6 ++----
> drivers/gpu/drm/sun4i/sun8i_dw_hdmi.c | 6 ++----
> drivers/gpu/drm/sun4i/sun8i_mixer.c | 6 ++----
> drivers/gpu/drm/sun4i/sun8i_tcon_top.c | 6 ++----
> drivers/gpu/drm/tegra/dpaux.c | 6 ++----
> drivers/gpu/drm/tests/drm_kunit_helpers.c | 5 ++---
> drivers/gpu/drm/tidss/tidss_drv.c | 6 ++----
> drivers/gpu/drm/tilcdc/tilcdc_panel.c | 6 ++----
> drivers/gpu/drm/tiny/arcpgu.c | 6 ++----
> drivers/gpu/drm/tiny/ofdrm.c | 6 ++----
> drivers/gpu/drm/tiny/simpledrm.c | 6 ++----
> drivers/gpu/drm/tve200/tve200_drv.c | 6 ++----
> drivers/gpu/drm/v3d/v3d_drv.c | 6 ++----
> drivers/gpu/drm/vc4/vc4_crtc.c | 5 ++---
> drivers/gpu/drm/vc4/vc4_dpi.c | 5 ++---
> drivers/gpu/drm/vc4/vc4_drv.c | 6 ++----
> drivers/gpu/drm/vc4/vc4_dsi.c | 6 ++----
> drivers/gpu/drm/vc4/vc4_hdmi.c | 5 ++---
> drivers/gpu/drm/vc4/vc4_hvs.c | 5 ++---
> drivers/gpu/drm/vc4/vc4_txp.c | 5 ++---
> drivers/gpu/drm/vc4/vc4_v3d.c | 5 ++---
> drivers/gpu/drm/vc4/vc4_vec.c | 5 ++---
> drivers/gpu/drm/xlnx/zynqmp_dpsub.c | 6 ++----
> 159 files changed, 319 insertions(+), 597 deletions(-)
>
>
> base-commit: 457391b0380335d5e9a5babdec90ac53928b23b4
--
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)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 840 bytes
Desc: OpenPGP digital signature
URL: <http://lists.ozlabs.org/pipermail/linux-aspeed/attachments/20230508/83f8e808/attachment-0001.sig>
next prev parent reply other threads:[~2023-05-08 7:06 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-07 16:25 [PATCH 00/53] drm: Convert to platform remove callback returning void Uwe Kleine-König
2023-05-07 16:25 ` [PATCH 05/53] drm/aspeed: " Uwe Kleine-König
2023-05-08 7:06 ` Thomas Zimmermann [this message]
2023-05-08 7:50 ` [PATCH 00/53] drm: " Uwe Kleine-König
2023-05-15 7:50 ` Inki Dae
2023-05-15 9:20 ` Uwe Kleine-König
2023-06-01 15:40 ` Uwe Kleine-König
2023-06-08 16:08 ` Doug Anderson
2023-06-08 16:26 ` Laurent Pinchart
2023-06-08 16:47 ` Doug Anderson
2023-06-08 17:19 ` Tomi Valkeinen
2023-06-08 17:38 ` Doug Anderson
2023-06-17 16:12 ` patches dropped from drm-misc-next [Was: Re: [PATCH 00/53] drm: Convert to platform remove callback returning] void Uwe Kleine-König
2023-06-17 16:51 ` Chen-Yu Tsai
2023-06-17 17:57 ` Doug Anderson
2023-06-18 12:39 ` Uwe Kleine-König
2023-06-18 14:02 ` Uwe Kleine-König
2023-06-18 14:32 ` Maxime Ripard
2023-06-18 16:29 ` Uwe Kleine-König
2023-06-19 9:45 ` Maxime Ripard
2023-06-19 10:53 ` Uwe Kleine-König
2023-06-19 12:47 ` Maxime Ripard
2023-06-19 13:25 ` Geert Uytterhoeven
2023-06-19 14:02 ` Maxime Ripard
2023-06-19 14:20 ` Geert Uytterhoeven
2023-06-19 13:58 ` Uwe Kleine-König
2023-09-09 14:37 ` [PATCH 00/53] drm: Convert to platform remove callback returning void Javier Martinez Canillas
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=935faac5-280b-b2e0-3fdb-d0424990e43a@suse.de \
--to=tzimmermann@suse.de \
--cc=linux-aspeed@lists.ozlabs.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox