On Thu, May 22, 2025 at 11:20:17AM +0800, Liu Ying wrote: > >> drm: convert many bridge drivers from devm_kzalloc() to > >> devm_drm_bridge_alloc() API > > > > 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: > > > > ARM/FREESCALE IMX / MXC ARM ARCHITECTURE > > M: Shawn Guo <shawn...@kernel.org> > > M: Sascha Hauer <s.ha...@pengutronix.de> > > R: Pengutronix Kernel Team <ker...@pengutronix.de> > > ... > > T: git git://git.kernel.org/pub/scm/linux/kernel/git/shawnguo/linux.git > > N: imx > > ... > > > > (https://gitlab.freedesktop.org/drm/misc/kernel/-/blob/drm-misc-next/MAINTAINERS?ref_type=heads#L2511-2528) > > > > 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. > > > > Shawn, can it be fixed by making it less generic? > > > > If not, can we at least add a band-aid 'X:' entry for > > drivers/gpu/drm/bridge/imx? > > > > I think the other matching entry is the one to consider: > > > > DRM DRIVERS FOR FREESCALE IMX BRIDGE > > M: Liu Ying <victor....@nxp.com> > > 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-combiner.yaml > > F: > > Documentation/devicetree/bindings/display/bridge/fsl,imx8qxp-pixel-link.yaml > > F: > > Documentation/devicetree/bindings/display/bridge/fsl,imx8qxp-pxl2dpi.yaml > > F: drivers/gpu/drm/bridge/imx/ > > > > (https://gitlab.freedesktop.org/drm/misc/kernel/-/blob/drm-misc-next/MAINTAINERS?ref_type=heads#L7940-7948) > > > > However it does not list any trees. I _guess_ drm-misc applies here as > > a fallback as well as common sense. > > > > Liu, should this entry have a 'T:' line for drm/misc? > > These bridge drivers also don't have a 'T:' line: > > DRM DRIVER FOR CHIPONE ICN6211 MIPI-DSI to RGB CONVERTER BRIDGE > DRM DRIVER FOR PARADE PS8640 BRIDGE CHIP > DRM DRIVER FOR TI DLPC3433 MIPI DSI TO DMD BRIDGE > DRM DRIVER FOR TI SN65DSI86 BRIDGE CHIP > LONTIUM LT8912B MIPI TO HDMI BRIDGE > MEGACHIPS STDPXXXX-GE-B850V3-FW LVDS/DP++ BRIDGES > MICROCHIP SAM9x7-COMPATIBLE LVDS CONTROLLER > > I think that they fallback to drm-misc since "DRM DRIVERS FOR BRIDGE CHIPS" > covers them. I don't have strong opinion on adding a "T" line to them, at > least to "DRM DRIVERS FOR FREESCALE IMX BRIDGE". Anyway, it would be good > to know comments from maintainers for "DRM DRIVERS FOR BRIDGE CHIPS" and > "DRM DRIVERS".
That's good enough to me. drivers/gpu/drm/bridge is indeed under the maintenance of drm-misc and there's no exception afaik. get_maintainers.pl also properly reports it, so I'm not sure we need to do anything there. Maxime
signature.asc
Description: PGP signature
