<div dir="ltr">While looking at the following commit, I noticed what might be an arithmetic issue potentially stemming from some merge/patch conflict resolution.<br><br>commit ad339f69114a6a145fc94d44376851c53dee3475<br>Author: Jaehyun Chung <<a href="mailto:jaehyun.chung@amd.com">jaehyun.chung@amd.com</a>><br>Date:   Thu Jun 18 15:27:35 2020 -0400<br><br>    drm/amd/display: Fix incorrect rounding for 10Hz refresh range<br>    <br>    [Why]<br>    In cases where refresh range is slightly below 10, FreeSync is not<br>    active or supported. Need to round values before checking refresh range<br>    in order to have FreeSync supported in these cases.<br>    <br>    [How]<br>    Remove redundant values and round values before checking valid refresh range.<br>    <br>    Signed-off-by: Jaehyun Chung <<a href="mailto:jaehyun.chung@amd.com">jaehyun.chung@amd.com</a>><br>    Reviewed-by: Aric Cyr <<a href="mailto:Aric.Cyr@amd.com">Aric.Cyr@amd.com</a>><br>    Acked-by: Anthony Koo <<a href="mailto:Anthony.Koo@amd.com">Anthony.Koo@amd.com</a>><br>    Acked-by: Eryk Brol <<a href="mailto:eryk.brol@amd.com">eryk.brol@amd.com</a>><br>    Signed-off-by: Alex Deucher <<a href="mailto:alexander.deucher@amd.com">alexander.deucher@amd.com</a>><br><br>There appears to be an errant plus sign when calculating the refresh_range - <a href="https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/gpu/drm/amd/display/modules/freesync/freesync.c?h=v5.10-rc1#n948">https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/gpu/drm/amd/display/modules/freesync/freesync.c?h=v5.10-rc1#n948</a><div><br></div><div><div>  refresh_range = div_u64(in_out_vrr->max_refresh_in_uhz + 500000, 1000000) -</div><div>+ div_u64(in_out_vrr->min_refresh_in_uhz + 500000, 1000000);</div><div><br></div>I am unfamiliar with the freesync codebase so I opted to present the potential issue here as opposed to preparing a patch.<br><br>Thanks.<br></div></div>