Re: [PATCH v8 10/29] drm/msm/dp: add link_ready to manage link-level operations
From: Dmitry Baryshkov
Date: Wed Nov 11 2026 - 09:49:32 EST
On Tue, Nov 10, 2026 at 12:54:31PM +0800, Yongxing Mou wrote:
> In MST mode, multiple streams share the same DP link. Track a link_ready
> state so msm_dp_display_prepare_link() runs only once per link and
> repeated calls are skipped.
>
> Signed-off-by: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>
> Signed-off-by: Yongxing Mou <yongxing.mou@xxxxxxxxxxxxxxxx>
> ---
> drivers/gpu/drm/msm/dp/dp_display.c | 48 +++++++++++++++++++++++++++----------
> drivers/gpu/drm/msm/dp/dp_display.h | 1 +
> 2 files changed, 37 insertions(+), 12 deletions(-)
>
> @@ -1570,6 +1587,11 @@ void msm_dp_display_unprepare(struct msm_dp *msm_dp_display)
>
> dp = container_of(msm_dp_display, struct msm_dp_display_private, msm_dp_display);
>
> + if (!msm_dp_display->link_ready) {
> + drm_dbg_dp(dp->drm_dev, "Link already setup, return\n");
Link already disabled.
With that fixed:
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxxxxxxxx>
--
With best wishes
Dmitry