Skip to content

Commit a3ef3c2

Browse files
ideakjlahtine-intel
authored andcommitted
drm/dp: Change AUX DPCD probe address from DPCD_REV to LANE0_1_STATUS
Reading DPCD registers has side-effects in general. In particular accessing registers outside of the link training register range (0x102-0x106, 0x202-0x207, 0x200c-0x200f, 0x2216) is explicitly forbidden by the DP v2.1 Standard, see 3.6.5.1 DPTX AUX Transaction Handling Mandates 3.6.7.4 128b/132b DP Link Layer LTTPR Link Training Mandates Based on my tests, accessing the DPCD_REV register during the link training of an UHBR TBT DP tunnel sink leads to link training failures. Solve the above by using the DP_LANE0_1_STATUS (0x202) register for the DPCD register access quirk. Cc: <[email protected]> Cc: Ville Syrjälä <[email protected]> Cc: Jani Nikula <[email protected]> Acked-by: Jani Nikula <[email protected]> Signed-off-by: Imre Deak <[email protected]> Link: https://lore.kernel.org/r/[email protected] (cherry picked from commit a40c5d7) Signed-off-by: Joonas Lahtinen <[email protected]>
1 parent 9205999 commit a3ef3c2

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/gpu/drm/display/drm_dp_helper.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -725,7 +725,7 @@ ssize_t drm_dp_dpcd_read(struct drm_dp_aux *aux, unsigned int offset,
725725
* monitor doesn't power down exactly after the throw away read.
726726
*/
727727
if (!aux->is_remote) {
728-
ret = drm_dp_dpcd_probe(aux, DP_DPCD_REV);
728+
ret = drm_dp_dpcd_probe(aux, DP_LANE0_1_STATUS);
729729
if (ret < 0)
730730
return ret;
731731
}

0 commit comments

Comments
 (0)