[PATCH 01/14] drm/amd/display: Fix DCN35 set min dispclk logic

[Hamza Mahfooz]

From: Nicholas Susanto <nicholas.susa...@amd.com>

[Why]

Setting min dispclk to 50Mhz outside clock lowering function causes
unnecessary calls to SMU to lower dispclk and causes dentist hangs when
there is no stream on the pipes.

[How]

Move the set minimum dispclk logic inside the lowering dispclk if
statement.

Fixes: 2dd29403b206 ("DCN35 set min dispclk to 50Mhz")
Reviewed-by: Nicholas Kazlauskas <nicholas.kazlaus...@amd.com>
Signed-off-by: Nicholas Susanto <nicholas.susa...@amd.com>
Signed-off-by: Hamza Mahfooz <hamza.mahf...@amd.com>
---
 .../gpu/drm/amd/display/dc/clk_mgr/dcn35/dcn35_clk_mgr.c    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/dc/clk_mgr/dcn35/dcn35_clk_mgr.c 
b/drivers/gpu/drm/amd/display/dc/clk_mgr/dcn35/dcn35_clk_mgr.c
index f50054089da7..97164b5585a8 100644
--- a/drivers/gpu/drm/amd/display/dc/clk_mgr/dcn35/dcn35_clk_mgr.c
+++ b/drivers/gpu/drm/amd/display/dc/clk_mgr/dcn35/dcn35_clk_mgr.c
@@ -305,9 +305,6 @@ void dcn35_update_clocks(struct clk_mgr *clk_mgr_base,
        if (new_clocks->dtbclk_en && !new_clocks->ref_dtbclk_khz)
                new_clocks->ref_dtbclk_khz = 600000;
 
-       if (dc->debug.min_disp_clk_khz > 0 && new_clocks->dispclk_khz < 
dc->debug.min_disp_clk_khz)
-               new_clocks->dispclk_khz = dc->debug.min_disp_clk_khz;
-
        /*
         * if it is safe to lower, but we are already in the lower state, we 
don't have to do anything
         * also if safe to lower is false, we just go in the higher state
@@ -385,6 +382,9 @@ void dcn35_update_clocks(struct clk_mgr *clk_mgr_base,
        if (should_set_clock(safe_to_lower, new_clocks->dispclk_khz, 
clk_mgr_base->clks.dispclk_khz)) {
                dcn35_disable_otg_wa(clk_mgr_base, context, safe_to_lower, 
true);
 
+               if (dc->debug.min_disp_clk_khz > 0 && new_clocks->dispclk_khz < 
dc->debug.min_disp_clk_khz)
+                       new_clocks->dispclk_khz = dc->debug.min_disp_clk_khz;
+
                clk_mgr_base->clks.dispclk_khz = new_clocks->dispclk_khz;
                dcn35_smu_set_dispclk(clk_mgr, clk_mgr_base->clks.dispclk_khz);
                dcn35_disable_otg_wa(clk_mgr_base, context, safe_to_lower, 
false);
-- 
2.46.0