From: Alex Hung <[email protected]>

[WHAT]
Remove pp_rn_set_wm_ranges and reuse the identical
pp_nv_set_wm_ranges for the DCN_VERSION_2_1 case instead.

Assisted-by: Copilot:Claude-Opus-4.6
Reviewed-by: Bhawanpreet Lakha <[email protected]>
Signed-off-by: Alex Hung <[email protected]>
Signed-off-by: Aurabindo Pillai <[email protected]>
---
 .../drm/amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c    | 13 +------------
 1 file changed, 1 insertion(+), 12 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c
index 2cdb8fea504a..2fda6fbed88f 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c
@@ -686,17 +686,6 @@ static enum pp_smu_status pp_rn_get_dpm_clock_table(
        return PP_SMU_RESULT_OK;
 }
 
-static enum pp_smu_status pp_rn_set_wm_ranges(struct pp_smu *pp,
-               struct pp_smu_wm_range_sets *ranges)
-{
-       const struct dc_context *ctx = pp->dm;
-       struct amdgpu_device *adev = ctx->driver_context;
-
-       amdgpu_dpm_set_watermarks_for_clocks_ranges(adev, ranges);
-
-       return PP_SMU_RESULT_OK;
-}
-
 void dm_pp_get_funcs(
                struct dc_context *ctx,
                struct pp_smu_funcs *funcs)
@@ -743,7 +732,7 @@ void dm_pp_get_funcs(
        case DCN_VERSION_2_1:
                funcs->ctx.ver = PP_SMU_VER_RN;
                funcs->rn_funcs.pp_smu.dm = ctx;
-               funcs->rn_funcs.set_wm_ranges = pp_rn_set_wm_ranges;
+               funcs->rn_funcs.set_wm_ranges = pp_nv_set_wm_ranges;
                funcs->rn_funcs.get_dpm_clock_table = pp_rn_get_dpm_clock_table;
                break;
        default:
-- 
2.54.0

Reply via email to