drm/amd/pm: add xgmi plpd mode selecting interface for smu v13.0.6

Add the interface to change xgmi per-link power down policy.

v2: split from sysfs interface code and miscellaneous updates
v3: check against XGMI_PLPD_DEFAULT/XGMI_PLPD_OPTIMIZED and
    pass PPSMC param

Signed-off-by: Le Ma <le.ma@amd.com>
Reviewed-by: Asad Kamal <asad.kamal@amd.com>
Reviewed-by: Lijo Lazar <lijo.lazar@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
index e6f1620acdd4a64eee75eda73e1dfaaa0eeab71b..893359b26418e2f3506698137687654e0b1f839f 100644 (file)
--- a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
@@ -3157,6 +3157,30 @@ static int smu_get_prv_buffer_details(void *handle, void **addr, size_t *size)
        return 0;
 }
 
+int smu_set_xgmi_plpd_mode(struct smu_context *smu,
+                          enum pp_xgmi_plpd_mode mode)
+{
+       int ret = -EOPNOTSUPP;
+
+       if (!smu->pm_enabled || !smu->adev->pm.dpm_enabled)
+               return ret;
+
+       /* PLPD policy is not supported if it's NONE */
+       if (smu->plpd_mode == XGMI_PLPD_NONE)
+               return ret;
+
+       if (smu->plpd_mode == mode)
+               return 0;
+
+       if (smu->ppt_funcs && smu->ppt_funcs->select_xgmi_plpd_policy)
+               ret = smu->ppt_funcs->select_xgmi_plpd_policy(smu, mode);
+
+       if (!ret)
+               smu->plpd_mode = mode;
+
+       return ret;
+}
+
 static const struct amd_pm_funcs swsmu_pm_funcs = {
        /* export for sysfs */
        .set_fan_control_mode    = smu_set_fan_control_mode,

diff --git a/drivers/gpu/drm/amd/pm/swsmu/inc/amdgpu_smu.h b/drivers/gpu/drm/amd/pm/swsmu/inc/amdgpu_smu.h
index 5356b91c62927af4f598296fa52afe620e56fa2c..e17169f681e87ef39d1f6527980ac2dcc91d5ba6 100644 (file)
--- a/drivers/gpu/drm/amd/pm/swsmu/inc/amdgpu_smu.h
+++ b/drivers/gpu/drm/amd/pm/swsmu/inc/amdgpu_smu.h
@@ -840,6 +840,12 @@ struct pptable_funcs {
         */
        int (*allow_xgmi_power_down)(struct smu_context *smu, bool en);
 
+       /**
+        * @select_xgmi_plpd_policy: Select xgmi per-link power down policy.
+        */
+       int (*select_xgmi_plpd_policy)(struct smu_context *smu,
+                                      enum pp_xgmi_plpd_mode mode);
+
        /**
         * @update_pcie_parameters: Update and upload the system's PCIe
         *                          capabilites to the SMU.
@@ -1487,6 +1493,9 @@ int smu_set_ac_dc(struct smu_context *smu);
 
 int smu_allow_xgmi_power_down(struct smu_context *smu, bool en);
 
+int smu_set_xgmi_plpd_mode(struct smu_context *smu,
+                          enum pp_xgmi_plpd_mode mode);
+
 int smu_get_entrycount_gfxoff(struct smu_context *smu, u64 *value);
 
 int smu_get_residency_gfxoff(struct smu_context *smu, u32 *value);

diff --git a/drivers/gpu/drm/amd/pm/swsmu/inc/smu_types.h b/drivers/gpu/drm/amd/pm/swsmu/inc/smu_types.h
index 7c300b4d95c79db9fdbc2eb48a6b6f5aa3c6aef2..4850e48bbef5c418abc74e7e4947bab648283a0a 100644 (file)
--- a/drivers/gpu/drm/amd/pm/swsmu/inc/smu_types.h
+++ b/drivers/gpu/drm/amd/pm/swsmu/inc/smu_types.h
@@ -252,7 +252,8 @@
        __SMU_DUMMY_MAP(QueryValidMcaCount),    \
        __SMU_DUMMY_MAP(QueryValidMcaCeCount),  \
        __SMU_DUMMY_MAP(McaBankDumpDW),         \
-       __SMU_DUMMY_MAP(McaBankCeDumpDW),
+       __SMU_DUMMY_MAP(McaBankCeDumpDW),       \
+       __SMU_DUMMY_MAP(SelectPLPDMode),
 
 #undef __SMU_DUMMY_MAP
 #define __SMU_DUMMY_MAP(type)  SMU_MSG_##type

diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c
index 11a6cd96c601200e9396653511de915fa718f4f5..b137c37903fc927f3258658f4b7adf0beacf5e06 100644 (file)
--- a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c
@@ -170,6 +170,7 @@ static const struct cmn2asic_msg_mapping smu_v13_0_6_message_map[SMU_MSG_MAX_COU
        MSG_MAP(QueryValidMcaCeCount,                PPSMC_MSG_QueryValidMcaCeCount,            0),
        MSG_MAP(McaBankDumpDW,                       PPSMC_MSG_McaBankDumpDW,                   0),
        MSG_MAP(McaBankCeDumpDW,                     PPSMC_MSG_McaBankCeDumpDW,                 0),
+       MSG_MAP(SelectPLPDMode,                      PPSMC_MSG_SelectPLPDMode,                  0),
 };
 
 static const struct cmn2asic_mapping smu_v13_0_6_clk_map[SMU_CLK_COUNT] = {
@@ -2716,6 +2717,35 @@ static const struct amdgpu_mca_smu_funcs smu_v13_0_6_mca_smu_funcs = {
        .mca_get_ras_mca_idx_array = mca_smu_get_ras_mca_idx_array,
 };
 
+static int smu_v13_0_6_select_xgmi_plpd_policy(struct smu_context *smu,
+                                              enum pp_xgmi_plpd_mode mode)
+{
+       struct amdgpu_device *adev = smu->adev;
+       int ret, param;
+
+       switch (mode) {
+       case XGMI_PLPD_DEFAULT:
+               param = PPSMC_PLPD_MODE_DEFAULT;
+               break;
+       case XGMI_PLPD_OPTIMIZED:
+               param = PPSMC_PLPD_MODE_OPTIMIZED;
+               break;
+       default:
+               return -EINVAL;
+       }
+
+       /* change xgmi per-link power down policy */
+       ret = smu_cmn_send_smc_msg_with_param(
+               smu, SMU_MSG_SelectPLPDMode, param, NULL);
+
+       if (ret)
+               dev_err(adev->dev,
+                       "select xgmi per-link power down policy %d failed\n",
+                       mode);
+
+       return ret;
+}
+
 static const struct pptable_funcs smu_v13_0_6_ppt_funcs = {
        /* init dpm */
        .get_allowed_feature_mask = smu_v13_0_6_get_allowed_feature_mask,
@@ -2756,6 +2786,7 @@ static const struct pptable_funcs smu_v13_0_6_ppt_funcs = {
        .od_edit_dpm_table = smu_v13_0_6_usr_edit_dpm_table,
        .set_df_cstate = smu_v13_0_6_set_df_cstate,
        .allow_xgmi_power_down = smu_v13_0_6_allow_xgmi_power_down,
+       .select_xgmi_plpd_policy = smu_v13_0_6_select_xgmi_plpd_policy,
        .log_thermal_throttling_event = smu_v13_0_6_log_thermal_throttling_event,
        .get_pp_feature_mask = smu_cmn_get_pp_feature_mask,
        .get_gpu_metrics = smu_v13_0_6_get_gpu_metrics,