From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E895D28D85C; Thu, 22 May 2025 14:57:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1747925854; cv=none; b=jfyDGWY+rxSCTQK1Zq3zhyQRiSE4KDMbPG0w/zS8MzefzjFnD3bv+v4yMY5yX9pXiC9RKnA7ggLABASUtf2RP3ZcTCzrVTYsnRv1EKJmU+Wt26w1xYqRO6XbN+L+eVjmSAh1DyPQUxDizDP/xJgzZ05cpxTc2AcLrvyyVpshJX4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1747925854; c=relaxed/simple; bh=5G+CU34zmKz9PE5a5J7jN90cu5Gxk5UtkArIIi8NlfY=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=ecmTH83/MAMfqkm5/6i1hJTldLjeQ/MXwiVOurGkTBteiqsHmv4mV4EgAAdrWPRvmrZmV/B0JOtoi0YnOhrlmIua7zku3FSMj/6qKOg4vPL8+x6c3yljRmwIEeei7sNGXaF08FcbxtW5WIEVtBP/nF8eM9NTJfelsxpkRZcb770= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=s3wma8Dg; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="s3wma8Dg" Received: by smtp.kernel.org (Postfix) with ESMTPSA id D3816C4CEEA; Thu, 22 May 2025 14:57:32 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1747925853; bh=5G+CU34zmKz9PE5a5J7jN90cu5Gxk5UtkArIIi8NlfY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=s3wma8DglmgLQ+hMVFjvtfjc4AGN1CkhvK3aJgPdEeop7ZdSKHWrh+jFnDK9fS1Bt k4DPjLZirixi4RiNSCPaqTeQgbbYCGh34naxvV1r976lLS74g+aZn2NIJWPxU9Bk2h 5lhBFpwnMYqpXLlbfQyJNLBIPkBOwrbgXjydFtJry7hB5vIcXzHiBLTIOBsTw60ofP Sbe3oHlRv+SrSkLUAyO4OvF2SWUXfIQoAWSdBVx6gLBXEtHN69lIJ+ngwI0pTb/j5i 982RPFTGUmNJf2N7qI0n6pT07RfEnvluDICTfTYoG8Smrgr1WlqJE6p9LO4kvWBltm OmAchw1yswU2w== Date: Thu, 22 May 2025 16:57:30 +0200 From: Maxime Ripard To: Luca Ceresoli Cc: Maarten Lankhorst , Thomas Zimmermann , David Airlie , Simona Vetter , Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , Jagan Teki , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Douglas Anderson , Chun-Kuang Hu , Krzysztof Kozlowski , Liu Ying , Anusha Srivatsa , Paul Kocialkowski , Dmitry Baryshkov , Hui Pu , Thomas Petazzoni , 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, Louis Chauvet , Alim Akhtar , Inki Dae , Kyungmin Park , Seung-Woo Kim , Manikandan Muralidharan , Adam Ford , Adrien Grassein , Aleksandr Mishin , Andy Yan , AngeloGioacchino Del Regno , Benson Leung , Biju Das , Christoph Fritz , Cristian Ciocaltea , Detlev Casanova , Dharma Balasubiramani , Guenter Roeck , Heiko Stuebner , Jani Nikula , Janne Grunau , Jerome Brunet , Jesse Van Gavere , Kevin Hilman , Kieran Bingham , Martin Blumenstingl , Matthias Brugger , Philipp Zabel , Phong LE , Sasha Finkelstein , Sugar Zhang , Sui Jingfeng , Tomi Valkeinen , Vitalii Mordan , "Rob Herring (Arm)" , Hsin-Te Yuan , Pin-yen Lin , Xin Ji , Aradhya Bhatia , Tomi Valkeinen , Ian Ray , Martyn Welch , Peter Senna Tschudin , Helge Deller , Kuninori Morimoto , Laurent Pinchart , Alexandre Torgue , Maxime Coquelin , Philippe Cornu , Raphael Gallais-Pou , Yannick Fertre , Alain Volmat , Raphael Gallais-Pou , Michal Simek , Jonathan Corbet , linux-doc@vger.kernel.org Subject: Re: [PATCH v3 00/22] drm: convert all bridges to devm_drm_bridge_alloc() Message-ID: <20250522-amphibian-shiny-chachalaca-cf05ba@houat> References: <20250509-drm-bridge-convert-to-alloc-api-v3-0-b8bc1f16d7aa@bootlin.com> <20250521162216.79dd3290@booty> Precedence: bulk X-Mailing-List: linux-doc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha384; protocol="application/pgp-signature"; boundary="fe7madz4h5tk5u3n" Content-Disposition: inline In-Reply-To: <20250521162216.79dd3290@booty> --fe7madz4h5tk5u3n Content-Type: text/plain; protected-headers=v1; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Subject: Re: [PATCH v3 00/22] drm: convert all bridges to devm_drm_bridge_alloc() MIME-Version: 1.0 On Wed, May 21, 2025 at 04:22:16PM +0200, Luca Ceresoli wrote: > Hello Maxime, Shawn, Liu, all, >=20 > On Fri, 09 May 2025 15:53:26 +0200 > Luca Ceresoli wrote: >=20 > > 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 milesto= ne > > towards removal of bridges from a still existing DRM pipeline without > > use-after-free. >=20 > 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 >=20 > Being my very first commits to drm-misc, I tried to be careful, and > double checked all the patches with Louis (thanks!). >=20 > Here are the pending questions and plan for the remaining patches. >=20 > > Revert "drm/exynos: mic: convert to devm_drm_bridge_alloc() API" >=20 > This reverts the commit applied my mistake: > https://gitlab.freedesktop.org/drm/misc/kernel/-/commit/91c5c7b5bb2dd09b4= 3b025bce6d790d3c79f4518 >=20 > Neither the original patch nor the revert has been reviewed/acked. >=20 > 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. Given the lack of answers, and that it looks correct to me, just leave it there. We can always revert later on if things turned out to be broken. > > drm: convert many bridge drivers from devm_kzalloc() to devm_drm_= bridge_alloc() API >=20 > This patch affects multiple drivers. Running get_maintainers.pl > points at Shawn Guo's repository. After reviewing the MAINTAINERS file, > this looks like due to the 'N:' line in: >=20 > ARM/FREESCALE IMX / MXC ARM ARCHITECTURE > M: Shawn Guo > M: Sascha Hauer > R: Pengutronix Kernel Team > ... > T: git git://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git > N: imx > ... >=20 > (https://gitlab.freedesktop.org/drm/misc/kernel/-/blob/drm-misc-next/MAIN= TAINERS?ref_type=3Dheads#L2511-2528) >=20 > Here 'imx' matches the 'drivers/gpu/drm/bridge/imx/imx-legacy-bridge.c' > file that is touched by the patch. That regexp appears overly generic to = me. I agree, or at least, we shouldn't wait for Shawn or Sasha... > Shawn, can it be fixed by making it less generic? >=20 > If not, can we at least add a band-aid 'X:' entry for > drivers/gpu/drm/bridge/imx? >=20 > I think the other matching entry is the one to consider: >=20 > DRM DRIVERS FOR FREESCALE IMX BRIDGE > M: Liu Ying > L: dri-devel@lists.freedesktop.org > S: Maintained > F: Documentation/devicetree/bindings/display/bridge/fsl,imx8qxp-ldb.yaml > F: Documentation/devicetree/bindings/display/bridge/fsl,imx8qxp-pixel-com= biner.yaml > F: Documentation/devicetree/bindings/display/bridge/fsl,imx8qxp-pixel-lin= k.yaml > F: Documentation/devicetree/bindings/display/bridge/fsl,imx8qxp-pxl2dpi.y= aml > F: drivers/gpu/drm/bridge/imx/ >=20 > (https://gitlab.freedesktop.org/drm/misc/kernel/-/blob/drm-misc-next/MAIN= TAINERS?ref_type=3Dheads#L7940-7948) =2E.. As long as Ying is fine with it, because it does look like they are the actual maintainer. > However it does not list any trees. I _guess_ drm-misc applies here as > a fallback as well as common sense. >=20 > Liu, should this entry have a 'T:' line for drm/misc? >=20 > > drm/bridge: imx8qxp-pixel-combiner: convert to devm_drm_bridge_al= loc() API >=20 > Not acked/reviewed, some discussion happened. I am resending it in v4, > possibly with updates based on the discussion. >=20 > But it has the same issue discussed above, with get_maintiners.pl > pointing at Shawn Guo's tree, so in the future I'm assuming this goes > to drm-misc unless there are news about that. >=20 > > drm/bridge: tc358767: convert to devm_drm_bridge_alloc() API >=20 > No feedback, resending in v4. >=20 > > drm/todo: add entry to remove devm_drm_put_bridge() >=20 > This involves documentation maintained on another tree. Where should it > be applied? There are two matching entries in MAINTAINERS: >=20 > * DRM DRIVERS -> the drm tree > * DRM DRIVERS AND MISC GPU PATCHES -> the drm-misc tree >=20 > To me it looks like the second is obviously the closest match as we are > dealing with DRM bridges, so I'm applying this as well on Friday unless > there are better instructions. Yes, they should be applied to drm-misc. That being said, putting a two days timeout on *any* email is really over-the-top. I doubt you reply to any of your mail in such a short timeframe. We have rules for a reason, I'd expect you to follow them, no matter how frustrating the lack of answers can be. Maxime --fe7madz4h5tk5u3n Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iJUEABMJAB0WIQTkHFbLp4ejekA/qfgnX84Zoj2+dgUCaC87WgAKCRAnX84Zoj2+ do0iAYDmirWaXC3jRPAWe+k6egrSRuRKSb9T+TBMm48WnW6SNhJrwyOYhX8DpsOL IHaDlR4BfiFZ6a345YL/OKfebcrONxA8UY50Y3/kmzH9OdWsvkfvJSWcHiCEeu26 nklxCgQuzA== =mXOq -----END PGP SIGNATURE----- --fe7madz4h5tk5u3n--