From: Rodrigo Siqueira <rodrigo.sique...@amd.com>

[WHY & HOW]
This commit just adds some simple comments to help understand the
calculation of V total duration for Freesync. Also, remove a legacy TODO
comment from link service type.

Acked-by: Alex Hung <alex.h...@amd.com>
Signed-off-by: Rodrigo Siqueira <rodrigo.sique...@amd.com>
---
 drivers/gpu/drm/amd/display/include/link_service_types.h | 1 -
 drivers/gpu/drm/amd/display/modules/freesync/freesync.c  | 4 ++--
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/include/link_service_types.h 
b/drivers/gpu/drm/amd/display/include/link_service_types.h
index 92dbff22a7c6..1867aac57cf2 100644
--- a/drivers/gpu/drm/amd/display/include/link_service_types.h
+++ b/drivers/gpu/drm/amd/display/include/link_service_types.h
@@ -73,7 +73,6 @@ struct link_training_settings {
        enum dc_pre_emphasis *pre_emphasis;
        enum dc_post_cursor2 *post_cursor2;
        bool should_set_fec_ready;
-       /* TODO - factor lane_settings out because it changes during LT */
        union dc_dp_ffe_preset *ffe_preset;
 
        uint16_t cr_pattern_time;
diff --git a/drivers/gpu/drm/amd/display/modules/freesync/freesync.c 
b/drivers/gpu/drm/amd/display/modules/freesync/freesync.c
index fbaa6effd0e3..b19ef58d1555 100644
--- a/drivers/gpu/drm/amd/display/modules/freesync/freesync.c
+++ b/drivers/gpu/drm/amd/display/modules/freesync/freesync.c
@@ -158,13 +158,13 @@ static unsigned int calc_v_total_from_duration(
        if (duration_in_us > vrr->max_duration_in_us)
                duration_in_us = vrr->max_duration_in_us;
 
-       if (dc_is_hdmi_signal(stream->signal)) {
+       if (dc_is_hdmi_signal(stream->signal)) { // change for HDMI to comply 
with spec
                uint32_t h_total_up_scaled;
 
                h_total_up_scaled = stream->timing.h_total * 10000;
                v_total = div_u64((unsigned long long)duration_in_us
                                        * stream->timing.pix_clk_100hz + 
(h_total_up_scaled - 1),
-                                       h_total_up_scaled);
+                                       h_total_up_scaled); //ceiling for MMax 
and MMin for MVRR
        } else {
                v_total = div64_u64(div64_u64(((unsigned long long)(
                                        duration_in_us) * 
(stream->timing.pix_clk_100hz / 10)),
-- 
2.34.1

Reply via email to