Commit b629167d authored by Kevin Wang's avatar Kevin Wang Committed by Alex Deucher

drm/amd/powerplay: remove redundancy debug log in smu

remove redundacy debug log in smu.
eg:
[ 6897.969447] amdgpu: [powerplay] smu 11 clk dpm feature 1 is not enabled
[ 6897.969448] amdgpu: [powerplay] smu 11 clk dpm feature 1 is not enabled
[ 6897.969448] amdgpu: [powerplay] smu 11 clk dpm feature 1 is not enabled
[ 6899.024114] amdgpu: [powerplay] Unsupported SMU message: 38
[ 6899.024151] amdgpu: [powerplay] smu 11 clk dpm feature 1 is not enabled
[ 6899.024151] amdgpu: [powerplay] smu 11 clk dpm feature 1 is not enabled
[ 6899.024152] amdgpu: [powerplay] smu 11 clk dpm feature 1 is not enabled
[ 6900.078296] amdgpu: [powerplay] Unsupported SMU message: 38
[ 6900.078332] amdgpu: [powerplay] smu 11 clk dpm feature 1 is not enabled
[ 6900.078332] amdgpu: [powerplay] smu 11 clk dpm feature 1 is not enabled
[ 6900.078333] amdgpu: [powerplay] smu 11 clk dpm feature 1 is not enabled
[ 6901.133230] amdgpu: [powerplay] Unsupported SMU message: 38
Signed-off-by: default avatarKevin Wang <kevin1.wang@amd.com>
Reviewed-by: default avatarKenneth Feng <kenneth.feng@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent ebf8fc31
...@@ -262,7 +262,6 @@ bool smu_clk_dpm_is_enabled(struct smu_context *smu, enum smu_clk_type clk_type) ...@@ -262,7 +262,6 @@ bool smu_clk_dpm_is_enabled(struct smu_context *smu, enum smu_clk_type clk_type)
} }
if(!smu_feature_is_enabled(smu, feature_id)) { if(!smu_feature_is_enabled(smu, feature_id)) {
pr_warn("smu %d clk dpm feature %d is not enabled\n", clk_type, feature_id);
return false; return false;
} }
......
...@@ -217,7 +217,6 @@ static int navi10_get_smu_msg_index(struct smu_context *smc, uint32_t index) ...@@ -217,7 +217,6 @@ static int navi10_get_smu_msg_index(struct smu_context *smc, uint32_t index)
mapping = navi10_message_map[index]; mapping = navi10_message_map[index];
if (!(mapping.valid_mapping)) { if (!(mapping.valid_mapping)) {
pr_warn("Unsupported SMU message: %d\n", index);
return -EINVAL; return -EINVAL;
} }
...@@ -233,7 +232,6 @@ static int navi10_get_smu_clk_index(struct smu_context *smc, uint32_t index) ...@@ -233,7 +232,6 @@ static int navi10_get_smu_clk_index(struct smu_context *smc, uint32_t index)
mapping = navi10_clk_map[index]; mapping = navi10_clk_map[index];
if (!(mapping.valid_mapping)) { if (!(mapping.valid_mapping)) {
pr_warn("Unsupported SMU clock: %d\n", index);
return -EINVAL; return -EINVAL;
} }
...@@ -249,7 +247,6 @@ static int navi10_get_smu_feature_index(struct smu_context *smc, uint32_t index) ...@@ -249,7 +247,6 @@ static int navi10_get_smu_feature_index(struct smu_context *smc, uint32_t index)
mapping = navi10_feature_mask_map[index]; mapping = navi10_feature_mask_map[index];
if (!(mapping.valid_mapping)) { if (!(mapping.valid_mapping)) {
pr_warn("Unsupported SMU feature: %d\n", index);
return -EINVAL; return -EINVAL;
} }
...@@ -265,7 +262,6 @@ static int navi10_get_smu_table_index(struct smu_context *smc, uint32_t index) ...@@ -265,7 +262,6 @@ static int navi10_get_smu_table_index(struct smu_context *smc, uint32_t index)
mapping = navi10_table_map[index]; mapping = navi10_table_map[index];
if (!(mapping.valid_mapping)) { if (!(mapping.valid_mapping)) {
pr_warn("Unsupported SMU table: %d\n", index);
return -EINVAL; return -EINVAL;
} }
...@@ -281,7 +277,6 @@ static int navi10_get_pwr_src_index(struct smu_context *smc, uint32_t index) ...@@ -281,7 +277,6 @@ static int navi10_get_pwr_src_index(struct smu_context *smc, uint32_t index)
mapping = navi10_pwr_src_map[index]; mapping = navi10_pwr_src_map[index];
if (!(mapping.valid_mapping)) { if (!(mapping.valid_mapping)) {
pr_warn("Unsupported power source: %d\n", index);
return -EINVAL; return -EINVAL;
} }
...@@ -298,7 +293,6 @@ static int navi10_get_workload_type(struct smu_context *smu, enum PP_SMC_POWER_P ...@@ -298,7 +293,6 @@ static int navi10_get_workload_type(struct smu_context *smu, enum PP_SMC_POWER_P
mapping = navi10_workload_map[profile]; mapping = navi10_workload_map[profile];
if (!(mapping.valid_mapping)) { if (!(mapping.valid_mapping)) {
pr_warn("Unsupported workload: %d\n", (int)profile);
return -EINVAL; return -EINVAL;
} }
......
...@@ -232,7 +232,6 @@ static int vega20_get_smu_table_index(struct smu_context *smc, uint32_t index) ...@@ -232,7 +232,6 @@ static int vega20_get_smu_table_index(struct smu_context *smc, uint32_t index)
mapping = vega20_table_map[index]; mapping = vega20_table_map[index];
if (!(mapping.valid_mapping)) { if (!(mapping.valid_mapping)) {
pr_warn("Unsupported SMU table: %d\n", index);
return -EINVAL; return -EINVAL;
} }
...@@ -248,7 +247,6 @@ static int vega20_get_pwr_src_index(struct smu_context *smc, uint32_t index) ...@@ -248,7 +247,6 @@ static int vega20_get_pwr_src_index(struct smu_context *smc, uint32_t index)
mapping = vega20_pwr_src_map[index]; mapping = vega20_pwr_src_map[index];
if (!(mapping.valid_mapping)) { if (!(mapping.valid_mapping)) {
pr_warn("Unsupported power source: %d\n", index);
return -EINVAL; return -EINVAL;
} }
...@@ -264,7 +262,6 @@ static int vega20_get_smu_feature_index(struct smu_context *smc, uint32_t index) ...@@ -264,7 +262,6 @@ static int vega20_get_smu_feature_index(struct smu_context *smc, uint32_t index)
mapping = vega20_feature_mask_map[index]; mapping = vega20_feature_mask_map[index];
if (!(mapping.valid_mapping)) { if (!(mapping.valid_mapping)) {
pr_warn("Unsupported SMU feature: %d\n", index);
return -EINVAL; return -EINVAL;
} }
...@@ -280,7 +277,6 @@ static int vega20_get_smu_clk_index(struct smu_context *smc, uint32_t index) ...@@ -280,7 +277,6 @@ static int vega20_get_smu_clk_index(struct smu_context *smc, uint32_t index)
mapping = vega20_clk_map[index]; mapping = vega20_clk_map[index];
if (!(mapping.valid_mapping)) { if (!(mapping.valid_mapping)) {
pr_warn("Unsupported SMU clock: %d\n", index);
return -EINVAL; return -EINVAL;
} }
...@@ -296,7 +292,6 @@ static int vega20_get_smu_msg_index(struct smu_context *smc, uint32_t index) ...@@ -296,7 +292,6 @@ static int vega20_get_smu_msg_index(struct smu_context *smc, uint32_t index)
mapping = vega20_message_map[index]; mapping = vega20_message_map[index];
if (!(mapping.valid_mapping)) { if (!(mapping.valid_mapping)) {
pr_warn("Unsupported SMU message: %d\n", index);
return -EINVAL; return -EINVAL;
} }
...@@ -312,7 +307,6 @@ static int vega20_get_workload_type(struct smu_context *smu, enum PP_SMC_POWER_P ...@@ -312,7 +307,6 @@ static int vega20_get_workload_type(struct smu_context *smu, enum PP_SMC_POWER_P
mapping = vega20_workload_map[profile]; mapping = vega20_workload_map[profile];
if (!(mapping.valid_mapping)) { if (!(mapping.valid_mapping)) {
pr_warn("Unsupported SMU workload: %d\n", (int)profile);
return -EINVAL; return -EINVAL;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment