* Re: [d-kernel] [PATCH 1/1] drm: rockchip: dwhdmiqp-rockchip: attach next bridge to the HDMI bridge
2026-07-14 6:15 ` [d-kernel] [PATCH 1/1] drm: rockchip: dwhdmiqp-rockchip: attach next bridge to the HDMI bridge Daniil Gnusarev
@ 2026-07-15 8:45 ` Vasiliy Kovalev
0 siblings, 0 replies; 3+ messages in thread
From: Vasiliy Kovalev @ 2026-07-15 8:45 UTC (permalink / raw)
To: Daniil Gnusarev; +Cc: ALT Linux kernel packages development
Добрый день,
On 7/14/26 09:15, 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;
> +
> 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;
> + if (hdmi_bridge)
> + drm_bridge_put(hdmi_bridge);
> + if (ret) {
> + dev_err(hdmi->dev, "failed to attach next bridge: %d\n", ret);
> + return ret;
> + }
> + }
> +
> connector = drm_bridge_connector_init(drm, encoder);
> if (IS_ERR(connector)) {
> ret = PTR_ERR(connector);
Патч трогает только rockchip-часть (dw_hdmi_qp-rockchip.c), но по сути
это обход того, что должна делать библиотека моста dw-hdmi-qp.c. Оба
файла активно сопровождаются - по git log правки свежие:
$ git log --format="%ad" --
./drivers/gpu/drm/rockchip/dw_hdmi_qp-rockchip.c | head -5
Sat Jul 4 11:12:02 2026 +0200
Tue Jun 9 14:44:04 2026 +0200
Tue Jun 9 14:44:03 2026 +0200
Thu Apr 23 11:17:24 2026 +0200
Tue Mar 10 00:44:34 2026 +0200
$ git log --format="%ad" --
./drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c | head -5
Fri Jun 19 14:24:39 2026 +0200
Tue Jun 9 14:44:02 2026 +0200
Wed May 20 16:43:39 2026 +0200
Mon Apr 27 09:02:57 2026 +0200
Thu Feb 5 10:33:06 2026 +0100
Так что имеет смысл предложить это напрямую в dri-devel + linux-rockchip,
а не заводить только в ALT.
По самому патчу, в mainline dw_hdmi_qp_bind() подключает только свой
мост и на этом останавливается:
(https://elixir.bootlin.com/linux/v7.2-rc3/source/drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c#L1369)
ret = drm_bridge_attach(encoder, &hdmi->bridge, NULL,
DRM_BRIDGE_ATTACH_NO_CONNECTOR);
Колбэка .attach в dw_hdmi_qp_bridge_funcs нет. При этом в обычном
dw-hdmi он есть и делает подключение следующего моста сам:
(https://elixir.bootlin.com/linux/v7.2-rc3/source/drivers/gpu/drm/bridge/synopsys/dw-hdmi.c#L2907)
static int dw_hdmi_bridge_attach(struct drm_bridge *bridge,
struct drm_encoder *encoder,
enum drm_bridge_attach_flags flags)
{
struct dw_hdmi *hdmi = bridge->driver_private;
/* DRM_BRIDGE_ATTACH_NO_CONNECTOR requires a remote-endpoint to
the next bridge */
if (WARN_ON((flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) &&
!hdmi->plat_data->output_port))
return -EINVAL;
if (flags & DRM_BRIDGE_ATTACH_NO_CONNECTOR) {
struct device_node *remote __free(device_node) =
of_graph_get_remote_node(hdmi->dev->of_node,
hdmi->plat_data->output_port, -1);
if (!remote)
return -ENODEV;
struct drm_bridge *next_bridge __free(drm_bridge_put) =
of_drm_find_and_get_bridge(remote);
if (!next_bridge)
return -EPROBE_DEFER;
return drm_bridge_attach(encoder, next_bridge, bridge,
flags);
}
return dw_hdmi_connector_create(hdmi);
}
Возможно логику лучше держать в самой библиотеке моста (.attach +
output_port в plat_data), а не в rockchip-части через
drm_bridge_chain_get_last_bridge() -
тогда там останется одна строка и это же решение должно покрыть и
другие чипы. Плюс drm_of_find_panel_or_bridge(), который вызывается
в патче, сам помечен как deprecated в комментарии
https://elixir.bootlin.com/linux/v7.2-rc3/source/drivers/gpu/drm/drm_of.c#L234:
* This function is deprecated and should not be used in new drivers. Use
* of_drm_get_bridge_by_endpoint() instead when not looking for a panel, or
* devm_drm_of_get_bridge() otherwise.
Для dw-hdmi такой подход в своё время принимали:
https://lore.kernel.org/all/20200526011505.31884-24-laurent.pinchart+renesas@ideasonboard.com/
но как правильнее, лучше спросить у мейнтейнеров.
В отличие от Байкал, патчи к которому апстрим не принимает в принципе
(https://lore.kernel.org/all/20260227072726.1142944-2-andriy.shevchenko@linux.intel.com/)
в этой ситуации форк не обязателен - всё делается через апстрим.
После принятия в mainline можно попробовать бэкпортировать в ALT 6.18
(в stable апстрима такое не возьмут - это не фикс безопасности), хотя
маловероятно: зависит от расхождения кода и от того, кто в ALT это потащит.
--
Vasiliy
^ permalink raw reply [flat|nested] 3+ messages in thread