Attachment 143724 Details for Bug 110138 – 0001-drm-amd-display-Only-allow-VRR-when-vrefresh-is-with.patch

[patch] 0001-drm-amd-display-Only-allow-VRR-when-vrefresh-is-with.patch

0001-drm-amd-display-Only-allow-VRR-when-vrefresh-is-with.patch (text/plain), 1.72 KB, created by Nicholas Kazlauskas on 2019-03-18 13:04:32 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Nicholas Kazlauskas

Created: 2019-03-18 13:04:32 UTC

Size: 1.72 KB

patch

obsolete

>From b6d0c42794824d94a10a6599ef110d7a1c371246 Mon Sep 17 00:00:00 2001
>From: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com>
>Date: Mon, 18 Mar 2019 09:00:16 -0400
>Subject: [PATCH] drm/amd/display: Only allow VRR when vrefresh is within
> supported range
>
>[Why]
>Black screens or artifacting can occur when enabling FreeSync outside
>of the supported range of the monitor. This can happen since the
>supported range isn't always the min/max vrefresh range available for
>the monitor.
>
>[How]
>There was previously a fix that prevented this from happening in the
>low range but it didn't cover the upper range. Expand the condition
>to include both.
>
>Cc: Sun peng Li <Sunpeng.Li@amd.com>
>Cc: Harry Wentland <Harry.Wentland@amd.com>
>Signed-off-by: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com>
>---
> drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
>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 cde0da3ae964..73e544f645ef 100644
>--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
>+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
>@@ -5555,9 +5555,11 @@ static void get_freesync_config_for_crtc(
> 	struct amdgpu_dm_connector *aconnector =
> 			to_amdgpu_dm_connector(new_con_state->base.connector);
> 	struct drm_display_mode *mode = &new_crtc_state->base.mode;
>+	int vrefresh = drm_mode_vrefresh(mode);
> 
> 	new_crtc_state->vrr_supported = new_con_state->freesync_capable &&
>-		aconnector->min_vfreq <= drm_mode_vrefresh(mode);
>+					vrefresh >= aconnector->min_vfreq &&
>+					vrefresh <= aconnector->max_vfreq;
> 
> 	if (new_crtc_state->vrr_supported) {
> 		new_crtc_state->stream->ignore_msa_timing_param = true;
>-- 
>2.17.1
>

Actions: View | Splinter Review

Attachments on bug 110138: 143688 | 143689 | 143724

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.