(subset) [PATCH v2 00/34] drm: convert all bridges to devm_drm_bridge_alloc()

[Louis Chauvet]

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