Message-ID: <20250523152304.5c66e195@booty>
Date: Fri, 23 May 2025 15:23:04 +0200
From: Luca Ceresoli <luca.ceresoli@...tlin.com>
To: Inki Dae <daeinki@...il.com>
Cc: Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>, Maxime Ripard
 <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>, David Airlie
 <airlied@...il.com>, Simona Vetter <simona@...ll.ch>, Andrzej Hajda
 <andrzej.hajda@...el.com>, Neil Armstrong <neil.armstrong@...aro.org>,
 Robert Foss <rfoss@...nel.org>, Laurent Pinchart
 <Laurent.pinchart@...asonboard.com>, Jonas Karlman <jonas@...boo.se>,
 Jernej Skrabec <jernej.skrabec@...il.com>, Jagan Teki
 <jagan@...rulasolutions.com>, Shawn Guo <shawnguo@...nel.org>, Sascha Hauer
 <s.hauer@...gutronix.de>, Pengutronix Kernel Team <kernel@...gutronix.de>,
 Fabio Estevam <festevam@...il.com>, Douglas Anderson
 <dianders@...omium.org>, Chun-Kuang Hu <chunkuang.hu@...nel.org>, Krzysztof
 Kozlowski <krzk@...nel.org>, Liu Ying <victor.liu@....com>, Anusha Srivatsa
 <asrivats@...hat.com>, Paul Kocialkowski <paulk@...-base.io>, Dmitry
 Baryshkov <lumag@...nel.org>, Hui Pu <Hui.Pu@...ealthcare.com>, Thomas
 Petazzoni <thomas.petazzoni@...tlin.com>, dri-devel@...ts.freedesktop.org,
 asahi@...ts.linux.dev, linux-kernel@...r.kernel.org,
 chrome-platform@...ts.linux.dev, imx@...ts.linux.dev,
 linux-arm-kernel@...ts.infradead.org, linux-mediatek@...ts.infradead.org,
 linux-amlogic@...ts.infradead.org, linux-renesas-soc@...r.kernel.org,
 platform-driver-x86@...r.kernel.org, linux-samsung-soc@...r.kernel.org,
 linux-arm-msm@...r.kernel.org, freedreno@...ts.freedesktop.org,
 linux-stm32@...md-mailman.stormreply.com, Louis Chauvet
 <louis.chauvet@...tlin.com>, Alim Akhtar <alim.akhtar@...sung.com>,
 Kyungmin Park <kyungmin.park@...sung.com>, Seung-Woo Kim
 <sw0312.kim@...sung.com>, Manikandan Muralidharan
 <manikandan.m@...rochip.com>, Adam Ford <aford173@...il.com>, Adrien
 Grassein <adrien.grassein@...il.com>, Aleksandr Mishin
 <amishin@...rgos.ru>, Andy Yan <andy.yan@...k-chips.com>, AngeloGioacchino
 Del Regno <angelogioacchino.delregno@...labora.com>, Benson Leung
 <bleung@...omium.org>, Biju Das <biju.das.jz@...renesas.com>, Christoph
 Fritz <chf.fritz@...glemail.com>, Cristian Ciocaltea
 <cristian.ciocaltea@...labora.com>, Detlev Casanova
 <detlev.casanova@...labora.com>, Dharma Balasubiramani
 <dharma.b@...rochip.com>, Guenter Roeck <groeck@...omium.org>, Heiko
 Stuebner <heiko@...ech.de>, Jani Nikula <jani.nikula@...el.com>, Janne
 Grunau <j@...nau.net>, Jerome Brunet <jbrunet@...libre.com>, Jesse Van
 Gavere <jesseevg@...il.com>, Kevin Hilman <khilman@...libre.com>, Kieran
 Bingham <kieran.bingham+renesas@...asonboard.com>, Martin Blumenstingl
 <martin.blumenstingl@...glemail.com>, Matthias Brugger
 <matthias.bgg@...il.com>, Philipp Zabel <p.zabel@...gutronix.de>, Phong LE
 <ple@...libre.com>, Sasha Finkelstein <fnkl.kernel@...il.com>, Sugar Zhang
 <sugar.zhang@...k-chips.com>, Sui Jingfeng <sui.jingfeng@...ux.dev>, Tomi
 Valkeinen <tomi.valkeinen+renesas@...asonboard.com>, Vitalii Mordan
 <mordan@...ras.ru>, "Rob Herring (Arm)" <robh@...nel.org>, Hsin-Te Yuan
 <yuanhsinte@...omium.org>, Pin-yen Lin <treapking@...omium.org>, Xin Ji
 <xji@...logixsemi.com>, Aradhya Bhatia <a-bhatia1@...com>, Tomi Valkeinen
 <tomi.valkeinen@...asonboard.com>, Ian Ray <ian.ray@...ealthcare.com>,
 Martyn Welch <martyn.welch@...labora.co.uk>, Peter Senna Tschudin
 <peter.senna@...il.com>, Helge Deller <deller@....de>, Kuninori Morimoto
 <kuninori.morimoto.gx@...esas.com>, Laurent Pinchart
 <laurent.pinchart+renesas@...asonboard.com>, Alexandre Torgue
 <alexandre.torgue@...s.st.com>, Maxime Coquelin
 <mcoquelin.stm32@...il.com>, Philippe Cornu <philippe.cornu@...s.st.com>,
 Raphael Gallais-Pou <raphael.gallais-pou@...s.st.com>, Yannick Fertre
 <yannick.fertre@...s.st.com>, Alain Volmat <alain.volmat@...s.st.com>,
 Raphael Gallais-Pou <rgallaispou@...il.com>, Michal Simek
 <michal.simek@....com>, Jonathan Corbet <corbet@....net>,
 linux-doc@...r.kernel.org
Subject: Re: [PATCH v3 00/22] drm: convert all bridges to
 devm_drm_bridge_alloc()

Hello Inki,

On Fri, 23 May 2025 00:11:24 +0900
Inki Dae <daeinki@...il.com> wrote:

> Hello Luca Ceresoli,
> 
> 2025년 5월 21일 (수) 오후 11:23, Luca Ceresoli <luca.ceresoli@...tlin.com>님이 작성:
> >
> > Hello Maxime, Shawn, Liu, all,
> >
> > On Fri, 09 May 2025 15:53:26 +0200
> > Luca Ceresoli <luca.ceresoli@...tlin.com> 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 the first milestone
> > > towards removal of bridges from a still existing DRM pipeline without
> > > use-after-free.  
> >
> > I applied on drm-misc-next patches 3-17,20-21 as they match all the
> > criteria:
> >  - At least a Acked-by (or R-by maintainers)
> >  - patch is for drm-misc
> >
> > Being my very first commits to drm-misc, I tried to be careful, and
> > double checked all the patches with Louis (thanks!).
> >
> > Here are the pending questions and plan for the remaining patches.
> >  
> > >       Revert "drm/exynos: mic: convert to devm_drm_bridge_alloc() API"  
> >
> > This reverts the commit applied my mistake:
> > https://gitlab.freedesktop.org/drm/misc/kernel/-/commit/91c5c7b5bb2dd09b43b025bce6d790d3c79f4518
> >
> > Neither the  original patch nor the revert has been reviewed/acked.
> >
> > As the commit was a mistake, I'm applying the revert by the end of this
> > week (i.e. on Friday) unless there are better instructions.  
> 
> Really sorry for late. I was made aware of it later through a
> colleague's remark. There is no need to proceed with the revert.
> Acked-by : Inki Dae <inki.dae@...sung.com>

Thanks for the feedback. As agreed with Maxime and approved by you, I'm
leaving the commit as is, without reverting and reapplying. Your
Acked-by is in the records anyway, so somehow reachable in case of need.

Luca

-- 
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

Powered by blists - more mailing lists