Re: [PATCH v11 2/3] drm/bridge: imx: Add i.MX93 parallel display format configuration support

From: Liu Ying

Date: Mon Mar 09 2026 - 22:56:42 EST


Hi Marco, Luca,

On Tue, Mar 03, 2026 at 11:34:27AM +0100, Marco Felsch wrote:

[...]

> + next_bridge = devm_drm_of_get_bridge(dev, dev->of_node, 1, 0);
> + if (IS_ERR(next_bridge))
> + return dev_err_probe(dev, PTR_ERR(next_bridge),
> + "failed to get next bridge\n");
> + pdfc->dev = dev;
> + pdfc->bridge.of_node = dev->of_node;
> + pdfc->bridge.type = DRM_MODE_CONNECTOR_DPI;
> + pdfc->bridge.next_bridge = next_bridge;

When I was reviewing another patch[1], I was aware of the necessity of
calling drm_bridge_get() for next_bridge to balance the next bridge's
refcount put from __drm_bridge_free() for this bridge. I'd be good if
Luca may confirm this is correct. Sorry for bringing this up late.

[1] https://lore.kernel.org/all/5edd40f3-f83e-4a1b-99d4-1e595fc0e4cb@xxxxxxx/

--
Regards,
Liu Ying