<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<p style="font-family:Arial;font-size:11pt;color:#0078D7;margin:5pt;" align="Left">
[AMD Official Use Only - Internal Distribution Only]<br>
</p>
<br>
<div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Acked-by: Alex Deucher <alexander.deucher@amd.com><br>
</div>
<div id="appendonsend"></div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<br>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="divRplyFwdMsg" dir="ltr"><font style="font-size:11pt" face="Calibri, sans-serif" color="#000000"><b>From:</b> amd-gfx <amd-gfx-bounces@lists.freedesktop.org> on behalf of Changfeng.Zhu <changfeng.zhu@amd.com><br>
<b>Sent:</b> Tuesday, December 8, 2020 9:06 PM<br>
<b>To:</b> amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org>; Huang, Ray <Ray.Huang@amd.com><br>
<b>Cc:</b> Zhu, Changfeng <Changfeng.Zhu@amd.com><br>
<b>Subject:</b> [PATCH] drm/amd/pm: update smu10.h WORKLOAD_PPLIB setting for raven</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt">
<div class="PlainText">From: changzhu <Changfeng.Zhu@amd.com><br>
<br>
From: Changfeng <Changfeng.Zhu@amd.com><br>
<br>
When using old WORKLOAD_PPLIB setting in smu10.h, there is problem that<br>
it can't be able to switch to mak gpu clk during compute workload.<br>
It needs to update WORKLOAD_PPLIB setting to fix this issue.<br>
<br>
Change-Id: Id2160a7b4a6cb8808d100de25e999714a7ccaebd<br>
Signed-off-by: Changfeng <Changfeng.Zhu@amd.com><br>
---<br>
 drivers/gpu/drm/amd/pm/inc/smu10.h                 | 14 ++++++--------<br>
 .../gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c   |  9 +++------<br>
 2 files changed, 9 insertions(+), 14 deletions(-)<br>
<br>
diff --git a/drivers/gpu/drm/amd/pm/inc/smu10.h b/drivers/gpu/drm/amd/pm/inc/smu10.h<br>
index b96520528240..9e837a5014c5 100644<br>
--- a/drivers/gpu/drm/amd/pm/inc/smu10.h<br>
+++ b/drivers/gpu/drm/amd/pm/inc/smu10.h<br>
@@ -136,14 +136,12 @@<br>
 #define FEATURE_CORE_CSTATES_MASK     (1 << FEATURE_CORE_CSTATES_BIT)<br>
 <br>
 /* Workload bits */<br>
-#define WORKLOAD_DEFAULT_BIT              0<br>
-#define WORKLOAD_PPLIB_FULL_SCREEN_3D_BIT 1<br>
-#define WORKLOAD_PPLIB_POWER_SAVING_BIT   2<br>
-#define WORKLOAD_PPLIB_VIDEO_BIT          3<br>
-#define WORKLOAD_PPLIB_VR_BIT             4<br>
-#define WORKLOAD_PPLIB_COMPUTE_BIT        5<br>
-#define WORKLOAD_PPLIB_CUSTOM_BIT         6<br>
-#define WORKLOAD_PPLIB_COUNT              7<br>
+#define WORKLOAD_PPLIB_FULL_SCREEN_3D_BIT 0<br>
+#define WORKLOAD_PPLIB_VIDEO_BIT          2<br>
+#define WORKLOAD_PPLIB_VR_BIT             3<br>
+#define WORKLOAD_PPLIB_COMPUTE_BIT        4<br>
+#define WORKLOAD_PPLIB_CUSTOM_BIT         5<br>
+#define WORKLOAD_PPLIB_COUNT              6<br>
 <br>
 typedef struct {<br>
         /* MP1_EXT_SCRATCH0 */<br>
diff --git a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c<br>
index 04226b1544e4..e57e64bbacdc 100644<br>
--- a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c<br>
+++ b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu10_hwmgr.c<br>
@@ -1298,15 +1298,9 @@ static int conv_power_profile_to_pplib_workload(int power_profile)<br>
         int pplib_workload = 0;<br>
 <br>
         switch (power_profile) {<br>
-       case PP_SMC_POWER_PROFILE_BOOTUP_DEFAULT:<br>
-               pplib_workload = WORKLOAD_DEFAULT_BIT;<br>
-               break;<br>
         case PP_SMC_POWER_PROFILE_FULLSCREEN3D:<br>
                 pplib_workload = WORKLOAD_PPLIB_FULL_SCREEN_3D_BIT;<br>
                 break;<br>
-       case PP_SMC_POWER_PROFILE_POWERSAVING:<br>
-               pplib_workload = WORKLOAD_PPLIB_POWER_SAVING_BIT;<br>
-               break;<br>
         case PP_SMC_POWER_PROFILE_VIDEO:<br>
                 pplib_workload = WORKLOAD_PPLIB_VIDEO_BIT;<br>
                 break;<br>
@@ -1316,6 +1310,9 @@ static int conv_power_profile_to_pplib_workload(int power_profile)<br>
         case PP_SMC_POWER_PROFILE_COMPUTE:<br>
                 pplib_workload = WORKLOAD_PPLIB_COMPUTE_BIT;<br>
                 break;<br>
+       case PP_SMC_POWER_PROFILE_CUSTOM:<br>
+               pplib_workload = WORKLOAD_PPLIB_CUSTOM_BIT;<br>
+               break;<br>
         }<br>
 <br>
         return pplib_workload;<br>
-- <br>
2.17.1<br>
<br>
_______________________________________________<br>
amd-gfx mailing list<br>
amd-gfx@lists.freedesktop.org<br>
<a href="https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&amp;data=04%7C01%7Calexander.deucher%40amd.com%7C2f57b9a4012a424d574908d89be7299b%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637430764514589029%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=OWntFjcijTjJa0Qrsi7YTvrEWQcIXM8dHsXxhOaKsng%3D&amp;reserved=0">https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&amp;data=04%7C01%7Calexander.deucher%40amd.com%7C2f57b9a4012a424d574908d89be7299b%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637430764514589029%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=OWntFjcijTjJa0Qrsi7YTvrEWQcIXM8dHsXxhOaKsng%3D&amp;reserved=0</a><br>
</div>
</span></font></div>
</div>
</body>
</html>