<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-2022-jp">
</head>
<body>
Thanks for that confirmation Evan. I will drop this patch.<br>
<br>
Kent<br>
<br>
KENT RUSSELL<br>
Sr. Software Engineer | Linux Compute Kernel<br>
1 Commerce Valley Drive East<br>
Markham, ON L3T 7X6<br>
O +(1) 289-695-2122 | Ext 72122<br>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Quan, Evan<br>
<b>Sent:</b> Monday, March 25, 2019 10:26:54 AM<br>
<b>To:</b> Russell, Kent; amd-gfx@lists.freedesktop.org<br>
<b>Cc:</b> Russell, Kent<br>
<b>Subject:</b> RE: [PATCH 3/3] drm/amdgpu: Allow changing to BOOTUP profile on smu7</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">As I know, BOOTUP profile is only available for Vega10 and later ASICs.<br>
The BOOTUP profile for SMU7 has only some dummy settings and is not expected to switch to such mode.<br>
The default power profile for SMU7 is FULLSCREEN3D(different with Vega10 and later ASICs).<br>
hwmgr->workload_mask = 1 << hwmgr->workload_prority[PP_SMC_POWER_PROFILE_FULLSCREEN3D];<br>
hwmgr->power_profile_mode = PP_SMC_POWER_PROFILE_FULLSCREEN3D;<br>
hwmgr->default_power_profile_mode = PP_SMC_POWER_PROFILE_FULLSCREEN3D;<br>
<br>
Regards,<br>
Evan<br>
> -----Original Message-----<br>
> From: amd-gfx <amd-gfx-bounces@lists.freedesktop.org> On Behalf Of<br>
> Russell, Kent<br>
> Sent: 2019年3月25日 20:42<br>
> To: amd-gfx@lists.freedesktop.org<br>
> Cc: Russell, Kent <Kent.Russell@amd.com><br>
> Subject: [PATCH 3/3] drm/amdgpu: Allow changing to BOOTUP profile on<br>
> smu7<br>
> <br>
> With SMU7 using case statements, against vega10/20 only checking for the<br>
> CUSTOM profile, SMU7 can't set the power profile back to the<br>
> BOOTUP_DEFAULT, while newer HWMGRs can. Add the case statement in to<br>
> align functionality with other ASICs<br>
> <br>
> Change-Id: Ibc7df3b94b1a9dabcb88934e534c91209fc75967<br>
> Signed-off-by: Kent Russell <kent.russell@amd.com><br>
> ---<br>
> drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c | 1 +<br>
> 1 file changed, 1 insertion(+)<br>
> <br>
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c<br>
> b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c<br>
> index 74d55b8fb74e..7c3eae2a34b4 100644<br>
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c<br>
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c<br>
> @@ -5005,6 +5005,7 @@ static int smu7_set_power_profile_mode(struct<br>
> pp_hwmgr *hwmgr, long *input, uint<br>
> hwmgr->power_profile_mode = mode;<br>
> }<br>
> break;<br>
> + case PP_SMC_POWER_PROFILE_BOOTUP_DEFAULT:<br>
> case PP_SMC_POWER_PROFILE_FULLSCREEN3D:<br>
> case PP_SMC_POWER_PROFILE_POWERSAVING:<br>
> case PP_SMC_POWER_PROFILE_VIDEO:<br>
> --<br>
> 2.17.1<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><br>
</div>
</span></font></div>
</body>
</html>