[AMD Official Use Only - AMD Internal Distribution Only] Hi Jerry,
Please fix the Author/committer name. -- Regards, Jay ________________________________ From: Fangzhi Zuo <jerry....@amd.com> Sent: Wednesday, June 18, 2025 11:33 AM To: amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org> Cc: Wentland, Harry <harry.wentl...@amd.com>; Li, Sun peng (Leo) <sunpeng...@amd.com>; Pillai, Aurabindo <aurabindo.pil...@amd.com>; Li, Roman <roman...@amd.com>; Lin, Wayne <wayne....@amd.com>; Chung, ChiaHsuan (Tom) <chiahsuan.ch...@amd.com>; Zuo, Jerry <jerry....@amd.com>; Wheeler, Daniel <daniel.whee...@amd.com>; Wu, Ray <ray...@amd.com>; Hung, Alex <alex.h...@amd.com>; Li, Allen <allen...@amd.com>; Chen, Robin <robin.c...@amd.com> Subject: [PATCH 06/13] drm/amd/display: Remove redundant macro of refresh rate From: weiguali <wei-guang...@amd.com> [Why&How] Found that we add redundant macro on refresh rate when calculating vtotal, so we remove it. Reviewed-by: Robin Chen <robin.c...@amd.com> Signed-off-by: Weiguang Li <wei-guang...@amd.com> Signed-off-by: Fangzhi Zuo <jerry....@amd.com> --- drivers/gpu/drm/amd/display/modules/freesync/freesync.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/display/modules/freesync/freesync.c b/drivers/gpu/drm/amd/display/modules/freesync/freesync.c index 3ba9b62ba70b..6a939f2f2fb9 100644 --- a/drivers/gpu/drm/amd/display/modules/freesync/freesync.c +++ b/drivers/gpu/drm/amd/display/modules/freesync/freesync.c @@ -147,7 +147,7 @@ unsigned int mod_freesync_calc_v_total_from_refresh( ((unsigned int)(div64_u64((1000000000ULL * 1000000), refresh_in_uhz))); - if (MICRO_HZ_TO_HZ(refresh_in_uhz) <= stream->timing.min_refresh_in_uhz) { + if (refresh_in_uhz <= stream->timing.min_refresh_in_uhz) { /* When the target refresh rate is the minimum panel refresh rate, * round down the vtotal value to avoid stretching vblank over * panel's vtotal boundary. -- 2.43.0