Hello Liu, On Thu, 22 May 2025 11:20:17 +0800 Liu Ying <victor....@nxp.com> wrote:
> > 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". I agree the fallback seems OK for this MAINTAINERS entry. The other entry (ARM/FREESCALE IMX / MXC ARM ARCHITECTURE) is another story. > >> drm/bridge: imx8qxp-pixel-combiner: convert to > >> devm_drm_bridge_alloc() API > > > > Not acked/reviewed, some discussion happened. I am resending it in v4, > > possibly with updates based on the discussion. > > I still think the main structures in imx8qxp-pixel-combiner.c and imx*-ldb.c > should have the same lifetime with the allocated bridges. I added a new > comment on this driver in v2 just now. Thanks, let's continue the conversation there. Luca -- Luca Ceresoli, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
