mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Yongxing Mou <yongxing.mou@oss.qualcomm.com>
To: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
Cc: Rob Clark <robin.clark@oss.qualcomm.com>,
	Dmitry Baryshkov <lumag@kernel.org>,
	Abhinav Kumar <abhinav.kumar@linux.dev>,
	Sean Paul <sean@poorly.run>,
	Marijn Suijten <marijn.suijten@somainline.org>,
	David Airlie <airlied@gmail.com>, Simona Vetter <simona@ffwll.ch>,
	Jessica Zhang <jesszhan0024@gmail.com>,
	linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
	freedreno@lists.freedesktop.org, linux-kernel@vger.kernel.org,
	Abhinav Kumar <quic_abhinavk@quicinc.com>
Subject: Re: [PATCH RESEND v5 08/25] drm/msm/dp: Add support for MST channel slot allocation
Date: Wed, 26 Aug 2026 15:48:34 +0800	[thread overview]
Message-ID: <3edc542f-3991-48cf-89cf-01c7fdecfaee@oss.qualcomm.com> (raw)
In-Reply-To: <3mj5fuv5gdoxjqquyllzk2tzbvyblh77yj2dttki3t64krljfm@cwpua5mtfz2o>



On 7/13/2026 2:57 AM, Dmitry Baryshkov wrote:
> On Mon, Jun 29, 2026 at 10:14:29PM +0800, Yongxing Mou wrote:
>> From: Abhinav Kumar <quic_abhinavk@quicinc.com>
>>
>> DP MST streams share 64 MTP slots in a time-multiplexed manner. Add
>> support for calculating the rate governor, slot allocation, and slot
>> reservation in the DP controller.
>>
>> Each MST stream can reserve its slots by calling
>> msm_dp_display_set_stream_info() from its bridge callbacks.
>>
>> Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
>> Signed-off-by: Yongxing Mou <yongxing.mou@oss.qualcomm.com>
>> ---
>>   drivers/gpu/drm/msm/dp/dp_ctrl.c    | 192 ++++++++++++++++++++++++++++++++++++
>>   drivers/gpu/drm/msm/dp/dp_ctrl.h    |   4 +
>>   drivers/gpu/drm/msm/dp/dp_display.c |  17 ++++
>>   drivers/gpu/drm/msm/dp/dp_display.h |   2 +
>>   drivers/gpu/drm/msm/dp/dp_panel.c   |   6 ++
>>   drivers/gpu/drm/msm/dp/dp_panel.h   |   1 +
>>   drivers/gpu/drm/msm/dp/dp_reg.h     |  10 ++
>>   7 files changed, 232 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/msm/dp/dp_ctrl.c b/drivers/gpu/drm/msm/dp/dp_ctrl.c
>> index 5b5149b160df..15df82a0caca 100644
>> --- a/drivers/gpu/drm/msm/dp/dp_ctrl.c
>> +++ b/drivers/gpu/drm/msm/dp/dp_ctrl.c
>> @@ -73,6 +73,7 @@
>>   #define MR_LINK_PRBS7 0x100
>>   #define MR_LINK_CUSTOM80 0x200
>>   #define MR_LINK_TRAINING4  0x40
>> +#define DP_MAX_TIME_SLOTS 64
>>   
>>   enum {
>>   	DP_TRAINING_NONE,
>> @@ -109,6 +110,11 @@ struct msm_dp_vc_tu_mapping_table {
>>   	u8 tu_size_minus1;
>>   };
>>   
>> +struct msm_dp_mst_ch_slot_info {
>> +	u32 start_slot;
>> +	u32 tot_slots;
>> +};
>> +
>>   struct msm_dp_ctrl_private {
>>   	struct msm_dp_ctrl msm_dp_ctrl;
>>   	struct drm_device *drm_dev;
>> @@ -143,6 +149,8 @@ struct msm_dp_ctrl_private {
>>   	bool link_clks_on;
>>   	bool stream_clks_on[DP_STREAM_MAX];
>>   	bool mst_active;
>> +
>> +	struct msm_dp_mst_ch_slot_info mst_ch_info[DP_STREAM_MAX];
>>   };
>>   
>>   static inline u32 msm_dp_read_ahb(const struct msm_dp_ctrl_private *ctrl, u32 offset)
>> @@ -289,6 +297,44 @@ static void msm_dp_ctrl_mst_config(struct msm_dp_ctrl_private *ctrl, bool enable
>>   	msm_dp_write_link(ctrl, REG_DP_MAINLINK_CTRL, mainlink_ctrl);
>>   }
>>   
>> +static void msm_dp_ctrl_mst_channel_alloc(struct msm_dp_ctrl_private *ctrl,
>> +					  enum msm_dp_stream_id stream_id, u32 ch_start_slot,
>> +					  u32 tot_slot_cnt)
>> +{
>> +	u32 slot_reg_1 = 0, slot_reg_2 = 0;
>> +
>> +	if (ch_start_slot > DP_MAX_TIME_SLOTS ||
>> +	    (ch_start_slot + tot_slot_cnt > DP_MAX_TIME_SLOTS)) {
>> +		DRM_ERROR("invalid slots start %d, tot %d\n",
>> +			  ch_start_slot, tot_slot_cnt);
> 
> Is this an actually possible error or is it defensive coding?
> 
This looks more like defensive coding. I'll remove it.The DRM framework 
will take care of this.
>> +		return;
>> +	}
>> +
>> +	drm_dbg_dp(ctrl->drm_dev, "stream_id %d, start_slot %d, tot_slot %d\n",
>> +		   stream_id, ch_start_slot, tot_slot_cnt);
>> +
>> +	if (ch_start_slot && tot_slot_cnt) {
>> +		u64 mask = GENMASK_ULL(ch_start_slot + tot_slot_cnt - 2, ch_start_slot - 1);
>> +
>> +		slot_reg_1 = mask & 0xFFFFFFFF;
>> +		slot_reg_2 = (mask >> 32) & 0xFFFFFFFF;
>> +	}
>> +
>> +	msm_dp_write_stream_link(ctrl, stream_id, REG_DP_DP0_TIMESLOT_1_32, slot_reg_1);
>> +	msm_dp_write_stream_link(ctrl, stream_id, REG_DP_DP0_TIMESLOT_33_63, slot_reg_2);
>> +}
>> +
>> +static void msm_dp_ctrl_update_rg(struct msm_dp_ctrl_private *ctrl,
>> +				  enum msm_dp_stream_id stream_id, u32 x_int, u32 y_frac_enum)
>> +{
>> +	u32 rg = y_frac_enum | (x_int << 16);
>> +
>> +	drm_dbg_dp(ctrl->drm_dev, "stream_id: %d x_int:%d y_frac_enum:%d rg:%d\n",
>> +		   stream_id, x_int, y_frac_enum, rg);
>> +
>> +	msm_dp_write_stream_link(ctrl, stream_id, REG_DP_DP0_RG, rg);
>> +}
>> +
>>   /*
>>    * NOTE: resetting DP controller will also clear any pending HPD related interrupts
>>    */
>> @@ -2619,6 +2665,117 @@ static void msm_dp_ctrl_config_msa(struct msm_dp_ctrl_private *ctrl,
>>   	msm_dp_write_stream_link(ctrl, panel->stream_id, REG_DP_SOFTWARE_NVID, nvid);
>>   }
>>   
>> +/*
>> + * Calculate MST Rate Governor parameters x_int and y_frac_enum (HPG 3.8.1.2).
> 
> HPG links are pretty useless.
> 
Got it.
>> + *
>> + * The RG paces symbol delivery per MTP via: M = x_int + y_frac_enum/256
>> + * where M is the target symbol count per MTP across all lanes.
>> + *
>> + * min_slot_cnt = (pclk * bpp/8) / (lclk * lanes) * 64   -- slots at 1.0x BW
>> + * max_slot_cnt = pbn * 54 / (lclk * lanes)               -- slots at PBN limit
>> + * raw_target_sc = (min + max) / 2                        -- midpoint (~1.003x)
>> + *
>> + * Quantize raw_target_sc to 1/(256*lanes) steps, then:
>> + *   M           = Chosen_TARGET_Slot_Count * lanes
>> + *   x_int       = INT(M)
>> + *   y_frac_enum = CEIL(256 * MOD(M, 1))
>> + */
>> +static void msm_dp_ctrl_mst_calculate_rg(struct msm_dp_ctrl_private *ctrl,
>> +					 struct msm_dp_panel *panel,
>> +					 u32 *p_x_int, u32 *p_y_frac_enum)
>> +{
>> +	u64 min_slot_cnt, max_slot_cnt;
>> +	u64 raw_target_sc, target_sc_fixp;
>> +	u64 ts_denom, ts_enum, ts_int;
>> +	u64 pclk = panel->msm_dp_mode.drm_mode.clock;
>> +	u64 lclk = 0;
>> +	u64 lanes = ctrl->link->link_params.num_lanes;
>> +	u64 bpp = panel->msm_dp_mode.bpp;
>> +	u64 pbn = panel->pbn;
>> +	u64 numerator, denominator, temp, temp1, temp2;
>> +	u32 x_int = 0, y_frac_enum = 0;
>> +	u64 target_strm_sym, ts_int_fixp, ts_frac_fixp, y_frac_enum_fixp;
>> +
>> +	lclk = ctrl->link->link_params.rate;
>> +
>> +	/* min_slot_cnt */
>> +	numerator = pclk * bpp * 64 * 1000;
>> +	denominator = lclk * lanes * 8 * 1000;
>> +	min_slot_cnt = drm_fixp_from_fraction(numerator, denominator);
>> +
>> +	/* max_slot_cnt */
>> +	numerator = pbn * 54 * 1000;
>> +	denominator = lclk * lanes;
>> +	max_slot_cnt = drm_fixp_from_fraction(numerator, denominator);
>> +
>> +	/* raw_target_sc */
>> +	numerator = max_slot_cnt + min_slot_cnt;
>> +	denominator = drm_fixp_from_fraction(2, 1);
>> +	raw_target_sc = drm_fixp_div(numerator, denominator);
>> +
>> +	/* target_sc */
>> +	temp = drm_fixp_from_fraction(256 * lanes, 1);
>> +	numerator = drm_fixp_mul(raw_target_sc, temp);
>> +	denominator = drm_fixp_from_fraction(256 * lanes, 1);
>> +	target_sc_fixp = drm_fixp_div(numerator, denominator);
>> +
>> +	ts_enum = 256 * lanes;
>> +	ts_denom = drm_fixp_from_fraction(256 * lanes, 1);
>> +	ts_int = drm_fixp2int(target_sc_fixp);
>> +
>> +	temp = drm_fixp2int_ceil(raw_target_sc);
>> +	if (temp != ts_int) {
>> +		temp = drm_fixp_from_fraction(ts_int, 1);
>> +		temp1 = raw_target_sc - temp;
>> +		temp2 = drm_fixp_mul(temp1, ts_denom);
>> +		ts_enum = drm_fixp2int(temp2);
>> +	}
>> +
>> +	/* target_strm_sym */
>> +	ts_int_fixp = drm_fixp_from_fraction(ts_int, 1);
>> +	ts_frac_fixp = drm_fixp_from_fraction(ts_enum, drm_fixp2int(ts_denom));
>> +	temp = ts_int_fixp + ts_frac_fixp;
>> +	temp1 = drm_fixp_from_fraction(lanes, 1);
>> +	target_strm_sym = drm_fixp_mul(temp, temp1);
>> +
>> +	/* x_int */
>> +	x_int = drm_fixp2int(target_strm_sym);
>> +
>> +	/* y_enum_frac */
>> +	temp = drm_fixp_from_fraction(x_int, 1);
>> +	temp1 = target_strm_sym - temp;
>> +	temp2 = drm_fixp_from_fraction(256, 1);
>> +	y_frac_enum_fixp = drm_fixp_mul(temp1, temp2);
>> +
>> +	temp1 = drm_fixp2int(y_frac_enum_fixp);
>> +	temp2 = drm_fixp2int_ceil(y_frac_enum_fixp);
>> +
>> +	y_frac_enum = (u32)((temp1 == temp2) ? temp1 : temp1 + 1);
>> +
>> +	*p_x_int = x_int;
>> +	*p_y_frac_enum = y_frac_enum;
>> +
>> +	drm_dbg_dp(ctrl->drm_dev, "MST lane_cnt:%llu, rate:%llu x_int:%d, y_frac:%d\n",
>> +		   lanes, lclk, x_int, y_frac_enum);
>> +}
>> +
>> +static void msm_dp_ctrl_mst_stream_setup(struct msm_dp_ctrl_private *ctrl,
>> +					 struct msm_dp_panel *panel)
>> +{
>> +	u32 x_int, y_frac_enum;
>> +
>> +	if (!ctrl->mst_active)
>> +		return;
>> +
>> +	drm_dbg_dp(ctrl->drm_dev, "MST stream channel allocation\n");
>> +
>> +	msm_dp_ctrl_mst_stream_channel_slot_setup(&ctrl->msm_dp_ctrl);
>> +
>> +	msm_dp_ctrl_mst_calculate_rg(ctrl, panel, &x_int, &y_frac_enum);
>> +
>> +	msm_dp_ctrl_update_rg(ctrl, panel->stream_id, x_int, y_frac_enum);
>> +}
>> +
>>   int msm_dp_ctrl_prepare_stream_on(struct msm_dp_ctrl *msm_dp_ctrl,
>>   				  struct msm_dp_panel *panel,
>>   				  bool force_link_train)
>> @@ -2708,6 +2865,8 @@ int msm_dp_ctrl_on_stream(struct msm_dp_ctrl *msm_dp_ctrl, struct msm_dp_panel *
>>   	if (!ctrl->mst_active)
>>   		msm_dp_ctrl_setup_tr_unit(ctrl, panel);
>>   
>> +	msm_dp_ctrl_mst_stream_setup(ctrl, panel);
>> +
>>   	msm_dp_write_link(ctrl, REG_DP_STATE_CTRL, DP_STATE_CTRL_SEND_VIDEO);
>>   
>>   	ret = msm_dp_ctrl_mst_send_act(msm_dp_ctrl, panel);
>> @@ -2760,6 +2919,39 @@ void msm_dp_ctrl_off_link(struct msm_dp_ctrl *msm_dp_ctrl,
>>   	phy_power_off(phy);
>>   }
>>   
>> +void msm_dp_ctrl_set_mst_channel_info(struct msm_dp_ctrl *msm_dp_ctrl,
>> +				      enum msm_dp_stream_id stream_id,
>> +				      u32 start_slot, u32 tot_slots)
>> +{
>> +	struct msm_dp_ctrl_private *ctrl;
>> +
>> +	if (!msm_dp_ctrl || stream_id >= DP_STREAM_MAX) {
>> +		DRM_ERROR("invalid input\n");
>> +		return;
>> +	}
>> +
>> +	ctrl = container_of(msm_dp_ctrl, struct msm_dp_ctrl_private, msm_dp_ctrl);
>> +
>> +	ctrl->mst_ch_info[stream_id].start_slot = start_slot;
>> +	ctrl->mst_ch_info[stream_id].tot_slots = tot_slots;
>> +}
>> +
>> +void msm_dp_ctrl_mst_stream_channel_slot_setup(struct msm_dp_ctrl *msm_dp_ctrl)
>> +{
>> +	struct msm_dp_ctrl_private *ctrl;
>> +	int i;
>> +
>> +	ctrl = container_of(msm_dp_ctrl, struct msm_dp_ctrl_private, msm_dp_ctrl);
>> +
>> +	if (!ctrl->mst_active)
>> +		return;
>> +
>> +	for (i = DP_STREAM_0; i < ctrl->num_pixel_clks; i++) {
>> +		msm_dp_ctrl_mst_channel_alloc(ctrl, i, ctrl->mst_ch_info[i].start_slot,
>> +					      ctrl->mst_ch_info[i].tot_slots);
>> +	}
>> +}
>> +
>>   irqreturn_t msm_dp_ctrl_isr(struct msm_dp_ctrl *msm_dp_ctrl,
>>   			    struct msm_dp_panel *panel)
>>   {
>> diff --git a/drivers/gpu/drm/msm/dp/dp_ctrl.h b/drivers/gpu/drm/msm/dp/dp_ctrl.h
>> index 6de028da85fb..e1d10ae20f70 100644
>> --- a/drivers/gpu/drm/msm/dp/dp_ctrl.h
>> +++ b/drivers/gpu/drm/msm/dp/dp_ctrl.h
>> @@ -61,4 +61,8 @@ void msm_dp_ctrl_reinit_phy(struct msm_dp_ctrl *msm_dp_ctrl);
>>   int msm_dp_ctrl_get_stream_cnt(struct msm_dp_ctrl *dp_ctrl);
>>   int msm_dp_ctrl_mst_send_act(struct msm_dp_ctrl *msm_dp_ctrl,
>>   			     struct msm_dp_panel *panel);
>> +void msm_dp_ctrl_mst_stream_channel_slot_setup(struct msm_dp_ctrl *msm_dp_ctrl);
>> +void msm_dp_ctrl_set_mst_channel_info(struct msm_dp_ctrl *msm_dp_ctrl,
>> +				      enum msm_dp_stream_id stream_id,
>> +				      u32 start_slot, u32 tot_slots);
>>   #endif /* _DP_CTRL_H_ */
>> diff --git a/drivers/gpu/drm/msm/dp/dp_display.c b/drivers/gpu/drm/msm/dp/dp_display.c
>> index acb581a8a541..36857d6ed313 100644
>> --- a/drivers/gpu/drm/msm/dp/dp_display.c
>> +++ b/drivers/gpu/drm/msm/dp/dp_display.c
>> @@ -749,6 +749,20 @@ static int msm_dp_display_disable(struct msm_dp_display_private *dp,
>>   	return 0;
>>   }
>>   
>> +int msm_dp_display_set_stream_info(struct msm_dp *msm_dp_display, struct msm_dp_panel *panel,
>> +				   u32 start_slot, u32 num_slots, u32 pbn)
> 
> Why is it a display function rather than a panel or a control one?
> 
>> +{
>> +	struct msm_dp_display_private *dp;
>> +
>> +	dp = container_of(msm_dp_display, struct msm_dp_display_private, msm_dp_display);
>> +
>> +	msm_dp_ctrl_set_mst_channel_info(dp->ctrl, panel->stream_id, start_slot, num_slots);
>> +
>> +	panel->pbn = pbn;
>> +
>> +	return 0;
>> +}
>> +
>>   /**
>>    * msm_dp_bridge_mode_valid - callback to determine if specified mode is valid
>>    * @dp: Pointer to dp display structure
>> @@ -1489,6 +1503,8 @@ void msm_dp_display_atomic_enable(struct msm_dp *msm_dp_display)
>>   
>>   	dp = container_of(msm_dp_display, struct msm_dp_display_private, msm_dp_display);
>>   
>> +	msm_dp_display_set_stream_info(msm_dp_display, dp->panel, 0, 0, 0);
> 
> Why is it being called in the SST case?
> 
Will move this.  In SST case, we don't need to set stream info.
>> +
>>   	rc = msm_dp_display_enable(dp, dp->panel);
>>   	if (rc)
>>   		DRM_ERROR("DP display enable failed, rc=%d\n", rc);
>> @@ -1509,6 +1525,7 @@ void msm_dp_display_atomic_disable(struct msm_dp *dp)
>>   	msm_dp_display = container_of(dp, struct msm_dp_display_private, msm_dp_display);
>>   
>>   	msm_dp_ctrl_push_idle(msm_dp_display->ctrl);
>> +	msm_dp_ctrl_mst_stream_channel_slot_setup(msm_dp_display->ctrl);
>>   	msm_dp_ctrl_mst_send_act(msm_dp_display->ctrl, msm_dp_display->panel);
>>   }
>>   
>> diff --git a/drivers/gpu/drm/msm/dp/dp_display.h b/drivers/gpu/drm/msm/dp/dp_display.h
>> index e987de80522c..45e2cc2d6add 100644
>> --- a/drivers/gpu/drm/msm/dp/dp_display.h
>> +++ b/drivers/gpu/drm/msm/dp/dp_display.h
>> @@ -43,5 +43,7 @@ void msm_dp_display_atomic_enable(struct msm_dp *dp_display);
>>   enum drm_mode_status msm_dp_display_mode_valid(struct msm_dp *dp,
>>   					       const struct drm_display_info *info,
>>   					       const struct drm_display_mode *mode);
>> +int msm_dp_display_set_stream_info(struct msm_dp *msm_dp_display, struct msm_dp_panel *panel,
>> +				   u32 start_slot, u32 num_slots, u32 pbn);
>>   
>>   #endif /* _DP_DISPLAY_H_ */
>> diff --git a/drivers/gpu/drm/msm/dp/dp_panel.c b/drivers/gpu/drm/msm/dp/dp_panel.c
>> index e0c0e8c9178c..ef2ded8ec4ea 100644
>> --- a/drivers/gpu/drm/msm/dp/dp_panel.c
>> +++ b/drivers/gpu/drm/msm/dp/dp_panel.c
>> @@ -57,6 +57,12 @@ u32 msm_dp_stream_reg(enum msm_dp_stream_id id, u32 reg)
>>   		return is_s1 ? REG_DP1_ACTIVE_HOR_VER : REG_DP_MSTLINK_ACTIVE_HOR_VER;
>>   	case REG_DP_MISC1_MISC0:
>>   		return is_s1 ? REG_DP1_MISC1_MISC0 : REG_DP_MSTLINK_MISC1_MISC0;
>> +	case REG_DP_DP0_TIMESLOT_1_32:
>> +		return is_s1 ? REG_DP_DP1_TIMESLOT_1_32 : REG_DP_MSTLINK_TIMESLOT_1_32;
>> +	case REG_DP_DP0_TIMESLOT_33_63:
>> +		return is_s1 ? REG_DP_DP1_TIMESLOT_33_63 : REG_DP_MSTLINK_TIMESLOT_33_63;
>> +	case REG_DP_DP0_RG:
>> +		return is_s1 ? REG_DP_DP1_RG : REG_DP_MSTLINK_DP_RG;
> 
> Get all registers handled by these functions at once. There is no need
> to keep it being patched over and over again.
> 
Got it. I'll split the register definitions and this function into a 
separate patch in the next revision, and place it before the patch that 
introduces multi-link support.
>>   	case MMSS_DP_SDP_CFG:
>>   		return is_s1 ? MMSS_DP1_SDP_CFG : MMSS_DP_MSTLINK_SDP_CFG;
>>   	case MMSS_DP_SDP_CFG2:
>> diff --git a/drivers/gpu/drm/msm/dp/dp_panel.h b/drivers/gpu/drm/msm/dp/dp_panel.h
>> index dc046fec24fc..3e78af9e430d 100644
>> --- a/drivers/gpu/drm/msm/dp/dp_panel.h
>> +++ b/drivers/gpu/drm/msm/dp/dp_panel.h
>> @@ -50,6 +50,7 @@ struct msm_dp_panel {
>>   	u32 hw_revision;
>>   
>>   	enum msm_dp_stream_id stream_id;
>> +	u32 pbn;
>>   
>>   	u32 max_bw_code;
>>   };
>> diff --git a/drivers/gpu/drm/msm/dp/dp_reg.h b/drivers/gpu/drm/msm/dp/dp_reg.h
>> index deb40ed24654..f2bd96f3bbd0 100644
>> --- a/drivers/gpu/drm/msm/dp/dp_reg.h
>> +++ b/drivers/gpu/drm/msm/dp/dp_reg.h
>> @@ -338,7 +338,13 @@
>>   #define DP_TPG_VIDEO_CONFIG_BPP_8BIT		(0x00000001)
>>   #define DP_TPG_VIDEO_CONFIG_RGB			(0x00000004)
>>   
>> +
>> +#define REG_DP_MSTLINK_DP_RG			(0X0000011C)
>>   #define REG_DP1_CONFIGURATION_CTRL		(0x00000400)
>> +#define REG_DP_DP0_TIMESLOT_1_32		(0x00000404)
>> +#define REG_DP_DP0_TIMESLOT_33_63		(0x00000408)
>> +#define REG_DP_DP1_TIMESLOT_1_32		(0x0000040C)
>> +#define REG_DP_DP1_TIMESLOT_33_63		(0x00000410)
>>   #define REG_DP1_SOFTWARE_MVID			(0x00000414)
>>   #define REG_DP1_SOFTWARE_NVID			(0x00000418)
>>   #define REG_DP1_TOTAL_HOR_VER			(0x0000041C)
>> @@ -359,8 +365,12 @@
>>   #define MMSS_DP1_SDP_CFG			(0x000004E0)
>>   #define MMSS_DP1_SDP_CFG2			(0x000004E4)
>>   #define MMSS_DP1_SDP_CFG3			(0x000004E8)
>> +#define REG_DP_DP0_RG				(0x000004F8)
>> +#define REG_DP_DP1_RG				(0x000004FC)
>>   
>>   #define REG_DP_MSTLINK_CONFIGURATION_CTRL	(0x00000034)
>> +#define REG_DP_MSTLINK_TIMESLOT_1_32		(0x00000038)
>> +#define REG_DP_MSTLINK_TIMESLOT_33_63		(0x0000003C)
>>   #define REG_MSTLINK_SOFTWARE_MVID		(0x00000040)
>>   #define REG_MSTLINK_SOFTWARE_NVID		(0x00000044)
>>   #define REG_DP_MSTLINK_TOTAL_HOR_VER		(0x00000048)
>>
>> -- 
>> 2.43.0
>>
> 


  reply	other threads:[~2026-08-26  7:48 UTC|newest]

Thread overview: 63+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-29 14:14 [PATCH RESEND v5 00/25] drm/msm/dp: Add MST support for MSM chipsets Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 01/25] drm/msm/dp: introduce stream_id for each DP panel Yongxing Mou
2026-07-12 11:11   ` Dmitry Baryshkov
2026-08-21  9:26     ` Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 02/25] drm/msm/dp: introduce max_streams for DP controller MST support Yongxing Mou
2026-07-12 11:17   ` Dmitry Baryshkov
2026-08-26  7:48     ` Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 03/25] drm/msm/dp: Add support for programming p1/p2/p3 register blocks Yongxing Mou
2026-07-12 11:23   ` Dmitry Baryshkov
2026-08-21  9:27     ` Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 04/25] drm/msm/dp: use stream_id to change offsets in dp_catalog Yongxing Mou
2026-07-12 11:29   ` Dmitry Baryshkov
2026-06-29 14:14 ` [PATCH RESEND v5 05/25] drm/msm/dp: add support to send ACT packets for MST Yongxing Mou
2026-07-12 13:56   ` Dmitry Baryshkov
2026-08-26  7:48     ` Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 06/25] drm/msm/dp: Add support to enable MST in mainlink control Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 07/25] drm/msm/dp: no need to update tu calculation for mst Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 08/25] drm/msm/dp: Add support for MST channel slot allocation Yongxing Mou
2026-07-12 18:57   ` Dmitry Baryshkov
2026-08-26  7:48     ` Yongxing Mou [this message]
2026-06-29 14:14 ` [PATCH RESEND v5 09/25] drm/msm/dp: Add support for sending VCPF packets in DP controller Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 10/25] drm/msm/dp: Always program MST_FIFO_CONSTANT_FILL for MST use cases Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 11/25] drm/msm/dp: move link-level teardown from display_disable to display_unprepare Yongxing Mou
2026-07-12 18:20   ` Dmitry Baryshkov
2026-08-25  6:50     ` Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 12/25] drm/msm/dp: factor out _helper variants of bridge ops accepting a panel Yongxing Mou
2026-07-12 18:43   ` Dmitry Baryshkov
2026-06-29 14:14 ` [PATCH RESEND v5 13/25] drm/msm/dp: replace power_on with active_stream_cnt for dp_display Yongxing Mou
2026-07-12 18:51   ` Dmitry Baryshkov
2026-08-26  7:48     ` Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 14/25] drm/msm/dp: Mark the SST bridge disconnected when mst is active Yongxing Mou
2026-07-12 21:00   ` Dmitry Baryshkov
2026-08-26  7:49     ` Yongxing Mou
2026-07-12 21:03   ` Dmitry Baryshkov
2026-06-29 14:14 ` [PATCH RESEND v5 15/25] drm/msm/dp: add an API to initialize MST on sink side Yongxing Mou
2026-07-12 21:35   ` Dmitry Baryshkov
2026-08-26  7:48     ` Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 16/25] drm/msm/dp: add msm_dp_display_get_panel() to initialize DP panel Yongxing Mou
2026-07-12 21:56   ` Dmitry Baryshkov
2026-06-29 14:14 ` [PATCH RESEND v5 17/25] drm/msm/dp: add link_ready to manage link-level operations Yongxing Mou
2026-07-12 23:46   ` Dmitry Baryshkov
2026-08-26  7:49     ` Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 18/25] drm/msm/dpu: initialize encoders per stream for DP MST Yongxing Mou
2026-07-12 23:55   ` Dmitry Baryshkov
2026-06-29 14:14 ` [PATCH RESEND v5 19/25] drm/msm/dp: initialize dp_mst module for each DP MST controller Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 20/25] drm/msm/dpu: expose dpu_encoder ops for DP MST reuse Yongxing Mou
2026-07-13  0:01   ` Dmitry Baryshkov
2026-08-26  7:48     ` Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 21/25] drm/msm/dpu: use msm_dp_get_mst_intf_id() to get the intf id Yongxing Mou
2026-07-13  0:09   ` Dmitry Baryshkov
2026-08-26  7:48     ` Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 22/25] drm/msm/dp: wire MST helpers into atomic check and commit paths Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 23/25] drm/msm/dp: add dp_mst_drm to manage DP MST encoder operations Yongxing Mou
2026-07-13  0:40   ` Dmitry Baryshkov
2026-08-26  7:49     ` Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 24/25] drm/msm/dp: add connector abstraction for DP MST Yongxing Mou
2026-07-13  0:52   ` Dmitry Baryshkov
2026-08-25  6:29     ` Yongxing Mou
2026-06-29 14:14 ` [PATCH RESEND v5 25/25] drm/msm/dp: add HPD callback for dp MST Yongxing Mou
2026-07-13  0:54   ` Dmitry Baryshkov
2026-08-25  6:29     ` Yongxing Mou
2026-07-12 11:04 ` [PATCH RESEND v5 00/25] drm/msm/dp: Add MST support for MSM chipsets Dmitry Baryshkov
2026-08-26  9:17   ` Yongxing Mou

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=3edc542f-3991-48cf-89cf-01c7fdecfaee@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®