Hello Dmitry，
At 2025-07-03 20:49:51, "Andy Yan" <[email protected]> wrote: > >In some application scenarios, we hope to get the corresponding >connector when the bridge's detect hook is invoked. > >For example, we may want to call drm_dp_read_sink_count_cap(which needs >a drm_connector) at the dp deteck hook, intel_dp and nouveau_dp do this >at it's connector's detetc_ctx/detect hook. > >But for a bridge driver, it's detect hook is initiated by the connector, >there is no connector passed down. > >In most cases, we can get the connector by >drm_atomic_get_connector_for_encoder >if the encoder attached to the bridge is enabled, however there will >still be some scenarios where the detect hook of the bridge is called >but the corresponding encoder has not been enabled yet. For instance, >this occurs when the device is hot plug in for the first time. > >Since the call to bridge's detect is initiated by the connector, passing >down the corresponding connector directly will make things simpler. > >Before preparing this patch, we have had some discussions on the details >here[0]. > >PATCH1 adjust the dp/hdmi_audio_* callback parameters order, make it >maintain the same parameter order as get_modes and edid_read. >PATCH2 add connector to detect hook. > >[0]https://patchwork.freedesktop.org/patch/640712/?series=143573&rev=5 Could you please take this series of patches? I hope after the patches are merged, I will be able to update a new version of the DP driver based on this. Moreover, I'm worried that if this wait too long, there might be conflicts with other new bridge drivers. > >Changes in v3: >- Remove redundant SoB > >Changes in v2: >- Make dp/hdmi_audio_* callback keep the same par get_modes > >Andy Yan (2): > drm/bridge: Make dp/hdmi_audio_* callback keep the same paramter order > with get_modes > drm/bridge: Pass down connector to drm bridge detect hook > > drivers/gpu/drm/bridge/adv7511/adv7511.h | 16 +++---- > .../gpu/drm/bridge/adv7511/adv7511_audio.c | 12 +++--- > drivers/gpu/drm/bridge/adv7511/adv7511_cec.c | 4 +- > drivers/gpu/drm/bridge/adv7511/adv7511_drv.c | 3 +- > drivers/gpu/drm/bridge/analogix/anx7625.c | 2 +- > .../drm/bridge/cadence/cdns-mhdp8546-core.c | 3 +- > drivers/gpu/drm/bridge/chrontel-ch7033.c | 2 +- > drivers/gpu/drm/bridge/display-connector.c | 11 +++-- > drivers/gpu/drm/bridge/ite-it6263.c | 3 +- > drivers/gpu/drm/bridge/ite-it6505.c | 2 +- > drivers/gpu/drm/bridge/ite-it66121.c | 3 +- > drivers/gpu/drm/bridge/lontium-lt8912b.c | 6 +-- > drivers/gpu/drm/bridge/lontium-lt9611.c | 15 +++---- > drivers/gpu/drm/bridge/lontium-lt9611uxc.c | 3 +- > .../bridge/megachips-stdpxxxx-ge-b850v3-fw.c | 3 +- > drivers/gpu/drm/bridge/sii902x.c | 3 +- > drivers/gpu/drm/bridge/simple-bridge.c | 2 +- > drivers/gpu/drm/bridge/synopsys/dw-hdmi-qp.c | 14 +++---- > drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 3 +- > drivers/gpu/drm/bridge/tc358767.c | 5 ++- > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 3 +- > drivers/gpu/drm/bridge/ti-tfp410.c | 2 +- > drivers/gpu/drm/bridge/ti-tpd12s015.c | 8 +++- > .../gpu/drm/display/drm_bridge_connector.c | 20 ++++----- > drivers/gpu/drm/drm_bridge.c | 5 ++- > drivers/gpu/drm/mediatek/mtk_dp.c | 3 +- > drivers/gpu/drm/mediatek/mtk_hdmi.c | 3 +- > drivers/gpu/drm/msm/dp/dp_audio.c | 8 ++-- > drivers/gpu/drm/msm/dp/dp_audio.h | 8 ++-- > drivers/gpu/drm/msm/dp/dp_drm.c | 3 +- > drivers/gpu/drm/msm/hdmi/hdmi.h | 10 ++--- > drivers/gpu/drm/msm/hdmi/hdmi_audio.c | 8 ++-- > drivers/gpu/drm/msm/hdmi/hdmi_bridge.c | 2 +- > drivers/gpu/drm/msm/hdmi/hdmi_hpd.c | 4 +- > drivers/gpu/drm/rockchip/rk3066_hdmi.c | 2 +- > drivers/gpu/drm/xlnx/zynqmp_dp.c | 3 +- > include/drm/drm_bridge.h | 42 ++++++++++--------- > 37 files changed, 139 insertions(+), 110 deletions(-) > >-- >2.43.0 > >base-commit: 56e5375b23f342dfa3179395aacc1b47395fddf7 >branch: drm-misc-next
