<p>Attached patch will change the code logic, if adev->pm.dpm_enabled is false, si_set_temperature_range(...) will  be called, this is wrong obvious.<br><br>----</p>
<div id="cs2c_mail_sigature"></div>
<p> </p>
<div id="re" style="margin-left: 0.5em; padding-left: 0.5em; border-left: 1px solid green;"><br><br><br>
<div style="background-color: #f5f7fa;"><strong>主 题:</strong><span id="subject">RE: [PATCH v2] drm/amdgpu: resove reboot exception for si oland</span> <br><strong>日 期:</strong><span id="date">2023-03-14 09:04</span> <br><strong>发件人:</strong><span id="from">Chen, Guchun</span> <br><strong>收件人:</strong><span id="to" style="word-break: break-all;">李真能;</span></div>
<br>
<div id="content">
<div class="viewer_part" style="position: relative; overflow: hidden;">
<div>Will attached patch help?<br><br>Regards,<br>Guchun<br><br>> -----Original Message-----<br>> From: Zhenneng Li <br>> Sent: Monday, March 13, 2023 10:57 AM<br>> To: Chen, Guchun <br>> Cc: Deucher, Alexander ; Koenig, Christian<br>> ; Pan, Xinhui ; David<br>> Airlie ; Daniel Vetter ; amd-<br>> gfx@lists.freedesktop.org; Zhenneng Li <br>> Subject: [PATCH v2] drm/amdgpu: resove reboot exception for si oland<br>> <br>> During reboot test on arm64 platform, it may failure on boot.<br>> <br>> The error message are as follows:<br>> [ 6.996395][ 7] [ T295] [drm:amdgpu_device_ip_late_init [amdgpu]]<br>> *ERROR*<br>> late_init of IP block failed -22<br>> [ 7.006919][ 7] [ T295] amdgpu 0000:04:00.0: amdgpu_device_ip_late_init<br>> failed<br>> [ 7.014224][ 7] [ T295] amdgpu 0000:04:00.0: Fatal error during GPU init<br>> ---<br>> drivers/gpu/drm/amd/pm/legacy-dpm/si_dpm.c | 12 ------------<br>> 1 file changed, 12 deletions(-)<br>> <br>> diff --git a/drivers/gpu/drm/amd/pm/legacy-dpm/si_dpm.c<br>> b/drivers/gpu/drm/amd/pm/legacy-dpm/si_dpm.c<br>> index d6d9e3b1b2c0..ca9bce895dbe 100644<br>> --- a/drivers/gpu/drm/amd/pm/legacy-dpm/si_dpm.c<br>> +++ b/drivers/gpu/drm/amd/pm/legacy-dpm/si_dpm.c<br>> @@ -7626,18 +7626,6 @@ static int si_dpm_process_interrupt(struct<br>> amdgpu_device *adev,<br>> <br>> static int si_dpm_late_init(void *handle) {<br>> - int ret;<br>> - struct amdgpu_device *adev = (struct amdgpu_device *)handle;<br>> -<br>> - if (!adev->pm.dpm_enabled)<br>> - return 0;<br>> -<br>> - ret = si_set_temperature_range(adev);<br>> - if (ret)<br>> - return ret;<br>> -#if 0 //TODO ?<br>> - si_dpm_powergate_uvd(adev, true);<br>> -#endif<br>> return 0;<br>> }<br>> <br>> --<br>> 2.25.1<br><br></div>
</div>
</div>
</div>