Add SMU13 fan temperature/pwm curve OD setting support.

Signed-off-by: Evan Quan <evan.q...@amd.com>
---
 .../gpu/drm/amd/include/kgd_pp_interface.h    |  2 +
 drivers/gpu/drm/amd/pm/amdgpu_pm.c            | 45 ++++++++++
 drivers/gpu/drm/amd/pm/inc/amdgpu_dpm.h       |  2 +
 drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c     |  2 +
 drivers/gpu/drm/amd/pm/swsmu/inc/smu_types.h  |  1 +
 .../drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c  | 87 ++++++++++++++++++-
 .../drm/amd/pm/swsmu/smu13/smu_v13_0_7_ppt.c  | 87 ++++++++++++++++++-
 7 files changed, 224 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/include/kgd_pp_interface.h 
b/drivers/gpu/drm/amd/include/kgd_pp_interface.h
index d51c639a3f31..3743777b45cb 100644
--- a/drivers/gpu/drm/amd/include/kgd_pp_interface.h
+++ b/drivers/gpu/drm/amd/include/kgd_pp_interface.h
@@ -114,6 +114,7 @@ enum pp_clock_type {
        OD_VDDGFX_OFFSET,
        OD_CCLK,
        OD_FAN_MODE,
+       OD_FAN_CURVE,
 };
 
 enum amd_pp_sensors {
@@ -188,6 +189,7 @@ enum PP_OD_DPM_TABLE_COMMAND {
        PP_OD_COMMIT_DPM_TABLE,
        PP_OD_EDIT_VDDGFX_OFFSET,
        PP_OD_EDIT_FAN_MODE,
+       PP_OD_EDIT_FAN_CURVE,
 };
 
 struct pp_states_info {
diff --git a/drivers/gpu/drm/amd/pm/amdgpu_pm.c 
b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
index 90c5f3c95307..7baebe45b912 100644
--- a/drivers/gpu/drm/amd/pm/amdgpu_pm.c
+++ b/drivers/gpu/drm/amd/pm/amdgpu_pm.c
@@ -3684,6 +3684,43 @@ static umode_t fan_mode_visible(struct amdgpu_device 
*adev)
        return umode;
 }
 
+static ssize_t fan_curve_show(struct kobject *kobj,
+                             struct kobj_attribute *attr,
+                             char *buf)
+{
+       struct od_kobj *container = container_of(kobj, struct od_kobj, kobj);
+       struct amdgpu_device *adev = (struct amdgpu_device *)container->priv;
+
+       return (ssize_t)amdgpu_retrieve_od_settings(adev, OD_FAN_CURVE, buf);
+}
+
+static ssize_t fan_curve_store(struct kobject *kobj,
+                              struct kobj_attribute *attr,
+                              const char *buf,
+                              size_t count)
+{
+       struct od_kobj *container = container_of(kobj, struct od_kobj, kobj);
+       struct amdgpu_device *adev = (struct amdgpu_device *)container->priv;
+
+       return (ssize_t)amdgpu_distribute_custom_od_settings(adev,
+                                                            
PP_OD_EDIT_FAN_CURVE,
+                                                            buf,
+                                                            count);
+}
+
+static umode_t fan_curve_visible(struct amdgpu_device *adev)
+{
+       umode_t umode = 0000;
+
+       if (adev->pm.od_feature_mask & OD_OPS_SUPPORT_FAN_CURVE_RETRIEVE)
+               umode |= S_IRUSR | S_IRGRP | S_IROTH;
+
+       if (adev->pm.od_feature_mask & OD_OPS_SUPPORT_FAN_CURVE_SET)
+               umode |= S_IWUSR;
+
+       return umode;
+}
+
 static struct od_feature_set amdgpu_od_set = {
        .containers = {
                [0] = {
@@ -3697,6 +3734,14 @@ static struct od_feature_set amdgpu_od_set = {
                                                .store = fan_mode_store,
                                        },
                                },
+                               [1] = {
+                                       .name = "fan_curve",
+                                       .ops = {
+                                               .is_visible = fan_curve_visible,
+                                               .show = fan_curve_show,
+                                               .store = fan_curve_store,
+                                       },
+                               },
                        },
                },
        },
diff --git a/drivers/gpu/drm/amd/pm/inc/amdgpu_dpm.h 
b/drivers/gpu/drm/amd/pm/inc/amdgpu_dpm.h
index b54f84e2408a..4463c60b710a 100644
--- a/drivers/gpu/drm/amd/pm/inc/amdgpu_dpm.h
+++ b/drivers/gpu/drm/amd/pm/inc/amdgpu_dpm.h
@@ -316,6 +316,8 @@ struct config_table_setting
 
 #define OD_OPS_SUPPORT_FAN_MODE_RETRIEVE               BIT(0)
 #define OD_OPS_SUPPORT_FAN_MODE_SET                    BIT(1)
+#define OD_OPS_SUPPORT_FAN_CURVE_RETRIEVE              BIT(2)
+#define OD_OPS_SUPPORT_FAN_CURVE_SET                   BIT(3)
 
 struct amdgpu_pm {
        struct mutex            mutex;
diff --git a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c 
b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
index 01ccfd219d6b..fe998df9dca2 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
@@ -2483,6 +2483,8 @@ static enum smu_clk_type smu_convert_to_smuclk(enum 
pp_clock_type type)
                clk_type = SMU_OD_CCLK; break;
        case OD_FAN_MODE:
                clk_type = SMU_OD_FAN_MODE; break;
+       case OD_FAN_CURVE:
+               clk_type = SMU_OD_FAN_CURVE; break;
        default:
                clk_type = SMU_CLK_COUNT; break;
        }
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 f37de1df8291..dc5f6e166907 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/inc/smu_types.h
+++ b/drivers/gpu/drm/amd/pm/swsmu/inc/smu_types.h
@@ -279,6 +279,7 @@ enum smu_clk_type {
        SMU_OD_RANGE,
        SMU_OD_VDDGFX_OFFSET,
        SMU_OD_FAN_MODE,
+       SMU_OD_FAN_CURVE,
        SMU_CLK_COUNT,
 };
 
diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c 
b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c
index 19406fa001d5..426652af4fc5 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c
@@ -102,6 +102,8 @@
 #define PP_OD_FEATURE_UCLK_FMAX                                3
 #define PP_OD_FEATURE_GFX_VF_CURVE                     4
 #define PP_OD_FEATURE_FAN_MODE                         5
+#define PP_OD_FEATURE_FAN_CURVE_TEMP                   6
+#define PP_OD_FEATURE_FAN_CURVE_PWM                    7
 
 static struct cmn2asic_msg_mapping smu_v13_0_0_message_map[SMU_MSG_MAX_COUNT] 
= {
        MSG_MAP(TestMessage,                    PPSMC_MSG_TestMessage,          
       1),
@@ -1088,6 +1090,14 @@ static void smu_v13_0_0_get_od_setting_limits(struct 
smu_context *smu,
                od_min_setting = overdrive_lowerlimits->FanMode;
                od_max_setting = overdrive_upperlimits->FanMode;
                break;
+       case PP_OD_FEATURE_FAN_CURVE_TEMP:
+               od_min_setting = overdrive_lowerlimits->FanLinearTempPoints;
+               od_max_setting = overdrive_upperlimits->FanLinearTempPoints;
+               break;
+       case PP_OD_FEATURE_FAN_CURVE_PWM:
+               od_min_setting = overdrive_lowerlimits->FanLinearPwmPoints;
+               od_max_setting = overdrive_upperlimits->FanLinearPwmPoints;
+               break;
        default:
                od_min_setting = od_max_setting = INT_MAX;
                break;
@@ -1315,6 +1325,35 @@ static int smu_v13_0_0_print_clk_levels(struct 
smu_context *smu,
                                        (int)od_table->OverDriveTable.FanMode);
                break;
 
+       case SMU_OD_FAN_CURVE:
+               if (!smu_v13_0_0_is_od_feature_supported(smu,
+                                                        
PP_OD_FEATURE_FAN_CURVE_BIT))
+                       break;
+
+               size += sysfs_emit_at(buf, size, "OD_FAN_CURVE:\n");
+               for (i = 0; i < NUM_OD_FAN_MAX_POINTS - 1; i++)
+                       size += sysfs_emit_at(buf, size, "%d: %dC %d%%\n",
+                                               i,
+                                               
(int)od_table->OverDriveTable.FanLinearTempPoints[i],
+                                               
(int)od_table->OverDriveTable.FanLinearPwmPoints[i]);
+
+               size += sysfs_emit_at(buf, size, "%s:\n", "OD_RANGE");
+               smu_v13_0_0_get_od_setting_limits(smu,
+                                                 PP_OD_FEATURE_FAN_CURVE_TEMP,
+                                                 &min_value,
+                                                 &max_value);
+               size += sysfs_emit_at(buf, size, "FAN_CURVE(hotspot temp): %uC 
%uC\n",
+                                     min_value, max_value);
+
+               smu_v13_0_0_get_od_setting_limits(smu,
+                                                 PP_OD_FEATURE_FAN_CURVE_PWM,
+                                                 &min_value,
+                                                 &max_value);
+               size += sysfs_emit_at(buf, size, "FAN_CURVE(fan speed): %u%% 
%u%%\n",
+                                     min_value, max_value);
+
+               break;
+
        case SMU_OD_RANGE:
                if (!smu_v13_0_0_is_od_feature_supported(smu, 
PP_OD_FEATURE_GFXCLK_BIT) &&
                    !smu_v13_0_0_is_od_feature_supported(smu, 
PP_OD_FEATURE_UCLK_BIT) &&
@@ -1528,6 +1567,44 @@ static int smu_v13_0_0_od_edit_dpm_table(struct 
smu_context *smu,
                od_table->OverDriveTable.FeatureCtrlMask |= 1U << 
PP_OD_FEATURE_GFX_VF_CURVE_BIT;
                break;
 
+       case PP_OD_EDIT_FAN_CURVE:
+               if (!smu_v13_0_0_is_od_feature_supported(smu, 
PP_OD_FEATURE_FAN_CURVE_BIT)) {
+                       dev_warn(adev->dev, "Fan curve setting not 
supported!\n");
+                       return -ENOTSUPP;
+               }
+
+               if (input[0] >= NUM_OD_FAN_MAX_POINTS - 1 ||
+                   input[0] < 0)
+                       return -EINVAL;
+
+               smu_v13_0_0_get_od_setting_limits(smu,
+                                                 PP_OD_FEATURE_FAN_CURVE_TEMP,
+                                                 &minimum,
+                                                 &maximum);
+               if (input[1] < minimum ||
+                   input[1] > maximum) {
+                       dev_info(adev->dev, "Fan curve temp setting(%ld) must 
be within [%d, %d]!\n",
+                                input[1], minimum, maximum);
+                       return -EINVAL;
+               }
+
+               smu_v13_0_0_get_od_setting_limits(smu,
+                                                 PP_OD_FEATURE_FAN_CURVE_PWM,
+                                                 &minimum,
+                                                 &maximum);
+               if (input[2] < minimum ||
+                   input[2] > maximum) {
+                       dev_info(adev->dev, "Fan curve pwm setting(%ld) must be 
within [%d, %d]!\n",
+                                input[2], minimum, maximum);
+                       return -EINVAL;
+               }
+
+               od_table->OverDriveTable.FanLinearTempPoints[input[0]] = 
input[1];
+               od_table->OverDriveTable.FanLinearPwmPoints[input[0]] = 
input[2];
+               od_table->OverDriveTable.FanMode = FAN_MODE_MANUAL_LINEAR;
+               od_table->OverDriveTable.FeatureCtrlMask |= 
BIT(PP_OD_FEATURE_FAN_CURVE_BIT);
+               break;
+
        case PP_OD_RESTORE_DEFAULT_TABLE:
                feature_ctrlmask = od_table->OverDriveTable.FeatureCtrlMask;
                memcpy(od_table,
@@ -1781,7 +1858,9 @@ static void 
smu_v13_0_0_set_supported_od_feature_mask(struct smu_context *smu)
 
        if (smu_v13_0_0_is_od_feature_supported(smu,
                                                PP_OD_FEATURE_FAN_CURVE_BIT))
-               adev->pm.od_feature_mask |= OD_OPS_SUPPORT_FAN_MODE_RETRIEVE;
+               adev->pm.od_feature_mask |= OD_OPS_SUPPORT_FAN_MODE_RETRIEVE |
+                                           OD_OPS_SUPPORT_FAN_CURVE_RETRIEVE |
+                                           OD_OPS_SUPPORT_FAN_CURVE_SET;
 }
 
 static int smu_v13_0_0_set_default_od_settings(struct smu_context *smu)
@@ -1835,6 +1914,12 @@ static int smu_v13_0_0_set_default_od_settings(struct 
smu_context *smu)
                                
user_od_table_bak.OverDriveTable.VoltageOffsetPerZoneBoundary[i];
                user_od_table->OverDriveTable.FanMode =
                        user_od_table_bak.OverDriveTable.FanMode;
+               for (i = 0; i < NUM_OD_FAN_MAX_POINTS - 1; i++) {
+                       user_od_table->OverDriveTable.FanLinearTempPoints[i] =
+                               
user_od_table_bak.OverDriveTable.FanLinearTempPoints[i];
+                       user_od_table->OverDriveTable.FanLinearPwmPoints[i] =
+                               
user_od_table_bak.OverDriveTable.FanLinearPwmPoints[i];
+               }
        }
 
        smu_v13_0_0_set_supported_od_feature_mask(smu);
diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_7_ppt.c 
b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_7_ppt.c
index 7db8a1e2c640..72bbbd2569e9 100644
--- a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_7_ppt.c
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_7_ppt.c
@@ -78,6 +78,8 @@
 #define PP_OD_FEATURE_UCLK_FMAX                                3
 #define PP_OD_FEATURE_GFX_VF_CURVE                     4
 #define PP_OD_FEATURE_FAN_MODE                         5
+#define PP_OD_FEATURE_FAN_CURVE_TEMP                   6
+#define PP_OD_FEATURE_FAN_CURVE_PWM                    7
 
 static struct cmn2asic_msg_mapping smu_v13_0_7_message_map[SMU_MSG_MAX_COUNT] 
= {
        MSG_MAP(TestMessage,                    PPSMC_MSG_TestMessage,          
       1),
@@ -1075,6 +1077,14 @@ static void smu_v13_0_7_get_od_setting_limits(struct 
smu_context *smu,
                od_min_setting = overdrive_lowerlimits->FanMode;
                od_max_setting = overdrive_upperlimits->FanMode;
                break;
+       case PP_OD_FEATURE_FAN_CURVE_TEMP:
+               od_min_setting = overdrive_lowerlimits->FanLinearTempPoints;
+               od_max_setting = overdrive_upperlimits->FanLinearTempPoints;
+               break;
+       case PP_OD_FEATURE_FAN_CURVE_PWM:
+               od_min_setting = overdrive_lowerlimits->FanLinearPwmPoints;
+               od_max_setting = overdrive_upperlimits->FanLinearPwmPoints;
+               break;
        default:
                od_min_setting = od_max_setting = INT_MAX;
                break;
@@ -1302,6 +1312,35 @@ static int smu_v13_0_7_print_clk_levels(struct 
smu_context *smu,
                                        (int)od_table->OverDriveTable.FanMode);
                break;
 
+       case SMU_OD_FAN_CURVE:
+               if (!smu_v13_0_7_is_od_feature_supported(smu,
+                                                        
PP_OD_FEATURE_FAN_CURVE_BIT))
+                       break;
+
+               size += sysfs_emit_at(buf, size, "OD_FAN_CURVE:\n");
+               for (i = 0; i < NUM_OD_FAN_MAX_POINTS - 1; i++)
+                       size += sysfs_emit_at(buf, size, "%d: %dC %d%%\n",
+                                               i,
+                                               
(int)od_table->OverDriveTable.FanLinearTempPoints[i],
+                                               
(int)od_table->OverDriveTable.FanLinearPwmPoints[i]);
+
+               size += sysfs_emit_at(buf, size, "%s:\n", "OD_RANGE");
+               smu_v13_0_7_get_od_setting_limits(smu,
+                                                 PP_OD_FEATURE_FAN_CURVE_TEMP,
+                                                 &min_value,
+                                                 &max_value);
+               size += sysfs_emit_at(buf, size, "FAN_CURVE(hotspot temp): %uC 
%uC\n",
+                                     min_value, max_value);
+
+               smu_v13_0_7_get_od_setting_limits(smu,
+                                                 PP_OD_FEATURE_FAN_CURVE_PWM,
+                                                 &min_value,
+                                                 &max_value);
+               size += sysfs_emit_at(buf, size, "FAN_CURVE(fan speed): %u%% 
%u%%\n",
+                                     min_value, max_value);
+
+               break;
+
        case SMU_OD_RANGE:
                if (!smu_v13_0_7_is_od_feature_supported(smu, 
PP_OD_FEATURE_GFXCLK_BIT) &&
                    !smu_v13_0_7_is_od_feature_supported(smu, 
PP_OD_FEATURE_UCLK_BIT) &&
@@ -1515,6 +1554,44 @@ static int smu_v13_0_7_od_edit_dpm_table(struct 
smu_context *smu,
                od_table->OverDriveTable.FeatureCtrlMask |= 1U << 
PP_OD_FEATURE_GFX_VF_CURVE_BIT;
                break;
 
+       case PP_OD_EDIT_FAN_CURVE:
+               if (!smu_v13_0_7_is_od_feature_supported(smu, 
PP_OD_FEATURE_FAN_CURVE_BIT)) {
+                       dev_warn(adev->dev, "Fan curve setting not 
supported!\n");
+                       return -ENOTSUPP;
+               }
+
+               if (input[0] >= NUM_OD_FAN_MAX_POINTS - 1 ||
+                   input[0] < 0)
+                       return -EINVAL;
+
+               smu_v13_0_7_get_od_setting_limits(smu,
+                                                 PP_OD_FEATURE_FAN_CURVE_TEMP,
+                                                 &minimum,
+                                                 &maximum);
+               if (input[1] < minimum ||
+                   input[1] > maximum) {
+                       dev_info(adev->dev, "Fan curve temp setting(%ld) must 
be within [%d, %d]!\n",
+                                input[1], minimum, maximum);
+                       return -EINVAL;
+               }
+
+               smu_v13_0_7_get_od_setting_limits(smu,
+                                                 PP_OD_FEATURE_FAN_CURVE_PWM,
+                                                 &minimum,
+                                                 &maximum);
+               if (input[2] < minimum ||
+                   input[2] > maximum) {
+                       dev_info(adev->dev, "Fan curve pwm setting(%ld) must be 
within [%d, %d]!\n",
+                                input[2], minimum, maximum);
+                       return -EINVAL;
+               }
+
+               od_table->OverDriveTable.FanLinearTempPoints[input[0]] = 
input[1];
+               od_table->OverDriveTable.FanLinearPwmPoints[input[0]] = 
input[2];
+               od_table->OverDriveTable.FanMode = FAN_MODE_MANUAL_LINEAR;
+               od_table->OverDriveTable.FeatureCtrlMask |= 
BIT(PP_OD_FEATURE_FAN_CURVE_BIT);
+               break;
+
        case PP_OD_RESTORE_DEFAULT_TABLE:
                feature_ctrlmask = od_table->OverDriveTable.FeatureCtrlMask;
                memcpy(od_table,
@@ -1763,7 +1840,9 @@ static void 
smu_v13_0_7_set_supported_od_feature_mask(struct smu_context *smu)
 
        if (smu_v13_0_7_is_od_feature_supported(smu,
                                                PP_OD_FEATURE_FAN_CURVE_BIT))
-               adev->pm.od_feature_mask |= OD_OPS_SUPPORT_FAN_MODE_RETRIEVE;
+               adev->pm.od_feature_mask |= OD_OPS_SUPPORT_FAN_MODE_RETRIEVE |
+                                           OD_OPS_SUPPORT_FAN_CURVE_RETRIEVE |
+                                           OD_OPS_SUPPORT_FAN_CURVE_SET;
 }
 
 static int smu_v13_0_7_set_default_od_settings(struct smu_context *smu)
@@ -1817,6 +1896,12 @@ static int smu_v13_0_7_set_default_od_settings(struct 
smu_context *smu)
                                
user_od_table_bak.OverDriveTable.VoltageOffsetPerZoneBoundary[i];
                user_od_table->OverDriveTable.FanMode =
                        user_od_table_bak.OverDriveTable.FanMode;
+               for (i = 0; i < NUM_OD_FAN_MAX_POINTS - 1; i++) {
+                       user_od_table->OverDriveTable.FanLinearTempPoints[i] =
+                               
user_od_table_bak.OverDriveTable.FanLinearTempPoints[i];
+                       user_od_table->OverDriveTable.FanLinearPwmPoints[i] =
+                               
user_od_table_bak.OverDriveTable.FanLinearPwmPoints[i];
+               }
        }
 
        smu_v13_0_7_set_supported_od_feature_mask(smu);
-- 
2.34.1

Reply via email to