Thanks review .

Initial thoughts use this patch for overall initializing the UMD_PSATET_* 
assignment and
some Pstate definitions are not used for patch#2. But combine those two patches 
also seems well.

Thanks,
Prike
From: Wang, Kevin(Yang) <kevin1.w...@amd.com>
Sent: Wednesday, September 4, 2019 2:32 PM
To: Liang, Prike <prike.li...@amd.com>; amd-gfx@lists.freedesktop.org
Cc: Deucher, Alexander <alexander.deuc...@amd.com>
Subject: Re: [PATCH 1/2] drm/amd/powerplay: Add UMD PState Renoir Msg Parameters

i think it's better to merge these two patches into one patch.
after fixed: Reviewed-by: Kevin Wang 
<kevin1.w...@amd.com<mailto:kevin1.w...@amd.com>>

Best Regards,
Kevin
________________________________
From: amd-gfx 
<amd-gfx-boun...@lists.freedesktop.org<mailto:amd-gfx-boun...@lists.freedesktop.org>>
 on behalf of Liang, Prike <prike.li...@amd.com<mailto:prike.li...@amd.com>>
Sent: Wednesday, September 4, 2019 2:01 PM
To: amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org> 
<amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org>>
Cc: Deucher, Alexander 
<alexander.deuc...@amd.com<mailto:alexander.deuc...@amd.com>>; Liang, Prike 
<prike.li...@amd.com<mailto:prike.li...@amd.com>>
Subject: [PATCH 1/2] drm/amd/powerplay: Add UMD PState Renoir Msg Parameters

Add UMD PState macro definition for PState update.

Signed-off-by: Prike Liang <prike.li...@amd.com<mailto:prike.li...@amd.com>>
reviewed-by: Evan Quan <evan.q...@amd.com<mailto:evan.q...@amd.com>>
[Kevin]: change "reviewed-by" to "Reviewed-by"
---
 drivers/gpu/drm/amd/powerplay/renoir_ppt.h | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/gpu/drm/amd/powerplay/renoir_ppt.h 
b/drivers/gpu/drm/amd/powerplay/renoir_ppt.h
index e9b7237..a017a47 100644
--- a/drivers/gpu/drm/amd/powerplay/renoir_ppt.h
+++ b/drivers/gpu/drm/amd/powerplay/renoir_ppt.h
@@ -25,4 +25,9 @@

 extern void renoir_set_ppt_funcs(struct smu_context *smu);

+//UMD PState Renoir Msg Parameters in MHz
+#define RENOIR_UMD_PSTATE_GFXCLK       700
+#define RENOIR_UMD_PSTATE_SOCCLK       678
+#define RENOIR_UMD_PSTATE_FCLK         800
+
 #endif
--
2.7.4

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org>
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to