<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Reviewed-by: Kevin Wang <kevin1.wang@amd.com></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,<br>
Kevin</div>
<div id="appendonsend"></div>
<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> amd-gfx <amd-gfx-bounces@lists.freedesktop.org> on behalf of Quan, Evan <Evan.Quan@amd.com><br>
<b>Sent:</b> Thursday, August 22, 2019 6:20 PM<br>
<b>To:</b> Quan, Evan <Evan.Quan@amd.com>; amd-gfx@lists.freedesktop.org <amd-gfx@lists.freedesktop.org><br>
<b>Cc:</b> Deucher, Alexander <Alexander.Deucher@amd.com><br>
<b>Subject:</b> RE: [PATCH 2/2] drm/amd/powerplay: correct the pp_feature output on Arcturus</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">Ping..<br>
<br>
> -----Original Message-----<br>
> From: Evan Quan <evan.quan@amd.com><br>
> Sent: Wednesday, August 21, 2019 4:42 PM<br>
> To: amd-gfx@lists.freedesktop.org<br>
> Cc: Quan, Evan <Evan.Quan@amd.com><br>
> Subject: [PATCH 2/2] drm/amd/powerplay: correct the pp_feature output on<br>
> Arcturus<br>
> <br>
> Fix for the commit below:<br>
> drm/amd/powerplay: implment sysfs feature status function in smu<br>
> <br>
> Change-Id: Id9a373f8d8866b97450be0aef0ba19d0835d40d8<br>
> Signed-off-by: Evan Quan <evan.quan@amd.com><br>
> ---<br>
> drivers/gpu/drm/amd/powerplay/arcturus_ppt.c | 2 ++<br>
> drivers/gpu/drm/amd/powerplay/inc/smu_types.h | 1 +<br>
> 2 files changed, 3 insertions(+)<br>
> <br>
> diff --git a/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c<br>
> b/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c<br>
> index dd6563358e8e..f1f072012fac 100644<br>
> --- a/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c<br>
> +++ b/drivers/gpu/drm/amd/powerplay/arcturus_ppt.c<br>
> @@ -141,6 +141,7 @@ static struct smu_11_0_cmn2aisc_mapping<br>
> arcturus_feature_mask_map[SMU_FEATURE_CO<br>
> FEA_MAP(DPM_SOCCLK),<br>
> FEA_MAP(DPM_FCLK),<br>
> FEA_MAP(DPM_MP0CLK),<br>
> + ARCTURUS_FEA_MAP(SMU_FEATURE_XGMI_BIT,<br>
> FEATURE_DPM_XGMI_BIT),<br>
> FEA_MAP(DS_GFXCLK),<br>
> FEA_MAP(DS_SOCCLK),<br>
> FEA_MAP(DS_LCLK),<br>
> @@ -149,6 +150,7 @@ static struct smu_11_0_cmn2aisc_mapping<br>
> arcturus_feature_mask_map[SMU_FEATURE_CO<br>
> FEA_MAP(GFX_ULV),<br>
> ARCTURUS_FEA_MAP(SMU_FEATURE_VCN_PG_BIT,<br>
> FEATURE_DPM_VCN_BIT),<br>
> FEA_MAP(RSMU_SMN_CG),<br>
> + FEA_MAP(WAFL_CG),<br>
> FEA_MAP(PPT),<br>
> FEA_MAP(TDC),<br>
> FEA_MAP(APCC_PLUS),<br>
> diff --git a/drivers/gpu/drm/amd/powerplay/inc/smu_types.h<br>
> b/drivers/gpu/drm/amd/powerplay/inc/smu_types.h<br>
> index 72962e842d69..c3c74098f614 100644<br>
> --- a/drivers/gpu/drm/amd/powerplay/inc/smu_types.h<br>
> +++ b/drivers/gpu/drm/amd/powerplay/inc/smu_types.h<br>
> @@ -207,6 +207,7 @@ enum smu_clk_type {<br>
> __SMU_DUMMY_MAP(TEMP_DEPENDENT_VMIN), \<br>
> __SMU_DUMMY_MAP(MMHUB_PG), \<br>
> __SMU_DUMMY_MAP(ATHUB_PG), \<br>
> + __SMU_DUMMY_MAP(WAFL_CG),<br>
> <br>
> #undef __SMU_DUMMY_MAP<br>
> #define __SMU_DUMMY_MAP(feature)<br>
> SMU_FEATURE_##feature##_BIT<br>
> --<br>
> 2.23.0<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>