Re: [PATCH] drm/amd/powerplay: issue no PPSMC_MSG_GetCurrPkgPwr on unsupported ASICs

2019-11-14 Thread Deucher, Alexander
Acked-by: Alex Deucher 

From: amd-gfx  on behalf of Evan Quan 

Sent: Thursday, November 14, 2019 2:33 AM
To: amd-gfx@lists.freedesktop.org 
Cc: Quan, Evan 
Subject: [PATCH] drm/amd/powerplay: issue no PPSMC_MSG_GetCurrPkgPwr on 
unsupported ASICs

Otherwise, the error message prompted will confuse user.

Change-Id: I44b9f870a8663714d715a1d5bf2aa24abe75bb8e
Signed-off-by: Evan Quan 
---
 .../gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c  | 23 +++
 1 file changed, 18 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c 
b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
index d3c3b3512a16..5c6b71b356e7 100644
--- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
+++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
@@ -3476,18 +3476,31 @@ static int smu7_get_pp_table_entry(struct pp_hwmgr 
*hwmgr,

 static int smu7_get_gpu_power(struct pp_hwmgr *hwmgr, u32 *query)
 {
+   struct amdgpu_device *adev = hwmgr->adev;
 int i;
 u32 tmp = 0;

 if (!query)
 return -EINVAL;

-   smum_send_msg_to_smc_with_parameter(hwmgr, PPSMC_MSG_GetCurrPkgPwr, 0);
-   tmp = cgs_read_register(hwmgr->device, mmSMC_MSG_ARG_0);
-   *query = tmp;
+   /*
+* PPSMC_MSG_GetCurrPkgPwr is not supported on:
+*  - Hawaii
+*  - Bonaire
+*  - Fiji
+*  - Tonga
+*/
+   if ((adev->asic_type != CHIP_HAWAII) &&
+   (adev->asic_type != CHIP_BONAIRE) &&
+   (adev->asic_type != CHIP_FIJI) &&
+   (adev->asic_type != CHIP_TONGA)) {
+   smum_send_msg_to_smc_with_parameter(hwmgr, 
PPSMC_MSG_GetCurrPkgPwr, 0);
+   tmp = cgs_read_register(hwmgr->device, mmSMC_MSG_ARG_0);
+   *query = tmp;

-   if (tmp != 0)
-   return 0;
+   if (tmp != 0)
+   return 0;
+   }

 smum_send_msg_to_smc(hwmgr, PPSMC_MSG_PmStatusLogStart);
 cgs_write_ind_register(hwmgr->device, CGS_IND_REG__SMC,
--
2.24.0

___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

[PATCH] drm/amd/powerplay: issue no PPSMC_MSG_GetCurrPkgPwr on unsupported ASICs

2019-11-13 Thread Evan Quan
Otherwise, the error message prompted will confuse user.

Change-Id: I44b9f870a8663714d715a1d5bf2aa24abe75bb8e
Signed-off-by: Evan Quan 
---
 .../gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c  | 23 +++
 1 file changed, 18 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c 
b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
index d3c3b3512a16..5c6b71b356e7 100644
--- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
+++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
@@ -3476,18 +3476,31 @@ static int smu7_get_pp_table_entry(struct pp_hwmgr 
*hwmgr,
 
 static int smu7_get_gpu_power(struct pp_hwmgr *hwmgr, u32 *query)
 {
+   struct amdgpu_device *adev = hwmgr->adev;
int i;
u32 tmp = 0;
 
if (!query)
return -EINVAL;
 
-   smum_send_msg_to_smc_with_parameter(hwmgr, PPSMC_MSG_GetCurrPkgPwr, 0);
-   tmp = cgs_read_register(hwmgr->device, mmSMC_MSG_ARG_0);
-   *query = tmp;
+   /*
+* PPSMC_MSG_GetCurrPkgPwr is not supported on:
+*  - Hawaii
+*  - Bonaire
+*  - Fiji
+*  - Tonga
+*/
+   if ((adev->asic_type != CHIP_HAWAII) &&
+   (adev->asic_type != CHIP_BONAIRE) &&
+   (adev->asic_type != CHIP_FIJI) &&
+   (adev->asic_type != CHIP_TONGA)) {
+   smum_send_msg_to_smc_with_parameter(hwmgr, 
PPSMC_MSG_GetCurrPkgPwr, 0);
+   tmp = cgs_read_register(hwmgr->device, mmSMC_MSG_ARG_0);
+   *query = tmp;
 
-   if (tmp != 0)
-   return 0;
+   if (tmp != 0)
+   return 0;
+   }
 
smum_send_msg_to_smc(hwmgr, PPSMC_MSG_PmStatusLogStart);
cgs_write_ind_register(hwmgr->device, CGS_IND_REG__SMC,
-- 
2.24.0

___
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx