Commit 5b25e5f1 authored by Harmanprit Tatla's avatar Harmanprit Tatla Committed by Alex Deucher

drm/amd/display: No audio endpoint for Dell MST display

[Why]
There are certain MST displays (i.e. Dell P2715Q)
that although have the MST feature set to off may still
report it is a branch device and a non-zero
value for downstream port present.
This can lead to us incorrectly classifying a
dp dongle connection as being active and
disabling the audio endpoint for the display.

[How]
Modified the placement and
condition used to assign
the is_branch_dev bit.
Signed-off-by: default avatarHarmanprit Tatla <harmanprit.tatla@amd.com>
Reviewed-by: default avatarAric Cyr <aric.cyr@amd.com>
Acked-by: default avatarAnthony Koo <Anthony.Koo@amd.com>
Acked-by: default avatarLeo Li <sunpeng.li@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent db31af12
...@@ -2230,11 +2230,18 @@ static void get_active_converter_info( ...@@ -2230,11 +2230,18 @@ static void get_active_converter_info(
link->dpcd_caps.dongle_type = DISPLAY_DONGLE_NONE; link->dpcd_caps.dongle_type = DISPLAY_DONGLE_NONE;
ddc_service_set_dongle_type(link->ddc, ddc_service_set_dongle_type(link->ddc,
link->dpcd_caps.dongle_type); link->dpcd_caps.dongle_type);
link->dpcd_caps.is_branch_dev = false;
return; return;
} }
/* DPCD 0x5 bit 0 = 1, it indicate it's branch device */ /* DPCD 0x5 bit 0 = 1, it indicate it's branch device */
link->dpcd_caps.is_branch_dev = ds_port.fields.PORT_PRESENT; if (ds_port.fields.PORT_TYPE == DOWNSTREAM_DP) {
link->dpcd_caps.is_branch_dev = false;
}
else {
link->dpcd_caps.is_branch_dev = ds_port.fields.PORT_PRESENT;
}
switch (ds_port.fields.PORT_TYPE) { switch (ds_port.fields.PORT_TYPE) {
case DOWNSTREAM_VGA: case DOWNSTREAM_VGA:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment