Commit 8b2cb32c authored by Hersen Wu's avatar Hersen Wu Committed by Alex Deucher
Browse files

drm/amd/display: FEC overhead should be checked once for mst slot nums 



[Why] Mst slot nums equals to pbn / pbn_div.

Today, pbn_div refers to dm_mst_get_pbn_divider ->
dc_link_bandwidth_kbps. In dp_link_bandwidth_kbps,
which includes effect of FEC overhead already. As
result, we should not include effect of FEC overhead
again while calculating pbn by kpbs_to_peak_pbn
(stream_kbps).

[How] Include FEC overhead within dp_link_bandwidth_kbps.
Remove FEC overhead from kbps_to_peak_pbn.

Reviewed-by: default avatarWayne Lin <wayne.lin@amd.com>
Acked-by: default avatarRoman Li <roman.li@amd.com>
Signed-off-by: default avatarHersen Wu <hersenxs.wu@amd.com>
Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 029faefb

drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c

+10 −27
Original line number Diff line number Diff line
@@ -791,25 +791,12 @@ struct dsc_mst_fairness_params { 
	struct amdgpu_dm_connector *aconnector;

};



static uint16_t get_fec_overhead_multiplier(struct dc_link *dc_link)

{

	u8 link_coding_cap;

	uint16_t fec_overhead_multiplier_x1000 = PBN_FEC_OVERHEAD_MULTIPLIER_8B_10B;



	link_coding_cap = dc_link_dp_mst_decide_link_encoding_format(dc_link);

	if (link_coding_cap == DP_128b_132b_ENCODING)

		fec_overhead_multiplier_x1000 = PBN_FEC_OVERHEAD_MULTIPLIER_128B_132B;



	return fec_overhead_multiplier_x1000;

}



static int kbps_to_peak_pbn(int kbps, uint16_t fec_overhead_multiplier_x1000)

static int kbps_to_peak_pbn(int kbps)

{

	u64 peak_kbps = kbps;



	peak_kbps *= 1006;

	peak_kbps *= fec_overhead_multiplier_x1000;

	peak_kbps = div_u64(peak_kbps, 1000 * 1000);

	peak_kbps = div_u64(peak_kbps, 1000);

	return (int) DIV64_U64_ROUND_UP(peak_kbps * 64, (54 * 8 * 1000));

}
@@ -910,12 +897,11 @@ static int increase_dsc_bpp(struct drm_atomic_state *state, 
	int link_timeslots_used;

	int fair_pbn_alloc;

	int ret = 0;

	uint16_t fec_overhead_multiplier_x1000 = get_fec_overhead_multiplier(dc_link);



	for (i = 0; i < count; i++) {

		if (vars[i + k].dsc_enabled) {

			initial_slack[i] =

			kbps_to_peak_pbn(params[i].bw_range.max_kbps, fec_overhead_multiplier_x1000) - vars[i + k].pbn;

			kbps_to_peak_pbn(params[i].bw_range.max_kbps) - vars[i + k].pbn;

			bpp_increased[i] = false;

			remaining_to_increase += 1;

		} else {
@@ -1011,7 +997,6 @@ static int try_disable_dsc(struct drm_atomic_state *state, 
	int next_index;

	int remaining_to_try = 0;

	int ret;

	uint16_t fec_overhead_multiplier_x1000 = get_fec_overhead_multiplier(dc_link);



	for (i = 0; i < count; i++) {

		if (vars[i + k].dsc_enabled
@@ -1041,7 +1026,7 @@ static int try_disable_dsc(struct drm_atomic_state *state, 
		if (next_index == -1)

			break;



		vars[next_index].pbn = kbps_to_peak_pbn(params[next_index].bw_range.stream_kbps, fec_overhead_multiplier_x1000);

		vars[next_index].pbn = kbps_to_peak_pbn(params[next_index].bw_range.stream_kbps);

		ret = drm_dp_atomic_find_time_slots(state,

						    params[next_index].port->mgr,

						    params[next_index].port,
@@ -1054,7 +1039,8 @@ static int try_disable_dsc(struct drm_atomic_state *state, 
			vars[next_index].dsc_enabled = false;

			vars[next_index].bpp_x16 = 0;

		} else {

			vars[next_index].pbn = kbps_to_peak_pbn(params[next_index].bw_range.max_kbps, fec_overhead_multiplier_x1000);

			vars[next_index].pbn = kbps_to_peak_pbn(

				params[next_index].bw_range.max_kbps);

			ret = drm_dp_atomic_find_time_slots(state,

							    params[next_index].port->mgr,

							    params[next_index].port,
@@ -1083,7 +1069,6 @@ static int compute_mst_dsc_configs_for_link(struct drm_atomic_state *state, 
	int count = 0;

	int i, k, ret;

	bool debugfs_overwrite = false;

	uint16_t fec_overhead_multiplier_x1000 = get_fec_overhead_multiplier(dc_link);



	memset(params, 0, sizeof(params));
@@ -1148,7 +1133,7 @@ static int compute_mst_dsc_configs_for_link(struct drm_atomic_state *state, 
	/* Try no compression */

	for (i = 0; i < count; i++) {

		vars[i + k].aconnector = params[i].aconnector;

		vars[i + k].pbn = kbps_to_peak_pbn(params[i].bw_range.stream_kbps, fec_overhead_multiplier_x1000);

		vars[i + k].pbn = kbps_to_peak_pbn(params[i].bw_range.stream_kbps);

		vars[i + k].dsc_enabled = false;

		vars[i + k].bpp_x16 = 0;

		ret = drm_dp_atomic_find_time_slots(state, params[i].port->mgr, params[i].port,
@@ -1167,7 +1152,7 @@ static int compute_mst_dsc_configs_for_link(struct drm_atomic_state *state, 
	/* Try max compression */

	for (i = 0; i < count; i++) {

		if (params[i].compression_possible && params[i].clock_force_enable != DSC_CLK_FORCE_DISABLE) {

			vars[i + k].pbn = kbps_to_peak_pbn(params[i].bw_range.min_kbps, fec_overhead_multiplier_x1000);

			vars[i + k].pbn = kbps_to_peak_pbn(params[i].bw_range.min_kbps);

			vars[i + k].dsc_enabled = true;

			vars[i + k].bpp_x16 = params[i].bw_range.min_target_bpp_x16;

			ret = drm_dp_atomic_find_time_slots(state, params[i].port->mgr,
@@ -1175,7 +1160,7 @@ static int compute_mst_dsc_configs_for_link(struct drm_atomic_state *state, 
			if (ret < 0)

				return ret;

		} else {

			vars[i + k].pbn = kbps_to_peak_pbn(params[i].bw_range.stream_kbps, fec_overhead_multiplier_x1000);

			vars[i + k].pbn = kbps_to_peak_pbn(params[i].bw_range.stream_kbps);

			vars[i + k].dsc_enabled = false;

			vars[i + k].bpp_x16 = 0;

			ret = drm_dp_atomic_find_time_slots(state, params[i].port->mgr,
@@ -1656,13 +1641,11 @@ enum dc_status dm_dp_mst_is_port_support_mode( 
		 */

		int pbn_div, slot_num, max_slot_num;

		enum dc_link_encoding_format link_encoding;

		uint16_t fec_overhead_multiplier_x1000 =

			get_fec_overhead_multiplier(stream->link);

		uint32_t stream_kbps =

			dc_bandwidth_in_kbps_from_timing(&stream->timing,

				dc_link_get_highest_encoding_format(stream->link));



		pbn = kbps_to_peak_pbn(stream_kbps, fec_overhead_multiplier_x1000);

		pbn = kbps_to_peak_pbn(stream_kbps);

		pbn_div = dm_mst_get_pbn_divider(stream->link);

		slot_num = DIV_ROUND_UP(pbn, pbn_div);

drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.h

+0 −3
Original line number Diff line number Diff line
@@ -46,9 +46,6 @@ 
#define SYNAPTICS_CASCADED_HUB_ID  0x5A

#define IS_SYNAPTICS_CASCADED_PANAMERA(devName, data) ((IS_SYNAPTICS_PANAMERA(devName) && ((int)data[2] == SYNAPTICS_CASCADED_HUB_ID)) ? 1 : 0)



#define PBN_FEC_OVERHEAD_MULTIPLIER_8B_10B	1031

#define PBN_FEC_OVERHEAD_MULTIPLIER_128B_132B	1000



enum mst_msg_ready_type {

	NONE_MSG_RDY_EVENT = 0,

	DOWN_REP_MSG_RDY_EVENT = 1,