drm/amd/display: Fix type of pp_smu_wm_set_range struct

[why]
Value read from SMU is 16 bits, not 32.

[How]
Fix type, and add wm_type enum in preparation for future interfaces.

Signed-off-by: Eric Yang <Eric.Yang2@amd.com>
Reviewed-by: Tony Cheng <Tony.Cheng@amd.com>
Acked-by: Leo Li <sunpeng.li@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

diff --git a/drivers/gpu/drm/amd/display/dc/dm_pp_smu.h b/drivers/gpu/drm/amd/display/dc/dm_pp_smu.h
index 9f7ebf6..471f3df 100644 (file)
--- a/drivers/gpu/drm/amd/display/dc/dm_pp_smu.h
+++ b/drivers/gpu/drm/amd/display/dc/dm_pp_smu.h
@@ -41,6 +41,7 @@ enum pp_smu_ver {
         */
        PP_SMU_UNSUPPORTED,
        PP_SMU_VER_RV,
+
        PP_SMU_VER_MAX
 };
 
@@ -56,12 +57,31 @@ struct pp_smu {
        const void *dm;
 };
 
+enum pp_smu_status {
+       PP_SMU_RESULT_UNDEFINED = 0,
+       PP_SMU_RESULT_OK = 1,
+       PP_SMU_RESULT_FAIL,
+       PP_SMU_RESULT_UNSUPPORTED
+};
+
+
+#define PP_SMU_WM_SET_RANGE_CLK_UNCONSTRAINED_MIN 0x0
+#define PP_SMU_WM_SET_RANGE_CLK_UNCONSTRAINED_MAX 0xFFFF
+
+enum wm_type {
+       WM_TYPE_PSTATE_CHG = 0,
+       WM_TYPE_RETRAINING = 1,
+};
+
+/* This structure is a copy of WatermarkRowGeneric_t defined by smuxx_driver_if.h*/
 struct pp_smu_wm_set_range {
-       unsigned int wm_inst;
-       uint32_t min_fill_clk_mhz;
-       uint32_t max_fill_clk_mhz;
-       uint32_t min_drain_clk_mhz;
-       uint32_t max_drain_clk_mhz;
+       uint16_t min_fill_clk_mhz;
+       uint16_t max_fill_clk_mhz;
+       uint16_t min_drain_clk_mhz;
+       uint16_t max_drain_clk_mhz;
+
+       uint8_t wm_inst;
+       uint8_t wm_type;
 };
 
 #define MAX_WATERMARK_SETS 4
@@ -122,6 +142,7 @@ struct pp_smu_funcs {
        struct pp_smu ctx;
        union {
                struct pp_smu_funcs_rv rv_funcs;
+
        };
 };