From: Aric Cyr <[email protected]>

[Why]
Some displays rely on the SPD verticle frequency maximum value.
Must round the calculated refresh rate to the nearest integer.

[How]
Round the nominal calculated refresh rate to the nearest whole
integer.

Change-Id: I4347428f35a899fad028a9ee1b02ea3c84231f22
Signed-off-by: Aric Cyr <[email protected]>
Reviewed-by: Anthony Koo <[email protected]>
Acked-by: Bhawanpreet Lakha <[email protected]>
---
 .../gpu/drm/amd/display/modules/freesync/freesync.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/modules/freesync/freesync.c 
b/drivers/gpu/drm/amd/display/modules/freesync/freesync.c
index 9ce56a8f027d..8fd3d4ec339c 100644
--- a/drivers/gpu/drm/amd/display/modules/freesync/freesync.c
+++ b/drivers/gpu/drm/amd/display/modules/freesync/freesync.c
@@ -743,6 +743,10 @@ void mod_freesync_build_vrr_params(struct mod_freesync 
*mod_freesync,
        nominal_field_rate_in_uhz =
                        mod_freesync_calc_nominal_field_rate(stream);
 
+       /* Rounded to the nearest Hz */
+       nominal_field_rate_in_uhz = 1000000ULL *
+                       div_u64(nominal_field_rate_in_uhz + 500000, 1000000);
+
        min_refresh_in_uhz = in_config->min_refresh_in_uhz;
        max_refresh_in_uhz = in_config->max_refresh_in_uhz;
 
@@ -1002,14 +1006,13 @@ unsigned long long mod_freesync_calc_nominal_field_rate(
                        const struct dc_stream_state *stream)
 {
        unsigned long long nominal_field_rate_in_uhz = 0;
+       unsigned int total = stream->timing.h_total * stream->timing.v_total;
 
-       /* Calculate nominal field rate for stream */
+       /* Calculate nominal field rate for stream, rounded up to nearest 
integer */
        nominal_field_rate_in_uhz = stream->timing.pix_clk_100hz / 10;
        nominal_field_rate_in_uhz *= 1000ULL * 1000ULL * 1000ULL;
-       nominal_field_rate_in_uhz = div_u64(nominal_field_rate_in_uhz,
-                                               stream->timing.h_total);
-       nominal_field_rate_in_uhz = div_u64(nominal_field_rate_in_uhz,
-                                               stream->timing.v_total);
+
+       nominal_field_rate_in_uhz =     div_u64(nominal_field_rate_in_uhz, 
total);
 
        return nominal_field_rate_in_uhz;
 }
-- 
2.17.1

_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to