<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
</head>
<body>
<p style="font-family:Arial;font-size:10pt;color:#0000FF;margin:5pt;font-style:normal;font-weight:normal;text-decoration:none;" align="Left">
[AMD Official Use Only - General]<br>
</p>
<br>
<div>
<div dir="ltr">
<div></div>
<div>
<div dir="ltr">Reviewed-by: Yang Wang <kevinyang.wang@amd.com></div>
<div dir="ltr"><br>
</div>
<div dir="ltr">Best Regards,</div>
<div dir="ltr">Kevin</div>
<div id="mail-editor-reference-message-container" class="ms-outlook-mobile-reference-message">
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif"><b>From:</b> amd-gfx <amd-gfx-bounces@lists.freedesktop.org> on behalf of Alex Deucher <alexander.deucher@amd.com><br>
<b>Sent:</b> Thursday, November 2, 2023 03:50<br>
<b>To:</b> amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org><br>
<b>Cc:</b> Deucher, Alexander <Alexander.Deucher@amd.com><br>
<b>Subject:</b> [PATCH] drm/amdgpu/smu13: drop compute workload workaround
<div> </div>
</font></div>
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><font size="2"><span style="font-size:11pt;">
<div class="PlainText">This was fixed in PMFW before launch and is no longer<br>
required.<br>
<br>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com><br>
---<br>
.../drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c | 32 ++-----------------<br>
1 file changed, 2 insertions(+), 30 deletions(-)<br>
<br>
diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c<br>
index ef0c30075658..82c4e1f1c6f0 100644<br>
--- a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c<br>
+++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c<br>
@@ -2530,38 +2530,10 @@ static int smu_v13_0_0_set_power_profile_mode(struct smu_context *smu,<br>
}<br>
}<br>
<br>
- if (smu->power_profile_mode == PP_SMC_POWER_PROFILE_COMPUTE &&<br>
- (((smu->adev->pdev->device == 0x744C) && (smu->adev->pdev->revision == 0xC8)) ||<br>
- ((smu->adev->pdev->device == 0x744C) && (smu->adev->pdev->revision == 0xCC)))) {<br>
- ret = smu_cmn_update_table(smu,<br>
- SMU_TABLE_ACTIVITY_MONITOR_COEFF,<br>
- WORKLOAD_PPLIB_COMPUTE_BIT,<br>
- (void *)(&activity_monitor_external),<br>
- false);<br>
- if (ret) {<br>
- dev_err(smu->adev->dev, "[%s] Failed to get activity monitor!", __func__);<br>
- return ret;<br>
- }<br>
-<br>
- ret = smu_cmn_update_table(smu,<br>
- SMU_TABLE_ACTIVITY_MONITOR_COEFF,<br>
- WORKLOAD_PPLIB_CUSTOM_BIT,<br>
- (void *)(&activity_monitor_external),<br>
- true);<br>
- if (ret) {<br>
- dev_err(smu->adev->dev, "[%s] Failed to set activity monitor!", __func__);<br>
- return ret;<br>
- }<br>
-<br>
- workload_type = smu_cmn_to_asic_specific_index(smu,<br>
- CMN2ASIC_MAPPING_WORKLOAD,<br>
- PP_SMC_POWER_PROFILE_CUSTOM);<br>
- } else {<br>
- /* conv PP_SMC_POWER_PROFILE* to WORKLOAD_PPLIB_*_BIT */<br>
- workload_type = smu_cmn_to_asic_specific_index(smu,<br>
+ /* conv PP_SMC_POWER_PROFILE* to WORKLOAD_PPLIB_*_BIT */<br>
+ workload_type = smu_cmn_to_asic_specific_index(smu,<br>
CMN2ASIC_MAPPING_WORKLOAD,<br>
smu->power_profile_mode);<br>
- }<br>
<br>
if (workload_type < 0)<br>
return -EINVAL;<br>
-- <br>
2.41.0<br>
<br>
</div>
</span></font></div>
</div>
</div>
</div>
</body>
</html>