Chrome platform driver development
 help / color / mirror / Atom feed
From: Louis Chauvet <louis.chauvet@bootlin.com>
To: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
	Maxime Ripard <mripard@kernel.org>,
	Thomas Zimmermann <tzimmermann@suse.de>,
	David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
	Andrzej Hajda <andrzej.hajda@intel.com>,
	Neil Armstrong <neil.armstrong@linaro.org>,
	Robert Foss <rfoss@kernel.org>,
	Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
	Jonas Karlman <jonas@kwiboo.se>,
	Jernej Skrabec <jernej.skrabec@gmail.com>,
	Jagan Teki <jagan@amarulasolutions.com>,
	Shawn Guo <shawnguo@kernel.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Fabio Estevam <festevam@gmail.com>,
	Douglas Anderson <dianders@chromium.org>,
	Chun-Kuang Hu <chunkuang.hu@kernel.org>,
	Krzysztof Kozlowski <krzk@kernel.org>,
	Luca Ceresoli <luca.ceresoli@bootlin.com>
Cc: "Anusha Srivatsa" <asrivats@redhat.com>,
	"Paul Kocialkowski" <paulk@sys-base.io>,
	"Dmitry Baryshkov" <lumag@kernel.org>,
	"Hui Pu" <Hui.Pu@gehealthcare.com>,
	"Thomas Petazzoni" <thomas.petazzoni@bootlin.com>,
	dri-devel@lists.freedesktop.org, asahi@lists.linux.dev,
	linux-kernel@vger.kernel.org, chrome-platform@lists.linux.dev,
	imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org,
	linux-mediatek@lists.infradead.org,
	linux-amlogic@lists.infradead.org,
	linux-renesas-soc@vger.kernel.org,
	platform-driver-x86@vger.kernel.org,
	linux-samsung-soc@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	freedreno@lists.freedesktop.org,
	linux-stm32@st-md-mailman.stormreply.com,
	"Adam Ford" <aford173@gmail.com>,
	"Adrien Grassein" <adrien.grassein@gmail.com>,
	"Aleksandr Mishin" <amishin@t-argos.ru>,
	"Andy Yan" <andy.yan@rock-chips.com>,
	"AngeloGioacchino Del Regno"
	<angelogioacchino.delregno@collabora.com>,
	"Benson Leung" <bleung@chromium.org>,
	"Biju Das" <biju.das.jz@bp.renesas.com>,
	"Christoph Fritz" <chf.fritz@googlemail.com>,
	"Cristian Ciocaltea" <cristian.ciocaltea@collabora.com>,
	"Detlev Casanova" <detlev.casanova@collabora.com>,
	"Dharma Balasubiramani" <dharma.b@microchip.com>,
	"Guenter Roeck" <groeck@chromium.org>,
	"Heiko Stuebner" <heiko@sntech.de>,
	"Jani Nikula" <jani.nikula@intel.com>,
	"Janne Grunau" <j@jannau.net>,
	"Jerome Brunet" <jbrunet@baylibre.com>,
	"Jesse Van Gavere" <jesseevg@gmail.com>,
	"Kevin Hilman" <khilman@baylibre.com>,
	"Kieran Bingham" <kieran.bingham+renesas@ideasonboard.com>,
	"Liu Ying" <victor.liu@nxp.com>,
	"Manikandan Muralidharan" <manikandan.m@microchip.com>,
	"Martin Blumenstingl" <martin.blumenstingl@googlemail.com>,
	"Matthias Brugger" <matthias.bgg@gmail.com>,
	"Philipp Zabel" <p.zabel@pengutronix.de>,
	"Phong LE" <ple@baylibre.com>,
	"Sasha Finkelstein" <fnkl.kernel@gmail.com>,
	"Sugar Zhang" <sugar.zhang@rock-chips.com>,
	"Sui Jingfeng" <sui.jingfeng@linux.dev>,
	"Tomi Valkeinen" <tomi.valkeinen+renesas@ideasonboard.com>,
	"Vitalii Mordan" <mordan@ispras.ru>,
	"Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>,
	"Bryan O'Donoghue" <bryan.odonoghue@linaro.org>,
	"Hans de Goede" <hdegoede@redhat.com>,
	"Uwe Kleine-König" <u.kleine-koenig@baylibre.com>,
	"Dmitry Baryshkov" <dmitry.baryshkov@oss.qualcomm.com>,
	"Rob Herring (Arm)" <robh@kernel.org>,
	"Hsin-Te Yuan" <yuanhsinte@chromium.org>,
	"Pin-yen Lin" <treapking@chromium.org>,
	"Xin Ji" <xji@analogixsemi.com>,
	"Aradhya Bhatia" <a-bhatia1@ti.com>,
	"Tomi Valkeinen" <tomi.valkeinen@ideasonboard.com>,
	"Ian Ray" <ian.ray@ge.com>,
	"Martyn Welch" <martyn.welch@collabora.co.uk>,
	"Peter Senna Tschudin" <peter.senna@gmail.com>,
	"Russell King" <linux@armlinux.org.uk>,
	"Herve Codina" <herve.codina@bootlin.com>,
	"Alim Akhtar" <alim.akhtar@samsung.com>,
	"Inki Dae" <inki.dae@samsung.com>,
	"Kyungmin Park" <kyungmin.park@samsung.com>,
	"Seung-Woo Kim" <sw0312.kim@samsung.com>,
	"Linus Walleij" <linus.walleij@linaro.org>,
	"Abhinav Kumar" <quic_abhinavk@quicinc.com>,
	"Bjorn Andersson" <quic_bjorande@quicinc.com>,
	"Marijn Suijten" <marijn.suijten@somainline.org>,
	"Rob Clark" <robdclark@gmail.com>, "Sean Paul" <sean@poorly.run>,
	"Helge Deller" <deller@gmx.de>,
	"Kuninori Morimoto" <kuninori.morimoto.gx@renesas.com>,
	"Laurent Pinchart" <laurent.pinchart+renesas@ideasonboard.com>,
	"Alexandre Torgue" <alexandre.torgue@foss.st.com>,
	"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
	"Philippe Cornu" <philippe.cornu@foss.st.com>,
	"Raphael Gallais-Pou" <raphael.gallais-pou@foss.st.com>,
	"Yannick Fertre" <yannick.fertre@foss.st.com>,
	"Maíra Canal" <mcanal@igalia.com>,
	"Dave Stevenson" <dave.stevenson@raspberrypi.com>,
	"Raspberry Pi Kernel Maintenance" <kernel-list@raspberrypi.com>,
	"Alain Volmat" <alain.volmat@foss.st.com>,
	"Raphael Gallais-Pou" <rgallaispou@gmail.com>,
	"Michal Simek" <michal.simek@amd.com>
Subject: Re: (subset) [PATCH v2 00/34] drm: convert all bridges to devm_drm_bridge_alloc()
Date: Tue, 29 Apr 2025 14:41:42 +0200	[thread overview]
Message-ID: <832a9db0-cf8a-4d35-8a98-08053fbd6723@bootlin.com> (raw)
In-Reply-To: <174591887152.961603.7706063017853945511.b4-ty@bootlin.com>



Le 29/04/2025 à 11:27, Louis Chauvet a écrit :
> 
> On Thu, 24 Apr 2025 20:59:07 +0200, Luca Ceresoli wrote:
>> devm_drm_bridge_alloc() [0] is the new API to allocate and initialize a DRM
>> bridge, and the only one supported from now on. It is also necessary for
>> implementing reference counting and thus needed to support removal of
>> bridges from a still existing DRM pipeline without use-after-free.
>>
>> This series converts all DRM bridges to the new API.
>>
>> [...]
> 
> Applied, thanks!
> 
> [02/34] platform: arm64: acer-aspire1-ec: convert to devm_drm_bridge_alloc() API
>          commit: 411465d35bc56877c33e2498ac697acfcf484e6b
> [03/34] drm/bridge: analogix-anx6345: convert to devm_drm_bridge_alloc() API
>          commit: 53ddeb25159781b029fda404226af600e76f975f
> [06/34] drm/bridge: display-connector: convert to devm_drm_bridge_alloc() API
>          commit: 4e90a3d96a6185e143041273f9867a1092dd4a71
> [07/34] drm/bridge: lt9611uxc: convert to devm_drm_bridge_alloc() API
>          commit: 6287ffd9eff6eea65865e64b9d4c45e115fa5ecf
> [11/34] drm/bridge: dw-hdmi: convert to devm_drm_bridge_alloc() API
>          commit: ed6987b674185873ebed7a619a646da6dd1a78fa
> [12/34] drm/bridge: tda998x: convert to devm_drm_bridge_alloc() API
>          commit: 7fe58bf1a9a24b533875c262a3222581a3f759e4
> [13/34] drm/bridge: ti-sn65dsi86: convert to devm_drm_bridge_alloc() API
>          commit: a4754ae9cfa76fbce79f023c268a5bac56f36321
> [14/34] drm/exynos: mic: convert to devm_drm_bridge_alloc() API
>          commit: 91c5c7b5bb2dd09b43b025bce6d790d3c79f4518

Hello all,

I made a mistake while applying those patches. Instead of taking 
2-3,11-13,15-18,27,31, I took 2-3,11-18,27,31, which includes the patch 14.

This patch is not R-by/A-by/SoB by non-Bootlin people. For me, the patch 
is correct, it does not break the build nor generate warnings.

What should I do? Is my SoB sufficient, or should I revert the patch and 
wait for R/A-by before taking it again?

Sorry for this,
Louis Chauvet

> [15/34] drm/mcde: convert to devm_drm_bridge_alloc() API
>          commit: 40c25b9ec641f43ba17c7b788ac16ec23f8daaa8
> [16/34] drm/msm/dp: convert to devm_drm_bridge_alloc() API
>          commit: b2aabe5c6b65516d88214aba4b12ce2ca78bac6c
> [17/34] drm/msm/dsi: convert to devm_drm_bridge_alloc() API
>          commit: fffc8847743e45604c4478f554d628481b985556
> [18/34] drm/msm/hdmi: convert to devm_drm_bridge_alloc() API
>          commit: e11532be87e437648521a8ed5358c56df11933b4
> [27/34] drm/vc4: convert to devm_drm_bridge_alloc() API
>          commit: 9545c91ed75ff65e114761a7729de0e1b440aec6
> [31/34] drm/bridge: imx8*-ldb: convert to devm_drm_bridge_alloc() API
>          commit: e74b84cd83962e357329a695ba348b3dfe37395c
> 
> Best regards,

-- 
Louis Chauvet, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com


  reply	other threads:[~2025-04-29 12:41 UTC|newest]

Thread overview: 81+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-04-24 18:59 [PATCH v2 00/34] drm: convert all bridges to devm_drm_bridge_alloc() Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 01/34] drm: convert many bridge drivers from devm_kzalloc() to devm_drm_bridge_alloc() API Luca Ceresoli
2025-04-28 12:44   ` Andy Yan
2025-04-28 15:00     ` [PATCH " Luca Ceresoli
2025-04-28 20:59   ` Doug Anderson
2025-04-30 10:35     ` Luca Ceresoli
2025-04-30 15:51       ` Doug Anderson
2025-04-30 16:42         ` Luca Ceresoli
2025-04-29  2:19   ` Liu Ying
2025-04-29  7:07     ` Luca Ceresoli
2025-04-30  9:42   ` Manikandan.M
2025-04-30 10:36     ` Luca Ceresoli
2025-05-05  5:20       ` Manikandan.M
2025-04-24 18:59 ` [PATCH v2 02/34] platform: arm64: acer-aspire1-ec: convert " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 03/34] drm/bridge: analogix-anx6345: " Luca Ceresoli
2025-04-28 12:29   ` Andy Yan
2025-04-24 18:59 ` [PATCH v2 04/34] drm/bridge: anx7625: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 05/34] drm/bridge: cdns-dsi: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 06/34] drm/bridge: display-connector: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 07/34] drm/bridge: lt9611uxc: " Luca Ceresoli
2025-04-29 12:09   ` Dmitry Baryshkov
2025-04-24 18:59 ` [PATCH v2 08/34] drm/bridge: megachips-stdpxxxx-ge-b850v3-fw: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 09/34] drm/bridge: nxp-ptn3460: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 10/34] drm/bridge: sii902x: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 11/34] drm/bridge: dw-hdmi: " Luca Ceresoli
2025-04-24 19:16   ` Cristian Ciocaltea
2025-04-24 18:59 ` [PATCH v2 12/34] drm/bridge: tda998x: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 13/34] drm/bridge: ti-sn65dsi86: " Luca Ceresoli
2025-04-28 20:53   ` Doug Anderson
2025-04-24 18:59 ` [PATCH v2 14/34] drm/exynos: mic: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 15/34] drm/mcde: " Luca Ceresoli
2025-04-29  8:40   ` Linus Walleij
2025-04-24 18:59 ` [PATCH v2 16/34] drm/msm/dp: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 17/34] drm/msm/dsi: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 18/34] drm/msm/hdmi: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 19/34] drm/omap: dss: dpi: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 20/34] drm/omap: dss: dsi: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 21/34] drm/omap: dss: hdmi4: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 22/34] drm/omap: dss: hdmi5: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 23/34] drm/omap: dss: sdi: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 24/34] drm/omap: dss: venc: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 25/34] drm/rcar-du: dsi: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 26/34] drm/bridge: stm_lvds: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 27/34] drm/vc4: " Luca Ceresoli
2025-04-28 15:45   ` Dave Stevenson
2025-04-24 18:59 ` [PATCH v2 28/34] drm/sti: dvo: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 29/34] drm: zynqmp_dp: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 30/34] drm/bridge: imx8qxp-pixel-combiner: " Luca Ceresoli
2025-04-29  2:10   ` Liu Ying
2025-04-30  9:29     ` Luca Ceresoli
2025-05-06  2:24       ` Liu Ying
2025-05-06 20:47         ` Luca Ceresoli
2025-05-07  2:10           ` Liu Ying
2025-05-07  7:12             ` Luca Ceresoli
2025-05-07 10:16               ` Liu Ying
2025-05-07 14:13                 ` Luca Ceresoli
2025-05-22  3:01           ` Liu Ying
2025-05-26  7:20             ` Luca Ceresoli
2025-05-27  1:42               ` Liu Ying
2025-04-24 18:59 ` [PATCH v2 31/34] drm/bridge: imx8*-ldb: " Luca Ceresoli
2025-04-29  2:35   ` Liu Ying
2025-04-24 18:59 ` [PATCH v2 32/34] drm/bridge: tc358767: " Luca Ceresoli
2025-04-24 18:59 ` [PATCH v2 33/34] drm/bridge: add devm_drm_put_bridge() Luca Ceresoli
2025-04-24 20:05 ` [PATCH v2 34/34] drm/bridge: panel: convert to devm_drm_bridge_alloc() API Luca Ceresoli
2025-04-28 11:39   ` Maxime Ripard
2025-04-28 15:25     ` Luca Ceresoli
2025-05-05  6:23       ` Maxime Ripard
2025-05-05 15:20         ` Luca Ceresoli
2025-04-28 15:24 ` [PATCH v2 00/34] drm: convert all bridges to devm_drm_bridge_alloc() Luca Ceresoli
2025-04-28 15:42   ` Maxime Ripard
2025-04-28 16:33     ` Luca Ceresoli
2025-04-29  9:27 ` (subset) " Louis Chauvet
2025-04-29 12:41   ` Louis Chauvet [this message]
2025-04-30  8:08     ` Maxime Ripard
2025-05-05 11:06       ` Luca Ceresoli
2025-05-05 11:58         ` Dmitry Baryshkov
2025-05-05 12:31           ` Luca Ceresoli
2025-04-29 14:42   ` Dmitry Baryshkov
2025-04-30  8:21     ` Louis Chauvet
2025-04-30 10:39       ` Maxime Ripard
2025-04-30 15:30         ` Louis Chauvet

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=832a9db0-cf8a-4d35-8a98-08053fbd6723@bootlin.com \
    --to=louis.chauvet@bootlin.com \
    --cc=Hui.Pu@gehealthcare.com \
    --cc=Laurent.pinchart@ideasonboard.com \
    --cc=a-bhatia1@ti.com \
    --cc=adrien.grassein@gmail.com \
    --cc=aford173@gmail.com \
    --cc=airlied@gmail.com \
    --cc=alain.volmat@foss.st.com \
    --cc=alexandre.torgue@foss.st.com \
    --cc=alim.akhtar@samsung.com \
    --cc=amishin@t-argos.ru \
    --cc=andrzej.hajda@intel.com \
    --cc=andy.yan@rock-chips.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=asahi@lists.linux.dev \
    --cc=asrivats@redhat.com \
    --cc=biju.das.jz@bp.renesas.com \
    --cc=bleung@chromium.org \
    --cc=bryan.odonoghue@linaro.org \
    --cc=chf.fritz@googlemail.com \
    --cc=chrome-platform@lists.linux.dev \
    --cc=chunkuang.hu@kernel.org \
    --cc=cristian.ciocaltea@collabora.com \
    --cc=dave.stevenson@raspberrypi.com \
    --cc=deller@gmx.de \
    --cc=detlev.casanova@collabora.com \
    --cc=dharma.b@microchip.com \
    --cc=dianders@chromium.org \
    --cc=dmitry.baryshkov@oss.qualcomm.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=festevam@gmail.com \
    --cc=fnkl.kernel@gmail.com \
    --cc=freedreno@lists.freedesktop.org \
    --cc=groeck@chromium.org \
    --cc=hdegoede@redhat.com \
    --cc=heiko@sntech.de \
    --cc=herve.codina@bootlin.com \
    --cc=ian.ray@ge.com \
    --cc=ilpo.jarvinen@linux.intel.com \
    --cc=imx@lists.linux.dev \
    --cc=inki.dae@samsung.com \
    --cc=j@jannau.net \
    --cc=jagan@amarulasolutions.com \
    --cc=jani.nikula@intel.com \
    --cc=jbrunet@baylibre.com \
    --cc=jernej.skrabec@gmail.com \
    --cc=jesseevg@gmail.com \
    --cc=jonas@kwiboo.se \
    --cc=kernel-list@raspberrypi.com \
    --cc=kernel@pengutronix.de \
    --cc=khilman@baylibre.com \
    --cc=kieran.bingham+renesas@ideasonboard.com \
    --cc=krzk@kernel.org \
    --cc=kuninori.morimoto.gx@renesas.com \
    --cc=kyungmin.park@samsung.com \
    --cc=laurent.pinchart+renesas@ideasonboard.com \
    --cc=linus.walleij@linaro.org \
    --cc=linux-amlogic@lists.infradead.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mediatek@lists.infradead.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux-stm32@st-md-mailman.stormreply.com \
    --cc=linux@armlinux.org.uk \
    --cc=luca.ceresoli@bootlin.com \
    --cc=lumag@kernel.org \
    --cc=maarten.lankhorst@linux.intel.com \
    --cc=manikandan.m@microchip.com \
    --cc=marijn.suijten@somainline.org \
    --cc=martin.blumenstingl@googlemail.com \
    --cc=martyn.welch@collabora.co.uk \
    --cc=matthias.bgg@gmail.com \
    --cc=mcanal@igalia.com \
    --cc=mcoquelin.stm32@gmail.com \
    --cc=michal.simek@amd.com \
    --cc=mordan@ispras.ru \
    --cc=mripard@kernel.org \
    --cc=neil.armstrong@linaro.org \
    --cc=p.zabel@pengutronix.de \
    --cc=paulk@sys-base.io \
    --cc=peter.senna@gmail.com \
    --cc=philippe.cornu@foss.st.com \
    --cc=platform-driver-x86@vger.kernel.org \
    --cc=ple@baylibre.com \
    --cc=quic_abhinavk@quicinc.com \
    --cc=quic_bjorande@quicinc.com \
    --cc=raphael.gallais-pou@foss.st.com \
    --cc=rfoss@kernel.org \
    --cc=rgallaispou@gmail.com \
    --cc=robdclark@gmail.com \
    --cc=robh@kernel.org \
    --cc=s.hauer@pengutronix.de \
    --cc=sean@poorly.run \
    --cc=shawnguo@kernel.org \
    --cc=simona@ffwll.ch \
    --cc=sugar.zhang@rock-chips.com \
    --cc=sui.jingfeng@linux.dev \
    --cc=sw0312.kim@samsung.com \
    --cc=thomas.petazzoni@bootlin.com \
    --cc=tomi.valkeinen+renesas@ideasonboard.com \
    --cc=tomi.valkeinen@ideasonboard.com \
    --cc=treapking@chromium.org \
    --cc=tzimmermann@suse.de \
    --cc=u.kleine-koenig@baylibre.com \
    --cc=victor.liu@nxp.com \
    --cc=xji@analogixsemi.com \
    --cc=yannick.fertre@foss.st.com \
    --cc=yuanhsinte@chromium.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