[Why] DisplayID vrr parsing was always attempted, even though it's data is only ever used in the DP/eDP path.
[How] Move the parsing to where it's actually needed Signed-off-by: Tomasz Pakuła <[email protected]> --- drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c index 903264b26d1f..be3bb5a86959 100644 --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c @@ -13326,12 +13326,11 @@ void amdgpu_dm_update_freesync_caps(struct drm_connector *connector, if (amdgpu_dm_connector->dc_link) dpcd_caps = amdgpu_dm_connector->dc_link->dpcd_caps; - /* Some eDP panels only have the refresh rate range info in DisplayID */ - if (is_monitor_range_invalid(connector)) - parse_edid_displayid_vrr(connector, edid); - if (sink->sink_signal == SIGNAL_TYPE_DISPLAY_PORT || sink->sink_signal == SIGNAL_TYPE_EDP) { + /* Some eDP panels only have the refresh rate range info in DisplayID */ + if (is_monitor_range_invalid(connector)) + parse_edid_displayid_vrr(connector, edid); if (dpcd_caps.allow_invalid_MSA_timing_param) freesync_capable = copy_range_to_amdgpu_connector(connector); -- 2.53.0
