Commit 94b38b89 authored by Zhikai Zhai's avatar Zhikai Zhai Committed by Alex Deucher
Browse files

drm/amd/display: Add align done check 



[WHY]
We Double-check link status if training successful,
but miss the lane align status.

[HOW]
Add the lane align status check

Cc: Mario Limonciello <mario.limonciello@amd.com>
Cc: Alex Deucher <alexander.deucher@amd.com>
Cc: stable@vger.kernel.org
Reviewed-by: default avatarWenjing Liu <wenjing.liu@amd.com>
Acked-by: default avatarAurabindo Pillai <aurabindo.pillai@amd.com>
Signed-off-by: default avatarZhikai Zhai <zhikai.zhai@amd.com>
Tested-by: default avatarDaniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 91636168

drivers/gpu/drm/amd/display/dc/link/protocols/link_dp_training.c

+4 −1
Original line number Diff line number Diff line
@@ -517,6 +517,7 @@ enum link_training_result dp_check_link_loss_status( 
{

	enum link_training_result status = LINK_TRAINING_SUCCESS;

	union lane_status lane_status;

	union lane_align_status_updated dpcd_lane_status_updated;

	uint8_t dpcd_buf[6] = {0};

	uint32_t lane;
@@ -532,10 +533,12 @@ enum link_training_result dp_check_link_loss_status( 
		 * check lanes status

		 */

		lane_status.raw = dp_get_nibble_at_index(&dpcd_buf[2], lane);

		dpcd_lane_status_updated.raw = dpcd_buf[4];



		if (!lane_status.bits.CHANNEL_EQ_DONE_0 ||

			!lane_status.bits.CR_DONE_0 ||

			!lane_status.bits.SYMBOL_LOCKED_0) {

			!lane_status.bits.SYMBOL_LOCKED_0 ||

			!dp_is_interlane_aligned(dpcd_lane_status_updated)) {

			/* if one of the channel equalization, clock

			 * recovery or symbol lock is dropped

			 * consider it as (link has been