[PATCH 1/5] drm/amdgpu: send msg to IMU for the front-door loading

Du, Xiaojian Xiaojian.Du at amd.com
Thu Jul 28 08:38:18 UTC 2022


Of course, I will move it under "smu->is_apu". 

Thanks,
Xiaojian

>-----Original Message-----
>From: Quan, Evan <Evan.Quan at amd.com> 
>Sent: 2022年7月28日 16:14
>To: Du, Xiaojian <Xiaojian.Du at amd.com>; amd-gfx at lists.freedesktop.org
>Cc: Deucher, Alexander <Alexander.Deucher at amd.com>; Huang, Tim <Tim.Huang at amd.com>; Du, Xiaojian <Xiaojian.Du at amd.com>; Zhang, Yifan <Yifan1.Zhang at amd.com>
>Subject: RE: [PATCH 1/5] drm/amdgpu: send msg to IMU for the front-door loading
>
>
>
>
>
>> -----Original Message-----
>> From: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> On Behalf Of 
>> Xiaojian Du
>> Sent: Thursday, July 28, 2022 3:04 PM
>> To: amd-gfx at lists.freedesktop.org
>> Cc: Deucher, Alexander <Alexander.Deucher at amd.com>; Huang, Tim 
>> <Tim.Huang at amd.com>; Du, Xiaojian <Xiaojian.Du at amd.com>; Zhang, Yifan 
>> <Yifan1.Zhang at amd.com>
>> Subject: [PATCH 1/5] drm/amdgpu: send msg to IMU for the front-door 
>> loading
>> 
>> This patch will make SMU send msg to IMU for the front-door loading, 
>> it is required by some ASICs.
>> 
>> Signed-off-by: Yifan Zhang <yifan1.zhang at amd.com>
>> Signed-off-by: Xiaojian Du <Xiaojian.Du at amd.com>
>> ---
>>  drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c | 8 ++++++++
>>  1 file changed, 8 insertions(+)
>> 
>> diff --git a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
>> b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
>> index 6d9b3c6af164..79c01fa4b875 100644
>> --- a/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
>> +++ b/drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c
>> @@ -1360,6 +1360,14 @@ static int smu_hw_init(void *handle)
>>  		return ret;
>>  	}
>> 
>> +	if (smu->ppt_funcs->set_gfx_power_up_by_imu) {
>> +		ret = smu->ppt_funcs->set_gfx_power_up_by_imu(smu);
>> +		if (ret) {
>> +			dev_err(adev->dev, "Failed to Enable gfx imu!\n");
>> +			return ret;
>> +		}
>> +	}
>[Quan, Evan] Per my understandings, this should be needed by APU only. Can you move this under "smu->is_apu" control as other features below?
>
>Evan
>> +
>>  	if (smu->is_apu) {
>>  		smu_dpm_set_vcn_enable(smu, true);
>>  		smu_dpm_set_jpeg_enable(smu, true);
>> --
>> 2.25.1
>


More information about the amd-gfx mailing list