[PATCH 00/45] drm/msm/dp: Add MST support for MSM chipsets

From: Abhinav Kumar
Date: Thu Dec 05 2024 - 23:35:37 EST


Add support for Multi-stream transport for MSM chipsets that allow
a single instance of DP controller to send multiple streams.

This series has been validated on sa8775p ride platform using multiple
MST dongles and also daisy chain method on both DP0 and DP1 upto 1080P.

With 4x4K monitors, due to lack of layer mixers that combination will not
work but this can be supported as well after some rework on the DPU side.

In addition, SST was re-validated with all these changes to ensure there
were no regressions.

This patch series was made on top of:

[1] : https://patchwork.freedesktop.org/patch/622243/ (to avoid a log spam)
[2] : https://patchwork.freedesktop.org/series/142010/ (to fix up HPD)
[3] : https://patchwork.freedesktop.org/patch/612740/ (to avoid blank screens)
[4] : https://patchwork.freedesktop.org/series/140216/ (MDSS DT for sa8775p)
[5] : https://patchwork.kernel.org/project/linux-arm-msm/list/?series=912200
(Display Port DT changes for sa8775p)

Bindings for the pixel clock for additional stream is available at :

[6] : https://patchwork.freedesktop.org/series/142016/

Overall, the patch series has been organized in the following way:

1) First set are a couple of fixes made while debugging MST but applicable
to SST as well so go ahead of everything else
2) Prepare the DP driver to get ready to handle multiple streams. This is the bulk
of the work as current DP driver design had to be adjusted to make this happen.
3) Finally, new files to handle MST related operations

Validation was done on the latest linux-next on top of above changes and
both FB console and weston compositors were validated with these changes.

To: Rob Clark <robdclark@xxxxxxxxx>
To: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
To: Sean Paul <sean@xxxxxxxxxx>
To: Marijn Suijten <marijn.suijten@xxxxxxxxxxxxxx>
To: David Airlie <airlied@xxxxxxxxx>
To: Simona Vetter <simona@xxxxxxxx>
To: Stephen Boyd <swboyd@xxxxxxxxxxxx>
To: Chandan Uddaraju <chandanu@xxxxxxxxxxxxxx>
To: Guenter Roeck <groeck@xxxxxxxxxxxx>
To: Kuogee Hsieh <quic_khsieh@xxxxxxxxxxx>
To: Bjorn Andersson <andersson@xxxxxxxxxx>
To: Konrad Dybcio <konradybcio@xxxxxxxxxx>
To: Rob Herring <robh@xxxxxxxxxx>
To: Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>
To: Conor Dooley <conor+dt@xxxxxxxxxx>
Cc: Vara Reddy <quic_varar@xxxxxxxxxxx>
Cc: Rob Clark <robdclark@xxxxxxxxxxxx>
Cc: Tanmay Shah <tanmay@xxxxxxxxxxxxxx>
Cc: linux-arm-msm@xxxxxxxxxxxxxxx
Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx
Cc: freedreno@xxxxxxxxxxxxxxxxxxxxx
Cc: linux-kernel@xxxxxxxxxxxxxxx
Cc: devicetree@xxxxxxxxxxxxxxx
Cc: Jessica Zhang <quic_jesszhan@xxxxxxxxxxx>
Cc: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>

Signed-off-by: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>
---
Abhinav Kumar (43):
drm/msm/dp: dont call dp_catalog_ctrl_mainlink_ctrl in dp_ctrl_configure_source_params()
drm/msm/dp: disable the opp table request even for dp_ctrl_off_link()
drm/msm/dp: fix the intf_type of MST interfaces
drm/msm/dp: split msm_dp_panel_read_sink_caps() into two parts
drm/msm/dp: add a helper to read mst caps for dp_panel
drm/msm/dp: remove dp_display's dp_mode and use dp_panel's instead
drm/msm/dp: break up dp_display_enable into two parts
drm/msm/dp: re-arrange dp_display_disable() into functional parts
drm/msm/dp: allow dp_ctrl stream APIs to use any panel passed to it
drm/msm/dp: move the pixel clock control to its own API
drm/msm/dp: split dp_ctrl_off() into stream and link parts
drm/msm/dp: make bridge helpers use dp_display to allow re-use
drm/msm/dp: separate dp_display_prepare() into its own API
drm/msm/dp: introduce stream_id for each DP panel
drm/msm/dp: convert dp_display_set_mode() to use dp_panel argument
drm/msm/dp: add support for programming p1 register block
drm/msm/dp: use stream_id to change offsets in dp_catalog
drm/msm/dp: add support to send ACT packets for MST
drm/msm/dp: add support to program mst support in mainlink
drm/msm/dp: no need to update tu calculation for mst
drm/msm/dp: add support for mst channel slot allocation
drm/msm/dp: add support to send vcpf packets in dp controller
drm/msm/dp: always program MST_FIFO_CONSTANT_FILL for MST
drm/msm/dp: abstract out the dp_display stream helpers to accept a panel
drm/msm/dp: move link related operations to dp_display_unprepare()
drm/msm/dp: replace power_on with active_stream_cnt for dp_display
drm/msm/dp: make the SST bridge disconnected when mst is active
drm/msm/dp: add an API to initialize MST on sink side
drm/msm/dp: skip reading the EDID for MST cases
drm/msm/dp: add dp_display_get_panel() to initialize DP panel
drm/msm/dp: add dp_mst_drm to manage DP MST bridge operations
drm/msm/dp: add connector abstraction for DP MST
drm/msm/dp: add irq hpd callback for dp mst
drm/msm/dp: add support to re-use and clear the panel edid
drm/msm/dp: add a mst session mutex to protect bridge ops
drm/msm: add support for non-blocking commits
drm/msm: initialize DRM MST encoders for DP controllers
drm/msm/dp: initialize dp_mst module for each DP MST controller
drm/msm: add a stream to intf map for DP controller
drm/msm/dpu: use msm_dp_get_mst_intf_id() to get the intf id
drm/msm/dp: mark ST_DISCONNECTED only if all streams are disabled
drm/msm/dp: populate the max_streams for sa8775 mst controller
arm64: dts: qcom: add mst support for pixel 1 stream clk for DP1

Yongxing Mou (2):
drm/msm/dp: propagate hpd state changes to dp mst module
arm64: dts: qcom: add mst support for pixel stream clk for DP0

arch/arm64/boot/dts/qcom/sa8775p.dtsi | 23 +-
drivers/gpu/drm/msm/Makefile | 3 +-
.../drm/msm/disp/dpu1/catalog/dpu_8_4_sa8775p.h | 8 +-
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 25 +-
drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h | 2 +
drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 29 +-
drivers/gpu/drm/msm/dp/dp_audio.c | 2 +-
drivers/gpu/drm/msm/dp/dp_aux.h | 1 +
drivers/gpu/drm/msm/dp/dp_catalog.c | 292 ++++-
drivers/gpu/drm/msm/dp/dp_catalog.h | 27 +
drivers/gpu/drm/msm/dp/dp_ctrl.c | 454 ++++++--
drivers/gpu/drm/msm/dp/dp_ctrl.h | 21 +-
drivers/gpu/drm/msm/dp/dp_display.c | 559 +++++++---
drivers/gpu/drm/msm/dp/dp_display.h | 33 +-
drivers/gpu/drm/msm/dp/dp_drm.c | 53 +-
drivers/gpu/drm/msm/dp/dp_drm.h | 12 -
drivers/gpu/drm/msm/dp/dp_mst_drm.c | 1118 ++++++++++++++++++++
drivers/gpu/drm/msm/dp/dp_mst_drm.h | 110 ++
drivers/gpu/drm/msm/dp/dp_panel.c | 41 +-
drivers/gpu/drm/msm/dp/dp_panel.h | 15 +-
drivers/gpu/drm/msm/dp/dp_reg.h | 25 +-
drivers/gpu/drm/msm/msm_atomic.c | 2 +
drivers/gpu/drm/msm/msm_drv.h | 29 +
drivers/gpu/drm/msm/msm_kms.c | 1 +
24 files changed, 2589 insertions(+), 296 deletions(-)
---
base-commit: b166256c1e6ce356fa1404d4c8531830e6f100a8
change-id: 20241205-dp_mst-d62fa86257ed

Best regards,
--
Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>