<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
</head>
<body>
<meta content="text/html; charset=UTF-8">
<style type="text/css" style="">
<!--
p
{margin-top:0;
margin-bottom:0}
-->
</style>
<div dir="ltr">
<div id="x_divtagdefaultwrapper" style="font-size:12pt; color:#000000; font-family:Calibri,Arial,Helvetica,sans-serif">
<p>Ok. i will add CI support in powerplay next week.</p>
<p><br>
</p>
<p>Best Regards</p>
<p>Rex</p>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Deucher, Alexander<br>
<b>Sent:</b> Friday, October 14, 2016 9:42:42 PM<br>
<b>To:</b> Zhu, Rex; amd-gfx@lists.freedesktop.org<br>
<b>Cc:</b> Zhu, Rex<br>
<b>Subject:</b> RE: [PATCH 1/7] drm/amdgpu: check min clock set by DAL before set ps.</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">> -----Original Message-----<br>
> From: amd-gfx [<a href="mailto:amd-gfx-bounces@lists.freedesktop.org">mailto:amd-gfx-bounces@lists.freedesktop.org</a>] On Behalf<br>
> Of Rex Zhu<br>
> Sent: Friday, October 14, 2016 8:07 AM<br>
> To: amd-gfx@lists.freedesktop.org<br>
> Cc: Zhu, Rex<br>
> Subject: [PATCH 1/7] drm/amdgpu: check min clock set by DAL before set ps.<br>
> <br>
> Change-Id: Id71b9a3329a8a143a189b275926b0a2054bf0bb8<br>
<br>
Series is:<br>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com><br>
<br>
For CI, I wonder if it would be better to add support for it to the powerplay smu7 module since it's pretty similar to the other smu7 dGPUs.<br>
<br>
Alex<br>
<br>
> ---<br>
> drivers/gpu/drm/amd/amdgpu/ci_dpm.c | 6 ++++++<br>
> 1 file changed, 6 insertions(+)<br>
> <br>
> diff --git a/drivers/gpu/drm/amd/amdgpu/ci_dpm.c<br>
> b/drivers/gpu/drm/amd/amdgpu/ci_dpm.c<br>
> index 1d8c375..fd2fe85 100644<br>
> --- a/drivers/gpu/drm/amd/amdgpu/ci_dpm.c<br>
> +++ b/drivers/gpu/drm/amd/amdgpu/ci_dpm.c<br>
> @@ -960,6 +960,12 @@ static void ci_apply_state_adjust_rules(struct<br>
> amdgpu_device *adev,<br>
> sclk = ps->performance_levels[0].sclk;<br>
> }<br>
> <br>
> + if (adev->pm.pm_display_cfg.min_core_set_clock > sclk)<br>
> + sclk = adev->pm.pm_display_cfg.min_core_set_clock;<br>
> +<br>
> + if (adev->pm.pm_display_cfg.min_mem_set_clock > mclk)<br>
> + mclk = adev->pm.pm_display_cfg.min_mem_set_clock;<br>
> +<br>
> if (rps->vce_active) {<br>
> if (sclk < adev->pm.dpm.vce_states[adev-<br>
> >pm.dpm.vce_level].sclk)<br>
> sclk = adev->pm.dpm.vce_states[adev-<br>
> >pm.dpm.vce_level].sclk;<br>
> --<br>
> 1.9.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>
</body>
</html>