[PATCH] drm/amd/pp: Clean register first to avoid read original value

[jinhuieric.huang@xxxxxxx (Eric Huang)]

On 03/30/2018 10:36 AM, Eric Huang wrote:
> It is not necessary to do that. The register will reset to 0 after 
> reading.
The register is not reset after reading. Actually after 
PPSMC_MSG_PmStatusLogSample sent, the register will be updated. So it is 
still not necessary to do that.

Eric
>
> Eric
>
>
> On 03/30/2018 03:33 AM, Rex Zhu wrote:
>> Signed-off-by: Rex Zhu <Rex.Zhu at amd.com>
>> ---
>> Â  drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c | 13 +++++++++++++
>> Â  1 file changed, 13 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c 
>> b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
>> index aaa9f5b..38cf3a1 100644
>> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
>> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
>> @@ -3368,6 +3368,19 @@ static int smu7_get_gpu_power(struct pp_hwmgr 
>> *hwmgr,
>> Â Â Â Â Â Â Â Â Â Â Â Â Â  "Failed to start pm status log!",
>> Â Â Â Â Â Â Â Â Â Â Â Â Â  return -1);
>> Â  +Â Â Â  cgs_write_ind_register(hwmgr->device,
>> +Â Â Â Â Â Â Â Â Â Â Â  CGS_IND_REG__SMC,
>> +Â Â Â Â Â Â Â Â Â Â Â  ixSMU_PM_STATUS_40, 0);
>> +Â Â Â  cgs_write_ind_register(hwmgr->device,
>> +Â Â Â Â Â Â Â Â Â Â Â  CGS_IND_REG__SMC,
>> +Â Â Â Â Â Â Â Â Â Â Â  ixSMU_PM_STATUS_49, 0);
>> +Â Â Â  cgs_write_ind_register(hwmgr->device,
>> +Â Â Â Â Â Â Â Â Â Â Â  CGS_IND_REG__SMC,
>> +Â Â Â Â Â Â Â Â Â Â Â  ixSMU_PM_STATUS_94, 0);
>> +Â Â Â  cgs_write_ind_register(hwmgr->device,
>> +Â Â Â Â Â Â Â Â Â Â Â  CGS_IND_REG__SMC,
>> +Â Â Â Â Â Â Â Â Â Â Â  ixSMU_PM_STATUS_95, 0);
>> +
>> Â Â Â Â Â  /* Sampling period from 50ms to 4sec */
>> Â Â Â Â Â  msleep_interruptible(200);
>