<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="divtagdefaultwrapper" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;">
<p><br>
</p>
<meta content="text/html; charset=UTF-8">
<div dir="ltr">
<div id="x_divtagdefaultwrapper" style="font-size:12pt; color:#000000; background-color:#FFFFFF; font-family:Calibri,Arial,Helvetica,sans-serif">
<p><span style="color:rgb(33,33,33); font-size:13.3333px">Yes, <span style="color:rgb(33,33,33); font-size:13.3333px">stretch_amount2 was not used on Polaris.</span></span></p>
<p><span style="color:rgb(33,33,33); font-size:13.3333px"><span style="color:rgb(33,33,33); font-size:13.3333px"><br>
</span></span></p>
<p><span style="color:rgb(33,33,33); font-size:13.3333px">Patch was Reviewed-by: <span style="color: rgb(33, 33, 33); font-size: 13.3333px;">Rex Zhu <Rex.Zhu@amd.com></span></span><br>
</p>
<p><span style="color:rgb(33,33,33); font-size:13.3333px"><span style="color: rgb(33, 33, 33); font-size: 13.3333px;"><br>
</span></span></p>
<p><span style="color:rgb(33,33,33); font-size:13.3333px"><span style="color: rgb(33, 33, 33); font-size: 13.3333px;">Thanks.</span></span></p>
<p><span style="color:rgb(33,33,33); font-size:13.3333px"><span style="color: rgb(33, 33, 33); font-size: 13.3333px;"><br>
</span></span></p>
<p><span style="color:rgb(33,33,33); font-size:13.3333px"><span style="color: rgb(33, 33, 33); font-size: 13.3333px;">Best Regards</span></span></p>
<p><span style="color:rgb(33,33,33); font-size:13.3333px"><span style="color: rgb(33, 33, 33); font-size: 13.3333px;">Rex</span></span></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> Matthias Beyer <mail@beyermatthias.de><br>
<b>Sent:</b> Friday, July 1, 2016 12:38:49 AM<br>
<b>To:</b> linuxdev.baldrick@gmail.com<br>
<b>Cc:</b> Deucher, Alexander; Koenig, Christian; airlied@linux.ie; dri-devel@lists.freedesktop.org; dcb314@hotmail.com; linux-kernel@vger.kernel.org; Zhu, Rex; Huang, JinHuiEric; nils.wallmenius@gmail.com; Matthias Beyer<br>
<b>Subject:</b> [PATCH] drivers: gpu: drm: amd: powerplay: hwmgr: Remove unused variable</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">Signed-off-by: Matthias Beyer <mail@beyermatthias.de><br>
---<br>
drivers/gpu/drm/amd/powerplay/hwmgr/polaris10_hwmgr.c | 9 +++------<br>
1 file changed, 3 insertions(+), 6 deletions(-)<br>
<br>
diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/polaris10_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/polaris10_hwmgr.c<br>
index 64ee78f..1dcd52d 100644<br>
--- a/drivers/gpu/drm/amd/powerplay/hwmgr/polaris10_hwmgr.c<br>
+++ b/drivers/gpu/drm/amd/powerplay/hwmgr/polaris10_hwmgr.c<br>
@@ -1761,7 +1761,7 @@ static int polaris10_populate_clock_stretcher_data_table(struct pp_hwmgr *hwmgr)<br>
{<br>
uint32_t ro, efuse, volt_without_cks, volt_with_cks, value, max, min;<br>
struct polaris10_hwmgr *data = (struct polaris10_hwmgr *)(hwmgr->backend);<br>
- uint8_t i, stretch_amount, stretch_amount2, volt_offset = 0;<br>
+ uint8_t i, stretch_amount, volt_offset = 0;<br>
struct phm_ppt_v1_information *table_info =<br>
(struct phm_ppt_v1_information *)(hwmgr->pptable);<br>
struct phm_ppt_v1_clock_voltage_dependency_table *sclk_table =<br>
@@ -1806,11 +1806,8 @@ static int polaris10_populate_clock_stretcher_data_table(struct pp_hwmgr *hwmgr)<br>
}<br>
<br>
/* Populate CKS Lookup Table */<br>
- if (stretch_amount == 1 || stretch_amount == 2 || stretch_amount == 5)<br>
- stretch_amount2 = 0;<br>
- else if (stretch_amount == 3 || stretch_amount == 4)<br>
- stretch_amount2 = 1;<br>
- else {<br>
+ if (stretch_amount != 1 && stretch_amount != 2 && stretch_amount != 3 &&<br>
+ stretch_amount != 4 && stretch_amount != 5) {<br>
phm_cap_unset(hwmgr->platform_descriptor.platformCaps,<br>
PHM_PlatformCaps_ClockStretcher);<br>
PP_ASSERT_WITH_CODE(false,<br>
-- <br>
2.9.0<br>
<br>
</div>
</span></font></div>
</body>
</html>