Skip to content

Commit 76d5ef4

Browse files
AMD-aricalexdeucher
authored andcommitted
drm/amd/display: Fix DP Compliance tests 4.3.2.1 and 4.3.2.2
[Why] Test expects that we also read HPD_IRQ_VECTOR when checking for symbol loss as well lane status. [How] Read bytes 0x200-0x205 instead of just 0x202-0x205 Signed-off-by: Aric Cyr <[email protected]> Reviewed-by: Jun Lei <[email protected]> Acked-by: Eryk Brol <[email protected]> Signed-off-by: Alex Deucher <[email protected]>
1 parent b5fe6aa commit 76d5ef4

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

drivers/gpu/drm/amd/display/dc/core/dc_link_dp.c

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1138,23 +1138,22 @@ static enum link_training_result check_link_loss_status(
11381138
const struct link_training_settings *link_training_setting)
11391139
{
11401140
enum link_training_result status = LINK_TRAINING_SUCCESS;
1141-
unsigned int lane01_status_address = DP_LANE0_1_STATUS;
11421141
union lane_status lane_status;
1143-
uint8_t dpcd_buf[4] = {0};
1142+
uint8_t dpcd_buf[6] = {0};
11441143
uint32_t lane;
11451144

11461145
core_link_read_dpcd(
1147-
link,
1148-
lane01_status_address,
1149-
(uint8_t *)(dpcd_buf),
1150-
sizeof(dpcd_buf));
1146+
link,
1147+
DP_SINK_COUNT,
1148+
(uint8_t *)(dpcd_buf),
1149+
sizeof(dpcd_buf));
11511150

11521151
/*parse lane status*/
11531152
for (lane = 0; lane < link->cur_link_settings.lane_count; lane++) {
11541153
/*
11551154
* check lanes status
11561155
*/
1157-
lane_status.raw = get_nibble_at_index(&dpcd_buf[0], lane);
1156+
lane_status.raw = get_nibble_at_index(&dpcd_buf[2], lane);
11581157

11591158
if (!lane_status.bits.CHANNEL_EQ_DONE_0 ||
11601159
!lane_status.bits.CR_DONE_0 ||

0 commit comments

Comments
 (0)