Message-ID: <207e2908-62ce-470f-9077-c5709d3e1a6a@bootlin.com>
Date: Wed, 30 Apr 2025 17:30:05 +0200
From: Louis Chauvet <louis.chauvet@...tlin.com>
To: Maxime Ripard <mripard@...nel.org>
Cc: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>,
 Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
 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>,
 Luca Ceresoli <luca.ceresoli@...tlin.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,
 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>,
 Liu Ying <victor.liu@....com>,
 Manikandan Muralidharan <manikandan.m@...rochip.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>,
 Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>,
 Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
 Hans de Goede <hdegoede@...hat.com>,
 Uwe Kleine-König <u.kleine-koenig@...libre.com>,
 "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@...com>,
 Martyn Welch <martyn.welch@...labora.co.uk>,
 Peter Senna Tschudin <peter.senna@...il.com>,
 Russell King <linux@...linux.org.uk>, Herve Codina
 <herve.codina@...tlin.com>, Alim Akhtar <alim.akhtar@...sung.com>,
 Inki Dae <inki.dae@...sung.com>, Kyungmin Park <kyungmin.park@...sung.com>,
 Seung-Woo Kim <sw0312.kim@...sung.com>,
 Linus Walleij <linus.walleij@...aro.org>,
 Abhinav Kumar <quic_abhinavk@...cinc.com>,
 Bjorn Andersson <quic_bjorande@...cinc.com>,
 Marijn Suijten <marijn.suijten@...ainline.org>,
 Rob Clark <robdclark@...il.com>, Sean Paul <sean@...rly.run>,
 Helge Deller <deller@....de>,
 Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>,
 Laurent Pinchart <laurent.pinchart+renesas@...asonboard.com>
Subject: Re: (subset) [PATCH v2 00/34] drm: convert all bridges to
 devm_drm_bridge_alloc()



Le 30/04/2025 à 12:39, Maxime Ripard a écrit :
> On Wed, Apr 30, 2025 at 10:21:48AM +0200, Louis Chauvet wrote:
>>
>>
>> Le 29/04/2025 à 16:42, Dmitry Baryshkov a écrit :
>>> On Tue, Apr 29, 2025 at 11:27:51AM +0200, Louis Chauvet wrote:
>>>>
>>>> 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!
>>>>
>>>
>>> [...]
>>>
>>>> [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
>>>
>>> Why? These drivers are explicitly handled outside of drm-misc. Please be
>>> more careful next time.
>>>
>>
>> Sorry, I was not aware that msm also have his own repository.
>>
>> TBH, I was not aware that other repositories existed for drm (I should have
>> looked at MAINTAINERS, it is totally my fault). DIM and doc[1] only list
>> drm, drm-misc, drm-xe, drm-intel, so I just tough "intel is special", not
>> "drm is divided in many repositories".
> 
> It's mentioned in the drm-misc section:
> 
> https://drm.pages.freedesktop.org/maintainer-tools/repositories/drm-misc.html
> 
>> This repository consists mostly of the core drm code as well as DRM
>> drivers that do not have a dedicated repository.
 >
> Feel free to send a patch to improve the doc. If you missed it, someone
> else will.

Done : 
https://gitlab.freedesktop.org/drm/maintainer-tools/-/merge_requests/77#c304368de02d740ca751f5812ddcd0cfac40d162

I added more details for the push process, clarified the existing drm 
repositories and added a small check in dim to warn user if the pushed 
commits does not belong to the targeted brach.

> Maxime

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

Powered by blists - more mailing lists