On Mon, Jun 09, 2025 at 08:21:50PM +0800, Yongxing Mou wrote:
> Introduce APIs to update the MST state change to MST framework when
> device is plugged/unplugged.
>
> Signed-off-by: Yongxing Mou <quic_yong...@quicinc.com>
> Signed-off-by: Abhinav Kumar <quic_abhin...@quicinc.com>
Hmm, who is the author of the patch?
> ---
> drivers/gpu/drm/msm/dp/dp_display.c | 20 ++++++++++++++++++++
> drivers/gpu/drm/msm/dp/dp_mst_drm.c | 15 +++++++++++++++
> drivers/gpu/drm/msm/dp/dp_mst_drm.h | 1 +
> 3 files changed, 36 insertions(+)
>
> diff --git a/drivers/gpu/drm/msm/dp/dp_display.c
> b/drivers/gpu/drm/msm/dp/dp_display.c
> index
> 323d1c0a9efa4fa30ce97317e873607c54409a11..9dbcf4553cad70c9e3722160a87403fc815765d7
> 100644
> --- a/drivers/gpu/drm/msm/dp/dp_display.c
> +++ b/drivers/gpu/drm/msm/dp/dp_display.c
> @@ -29,6 +29,7 @@
> #include "dp_drm.h"
> #include "dp_audio.h"
> #include "dp_debug.h"
> +#include "dp_mst_drm.h"
>
> static bool psr_enabled = false;
> module_param(psr_enabled, bool, 0);
> @@ -420,6 +421,17 @@ static void msm_dp_display_mst_init(struct
> msm_dp_display_private *dp)
> msm_dp->mst_active = true;
> }
>
> +static void msm_dp_display_set_mst_mgr_state(struct msm_dp_display_private
> *dp,
> + bool state)
> +{
> + if (!dp->msm_dp_display.mst_active)
> + return;
> +
> + msm_dp_mst_display_set_mgr_state(&dp->msm_dp_display, state);
> +
> + drm_dbg_dp(dp->drm_dev, "mst_mgr_state: %d\n", state);
Yet-another-oneliner? Noooo. Really, no.
> +}
> +
> static int msm_dp_display_process_hpd_high(struct msm_dp_display_private *dp)
> {
> struct drm_connector *connector = dp->msm_dp_display.connector;
--
With best wishes
Dmitry
