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

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

On Wed, Jul 15, 2026 at 11:45:49AM +0300, Vasiliy Kovalev wrote:
> Добрый день,
> 
> 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>
> Возможно логику лучше держать в самой библиотеке моста (.attach +
> output_port в plat_data), а не в rockchip-части через
> drm_bridge_chain_get_last_bridge() -

Согласен, тем более что концептуально присоединять следующий bridge
в .attach правильнее:

https://github.com/torvalds/linux/blob/master/drivers/gpu/drm/drm_bridge.c#L74

И именно так скорее всего будет действовать апстрим.

> тогда там останется одна строка и это же решение должно покрыть и
> другие чипы. Плюс 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.

of_drm_get_bridge_by_endpoint в 6.18 ещё нет.

devm_drm_of_get_bridge заворачивает панель в бридж, если следующим
узлом оказвывается панель, а не бридж. Нужно ли это здесь?

> Для 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/)
> в этой ситуации форк не обязателен - всё делается через апстрим.
[...]

К сожалению, на вопрос мейнейнеров "а на каком оборудовании
вы заметили эту проблему и проверили это решение" мы не сможем
дать приемлемого ответа.

-- 
  wbr,
    iv m.