<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">
<div id="appendonsend" style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
this patch, i think the smu driver should be unify the driver code format,</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
it is very useful for optimize and maintain smu driver code in the furture.</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<br>
</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
you can reference the navi10_ppt.c and arcuturs_ppt.c<br>
funciton: arcturus_get_workload_type<br>
</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
<br>
</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
Best Regards,</div>
<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">
Kevin</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> amd-gfx <amd-gfx-bounces@lists.freedesktop.org> on behalf of Liang, Prike <Prike.Liang@amd.com><br>
<b>Sent:</b> Thursday, September 26, 2019 11:50 AM<br>
<b>To:</b> amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org><br>
<b>Cc:</b> Liang, Prike <Prike.Liang@amd.com>; Quan, Evan <Evan.Quan@amd.com>; Huang, Ray <Ray.Huang@amd.com>; keneth.feng@amd.com <keneth.feng@amd.com><br>
<b>Subject:</b> [PATCH 4/8] drm/amd/powerplay: add interface for getting workload type</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt">
<div class="PlainText">The workload type was got from the input of power profile mode.<br>
<br>
Signed-off-by: Prike Liang <Prike.Liang@amd.com><br>
Reviewed-by: Kevin Wang <kevin1.wang@amd.com><br>
Reviewed-by: Evan Quan <evan.quan@amd.com><br>
---<br>
drivers/gpu/drm/amd/powerplay/renoir_ppt.c | 29 +++++++++++++++++++++++++++++<br>
1 file changed, 29 insertions(+)<br>
<br>
diff --git a/drivers/gpu/drm/amd/powerplay/renoir_ppt.c b/drivers/gpu/drm/amd/powerplay/renoir_ppt.c<br>
index 8ec3663..dc945b8 100644<br>
--- a/drivers/gpu/drm/amd/powerplay/renoir_ppt.c<br>
+++ b/drivers/gpu/drm/amd/powerplay/renoir_ppt.c<br>
@@ -365,6 +365,34 @@ static int renoir_unforce_dpm_levels(struct smu_context *smu) {<br>
return ret;<br>
}<br>
<br>
+static int renoir_get_workload_type(struct smu_context *smu, uint32_t profile)<br>
+{<br>
+<br>
+ uint32_t pplib_workload = 0;<br>
+<br>
+ switch (profile) {<br>
+ case PP_SMC_POWER_PROFILE_FULLSCREEN3D:<br>
+ pplib_workload = WORKLOAD_PPLIB_FULL_SCREEN_3D_BIT;<br>
+ break;<br>
+ case PP_SMC_POWER_PROFILE_CUSTOM:<br>
+ pplib_workload = WORKLOAD_PPLIB_COUNT;<br>
+ break;<br>
+ case PP_SMC_POWER_PROFILE_VIDEO:<br>
+ pplib_workload = WORKLOAD_PPLIB_VIDEO_BIT;<br>
+ break;<br>
+ case PP_SMC_POWER_PROFILE_VR:<br>
+ pplib_workload = WORKLOAD_PPLIB_VR_BIT;<br>
+ break;<br>
+ case PP_SMC_POWER_PROFILE_COMPUTE:<br>
+ pplib_workload = WORKLOAD_PPLIB_COMPUTE_BIT;<br>
+ break;<br>
+ default:<br>
+ return -EINVAL;<br>
+ }<br>
+<br>
+ return pplib_workload;<br>
+}<br>
+<br>
static const struct pptable_funcs renoir_ppt_funcs = {<br>
.get_smu_msg_index = renoir_get_smu_msg_index,<br>
.get_smu_table_index = renoir_get_smu_table_index,<br>
@@ -376,6 +404,7 @@ static const struct pptable_funcs renoir_ppt_funcs = {<br>
.dpm_set_uvd_enable = renoir_dpm_set_uvd_enable,<br>
.force_dpm_limit_value = renoir_force_dpm_limit_value,<br>
.unforce_dpm_levels = renoir_unforce_dpm_levels,<br>
+ .get_workload_type = renoir_get_workload_type,<br>
};<br>
<br>
void renoir_set_ppt_funcs(struct smu_context *smu)<br>
-- <br>
2.7.4<br>
<br>
_______________________________________________<br>
amd-gfx mailing list<br>
amd-gfx@lists.freedesktop.org<br>
<a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a></div>
</span></font></div>
</body>
</html>