drm/amd/display: Fix LFC multiplier changing erratically
authorAnthony Koo <Anthony.Koo@amd.com>
Wed, 29 Jul 2020 21:33:27 +0000 (17:33 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 10 Aug 2020 22:08:53 +0000 (18:08 -0400)
commite4ed4dbbc8383d42a197da8fe7ca6434b0f14def
tree867efaddd054f1b704acacaea76d09f9b2ae50cf
parenta49f6727e14caff32419cc3002b9ae9cafb750d7
drm/amd/display: Fix LFC multiplier changing erratically

[Why]
1. There is a calculation that is using frame_time_in_us instead of
last_render_time_in_us to calculate whether choosing an LFC multiplier
would cause the inserted frame duration to be outside of range.

2. We do not handle unsigned integer subtraction correctly and it underflows
to a really large value, which causes some logic errors.

[How]
1. Fix logic to calculate 'within range' using last_render_time_in_us
2. Split out delta_from_mid_point_delta_in_us calculation to ensure
we don't underflow and wrap around

Signed-off-by: Anthony Koo <Anthony.Koo@amd.com>
Reviewed-by: Aric Cyr <Aric.Cyr@amd.com>
Acked-by: Qingqing Zhuo <qingqing.zhuo@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/modules/freesync/freesync.c