Re: [d-kernel] [PATCH 1/1] drm: rockchip: dwhdmiqp-rockchip: attach next bridge to the HDMI bridge

["Ivan A. Melnikov" <iv@altlinux.org>]

On Tue, Jul 14, 2026 at 10:15:02AM +0400, Daniil Gnusarev wrote:
> For embedded systems, additional bridges may be connected after
> the HDMI bridge being created. To ensure full functionality, they
> must be added to the DRM bridge chain.
> 
> Signed-off-by: Daniil Gnusarev <gnusarevda@basealt.ru>
> ---
>  .../gpu/drm/rockchip/dw_hdmi_qp-rockchip.c    | 22 +++++++++++++++++++
>  1 file changed, 22 insertions(+)
> 
> diff --git a/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c b/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c
> index 409f1a1e82a061..9485d85a16f35c 100644
> --- a/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c
> +++ b/drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c
> @@ -430,6 +430,8 @@ static int dw_hdmi_qp_rockchip_bind(struct device *dev, struct device *master,
>  	struct drm_connector *connector;
>  	struct drm_encoder *encoder;
>  	struct rockchip_hdmi_qp *hdmi;
> +	struct drm_bridge *hdmi_bridge;
> +	struct drm_bridge *next_bridge;
>  	struct resource *res;
>  	struct clk_bulk_data *clks;
>  	int ret, irq, i;
> @@ -441,6 +443,11 @@ static int dw_hdmi_qp_rockchip_bind(struct device *dev, struct device *master,
>  	if (!hdmi)
>  		return -ENOMEM;
>  
> +	next_bridge = NULL;
> +	ret = drm_of_find_panel_or_bridge(pdev->dev.of_node, 1, 0, NULL, &next_bridge);
> +	if (ret && ret != -ENODEV)
> +		return ret;

Довольно (недовольно) интересную ситуацию придумал здесь upstream:
чуть выше через devm_kzalloc выделяется память под struct
rockchip_hdmi_qp, и в случае ошиьбки она остаётся выделенной
пока живо устройство.

Вызов drm_of_find_panel_or_bridge делает эту ситуацию ещё
интереснее, поскольку drm_of_find_panel_or_bridge может
вернуть  ︎-EPROBE_DEFER.

> +
>  	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
>  	if (!res)
>  		return -ENODEV;
> @@ -556,6 +563,21 @@ static int dw_hdmi_qp_rockchip_bind(struct device *dev, struct device *master,
>  		return ret;
>  	}
>  
> +	if (next_bridge) {
> +		hdmi_bridge = drm_bridge_chain_get_last_bridge(encoder);
> +		if (hdmi_bridge)
> +			ret = drm_bridge_attach(encoder, next_bridge, hdmi_bridge,
> +						DRM_BRIDGE_ATTACH_NO_CONNECTOR);
> +		else
> +			ret = -ENODEV;

Just a note: этот код никогда не выполнится, у encoder'а хотя бы
один bridge в этом месте точно есть.

> +		if (hdmi_bridge)
> +			drm_bridge_put(hdmi_bridge);

Почему бы не присоединить этот if к предыдущему? Ну или воспользоваться
__free(drm_bridge_put).

> +		if (ret) {
> +			dev_err(hdmi->dev, "failed to attach next bridge: %d\n", ret);
> +			return ret;

Кажется, в этом месте должен быть вызов `drm_encoder_cleanup`.

Как и в следующем early return, но это уже апстримный баг.

> +		}
> +	}
> +
>  	connector = drm_bridge_connector_init(drm, encoder);
>  	if (IS_ERR(connector)) {
>  		ret = PTR_ERR(connector);
> -- 
> 2.42.2

Короче, прям вот неудобно присоединять следующий bridge отсюда.

-- 
  wbr,
    iv m.