All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yongxing Mou <yongxing.mou@oss.qualcomm.com>
To: Rob Clark <robin.clark@oss.qualcomm.com>,
	Dmitry Baryshkov <lumag@kernel.org>,
	Abhinav Kumar <abhinav.kumar@linux.dev>,
	Jessica Zhang <jesszhan0024@gmail.com>,
	Sean Paul <sean@poorly.run>,
	Marijn Suijten <marijn.suijten@somainline.org>,
	David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>
Cc: linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
	freedreno@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>,
	Abhinav Kumar <quic_abhinavk@quicinc.com>
Subject: Re: [PATCH v7 00/15] drm/msm/dp: Prerequisite cleanup for upcoming MST support
Date: Wed, 24 Jun 2026 11:16:39 +0800	[thread overview]
Message-ID: <b05487fa-e282-4236-8f10-016e7ee4dd87@oss.qualcomm.com> (raw)
In-Reply-To: <20260609-dp_mstclean-v7-0-ea04113e8233@oss.qualcomm.com>



On 6/9/2026 5:46 PM, Yongxing Mou wrote:
> This series is the SST-only prerequisite portion of the MSM DP MST
> work. It refactors the existing DP code paths so that MST can
> plug in cleanly in a follow-up series, without bundling the cleanup
> with MST functionality in the same submission.
> 
> SST behaviour is preserved end-to-end; no new functionality is added
> here. The intent is to land these refactors first to keep the
> follow-up MST series focused, smaller, and easier to review.
> 
> Signed-off-by: Yongxing Mou <yongxing.mou@oss.qualcomm.com>
> ---
> Changes in v7:
> - patch 7: Use WARN_ON_ONCE() when pixel clock is already on. [Konrad]
> - Link to v6: https://lore.kernel.org/r/20260602-dp_mstclean-v6-0-2c17ff40a9b2@oss.qualcomm.com
> 
> Changes in v6:
> - patch 4: fixed the RMW comments. [Dmitry]
> - patch 10: do not rename the existing struct. [Dmitry]
> - Link to v5: https://lore.kernel.org/r/20260528-dp_mstclean-v5-0-a9221c1f1f3b@oss.qualcomm.com
> 
> Changes in v5:
> - Split out from v4: only the SST prerequisite cleanup (patches 1-15)
>    is sent here; the MST implementation will follow on top.
> - Rebased onto linux-next-20260527; adapted bridge callbacks to the
>    new drm_atomic_commit* API.
> - patch 3: added Suggested-by: Dmitry Baryshkov.
> - patch 4: fixed "splite" typo, reworded body, added RMW comment
>    covering both SST and MST paths. [Dmitry]
> - patch 10: drop cached panel from msm_dp_ctrl_private; pass panel
>    explicitly to all stream-related dp_ctrl APIs. [Dmitry]
> - patch 13/14: introduce bridge wrappers and atomic_prepare with
>    drm_atomic_commit* from the start to preserve bisectability.
> - patch 15: fixed pass panel inside the func. [Dmitry]
> - Link to v4: https://lore.kernel.org/all/20260410-msm-dp-mst-v4-0-b20518dea8de@oss.qualcomm.com/
> 
> ---
> Abhinav Kumar (6):
>        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: 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
> 
> Yongxing Mou (9):
>        drm/msm/dp: remove cached drm_edid from panel
>        drm/msm/dp: drop deprecated .mode_set() and use .atomic_enable
>        drm/msm/dp: move mode setup into msm_dp_panel_init_panel_info()
>        drm/msm/dp: split msm_dp_ctrl_config_ctrl() into link parts and stream parts
>        drm/msm/dp: extract MISC1_MISC0 configuration into a separate function
>        drm/msm/dp: split link setup from source params
>        drm/msm/dp: move the pixel clock control to its own API
>        drm/msm/dp: simplify link and clock disable sequence
>        drm/msm/dp: pass panel to display enable/disable helpers
> 
>   drivers/gpu/drm/msm/dp/dp_ctrl.c    | 382 ++++++++++++++++++++----------------
>   drivers/gpu/drm/msm/dp/dp_ctrl.h    |  30 ++-
>   drivers/gpu/drm/msm/dp/dp_display.c | 273 +++++++++++++-------------
>   drivers/gpu/drm/msm/dp/dp_display.h |   8 +
>   drivers/gpu/drm/msm/dp/dp_drm.c     |  43 +++-
>   drivers/gpu/drm/msm/dp/dp_drm.h     |  12 --
>   drivers/gpu/drm/msm/dp/dp_panel.c   |  75 ++-----
>   drivers/gpu/drm/msm/dp/dp_panel.h   |  17 +-
>   8 files changed, 451 insertions(+), 389 deletions(-)
> ---
> base-commit: e7d700e14934e68f86338c5610cf2ae76798b663
> change-id: 20260528-dp_mstclean-f094cea8ca24
> 
> Best regards,
Hi Dmitry, will this series be merged next cycle?

      parent reply	other threads:[~2026-06-24  3:16 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-09  9:46 [PATCH v7 00/15] drm/msm/dp: Prerequisite cleanup for upcoming MST support Yongxing Mou
2026-06-09  9:46 ` [PATCH v7 01/15] drm/msm/dp: remove cached drm_edid from panel Yongxing Mou
2026-06-09 10:04   ` sashiko-bot
2026-06-09  9:46 ` [PATCH v7 02/15] drm/msm/dp: drop deprecated .mode_set() and use .atomic_enable Yongxing Mou
2026-06-09 10:06   ` sashiko-bot
2026-06-09  9:46 ` [PATCH v7 03/15] drm/msm/dp: move mode setup into msm_dp_panel_init_panel_info() Yongxing Mou
2026-06-09  9:46 ` [PATCH v7 04/15] drm/msm/dp: split msm_dp_ctrl_config_ctrl() into link parts and stream parts Yongxing Mou
2026-06-09 10:04   ` sashiko-bot
2026-06-09  9:46 ` [PATCH v7 05/15] drm/msm/dp: extract MISC1_MISC0 configuration into a separate function Yongxing Mou
2026-06-09  9:46 ` [PATCH v7 06/15] drm/msm/dp: split link setup from source params Yongxing Mou
2026-06-09 10:05   ` sashiko-bot
2026-06-09  9:46 ` [PATCH v7 07/15] drm/msm/dp: move the pixel clock control to its own API Yongxing Mou
2026-06-09 10:04   ` sashiko-bot
2026-06-09  9:46 ` [PATCH v7 08/15] drm/msm/dp: break up dp_display_enable into two parts Yongxing Mou
2026-06-09  9:59   ` sashiko-bot
2026-06-09  9:46 ` [PATCH v7 09/15] drm/msm/dp: re-arrange dp_display_disable() into functional parts Yongxing Mou
2026-06-09 10:17   ` sashiko-bot
2026-06-09  9:46 ` [PATCH v7 10/15] drm/msm/dp: allow dp_ctrl stream APIs to use any panel passed to it Yongxing Mou
2026-06-09  9:46 ` [PATCH v7 11/15] drm/msm/dp: split dp_ctrl_off() into stream and link parts Yongxing Mou
2026-06-09 10:04   ` sashiko-bot
2026-06-09  9:46 ` [PATCH v7 12/15] drm/msm/dp: simplify link and clock disable sequence Yongxing Mou
2026-06-09 10:02   ` sashiko-bot
2026-06-09  9:46 ` [PATCH v7 13/15] drm/msm/dp: make bridge helpers use dp_display to allow re-use Yongxing Mou
2026-06-09 10:09   ` sashiko-bot
2026-06-09  9:46 ` [PATCH v7 14/15] drm/msm/dp: separate dp_display_prepare() into its own API Yongxing Mou
2026-06-09 10:10   ` sashiko-bot
2026-06-09  9:46 ` [PATCH v7 15/15] drm/msm/dp: pass panel to display enable/disable helpers Yongxing Mou
2026-06-09 10:13   ` sashiko-bot
2026-06-24  3:16 ` Yongxing Mou [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b05487fa-e282-4236-8f10-016e7ee4dd87@oss.qualcomm.com \
    --to=yongxing.mou@oss.qualcomm.com \
    --cc=abhinav.kumar@linux.dev \
    --cc=airlied@gmail.com \
    --cc=dmitry.baryshkov@oss.qualcomm.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=freedreno@lists.freedesktop.org \
    --cc=jesszhan0024@gmail.com \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lumag@kernel.org \
    --cc=marijn.suijten@somainline.org \
    --cc=quic_abhinavk@quicinc.com \
    --cc=robin.clark@oss.qualcomm.com \
    --cc=sean@poorly.run \
    --cc=simona@ffwll.ch \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.