<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<!--[if !mso]><style>v\:* {behavior:url(#default#VML);}
o\:* {behavior:url(#default#VML);}
w\:* {behavior:url(#default#VML);}
.shape {behavior:url(#default#VML);}
</style><![endif]--><style><!--
/* Font Definitions */
@font-face
        {font-family:SimSun;
        panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:SimSun;
        panose-1:2 1 6 0 3 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
p.msonormal0, li.msonormal0, div.msonormal0
        {mso-style-name:msonormal;
        margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
span.EmailStyle20
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.25in 1.0in 1.25in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal">It’s a bug fix for uploading new pptable from sysfs.<o:p></o:p></p>
<p class="MsoNormal">And it’s not hardware involved. The real hardware operation involved with this is in smu_v11_0_enable_thermal_alert.<o:p></o:p></p>
<p class="MsoNormal">And in old powerplay routine, this is also put in sw_init and we do not find any problem with that until now.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Regards,<o:p></o:p></p>
<p class="MsoNormal">Evan<o:p></o:p></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt">
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b>From:</b> Wang, Kevin(Yang) <Kevin1.Wang@amd.com> <br>
<b>Sent:</b> Thursday, July 25, 2019 9:08 PM<br>
<b>To:</b> Quan, Evan <Evan.Quan@amd.com>; amd-gfx@lists.freedesktop.org<br>
<b>Subject:</b> Re: [PATCH 2/2] drm/amd/powerplay: enable SW SMU reset functionality<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div id="appendonsend">
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">I don't recommend that, because obviously this is a
<b>hardware</b> <b>operation</b>, so why need to move it  into <b>sw</b> <b>init</b>?<o:p></o:p></span></p>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">Is this a workaround solution, or is it specific to which asic?<o:p></o:p></span></p>
</div>
</div>
<div id="appendonsend">
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">and set pptable from sysfs and do baco reset operation need to call these functions, have you test it?<o:p></o:p></span></p>
</div>
<div id="appendonsend">
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">the function in amdgpu_smu.c and smu_v11_0.c need work correctly in all asic,<o:p></o:p></span></p>
</div>
<div id="appendonsend">
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">it needs to be done carefully, unless there is a good reason.<o:p></o:p></span></p>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black"><o:p> </o:p></span></p>
</div>
</div>
<div id="appendonsend">
<p class="MsoNormal"><span style="font-size:12.0pt;color:black">Best Regards,<br>
Kevin<o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:12.0pt;color:black"><o:p> </o:p></span></p>
</div>
<div class="MsoNormal" align="center" style="text-align:center">
<hr size="2" width="98%" align="center">
</div>
<div id="divRplyFwdMsg">
<p class="MsoNormal"><b><span style="color:black">From:</span></b><span style="color:black"> amd-gfx <<a href="mailto:amd-gfx-bounces@lists.freedesktop.org">amd-gfx-bounces@lists.freedesktop.org</a>> on behalf of Evan Quan <<a href="mailto:evan.quan@amd.com">evan.quan@amd.com</a>><br>
<b>Sent:</b> Thursday, July 25, 2019 10:39 AM<br>
<b>To:</b> <a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> <<a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a>><br>
<b>Cc:</b> Quan, Evan <<a href="mailto:Evan.Quan@amd.com">Evan.Quan@amd.com</a>><br>
<b>Subject:</b> [PATCH 2/2] drm/amd/powerplay: enable SW SMU reset functionality</span>
<o:p></o:p></p>
<div>
<p class="MsoNormal"> <o:p></o:p></p>
</div>
</div>
<div>
<div>
<p class="MsoNormal">Move SMU irq handler register to sw_init as that's totally<br>
software related. Otherwise, it will prevent SMU reset working.<br>
<br>
Change-Id: Ibd3e48ae9a90ab57f42b3f2ddbb736deeebc8715<br>
Signed-off-by: Evan Quan <<a href="mailto:evan.quan@amd.com">evan.quan@amd.com</a>><br>
---<br>
 drivers/gpu/drm/amd/powerplay/amdgpu_smu.c | 16 +++++++++-------<br>
 1 file changed, 9 insertions(+), 7 deletions(-)<br>
<br>
diff --git a/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c b/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c<br>
index 6935a00cd389..a5079b93caa3 100644<br>
--- a/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c<br>
+++ b/drivers/gpu/drm/amd/powerplay/amdgpu_smu.c<br>
@@ -741,6 +741,12 @@ static int smu_sw_init(void *handle)<br>
                 return ret;<br>
         }<br>
 <br>
+       ret = smu_register_irq_handler(smu);<br>
+       if (ret) {<br>
+               pr_err("Failed to register smc irq handler!\n");<br>
+               return ret;<br>
+       }<br>
+<br>
         return 0;<br>
 }<br>
 <br>
@@ -750,6 +756,9 @@ static int smu_sw_fini(void *handle)<br>
         struct smu_context *smu = &adev->smu;<br>
         int ret;<br>
 <br>
+       kfree(smu->irq_source);<br>
+       smu->irq_source = NULL;<br>
+<br>
         ret = smu_smc_table_sw_fini(smu);<br>
         if (ret) {<br>
                 pr_err("Failed to sw fini smc table!\n");<br>
@@ -1061,10 +1070,6 @@ static int smu_hw_init(void *handle)<br>
         if (ret)<br>
                 goto failed;<br>
 <br>
-       ret = smu_register_irq_handler(smu);<br>
-       if (ret)<br>
-               goto failed;<br>
-<br>
         if (!smu->pm_enabled)<br>
                 adev->pm.dpm_enabled = false;<br>
         else<br>
@@ -1094,9 +1099,6 @@ static int smu_hw_fini(void *handle)<br>
         kfree(table_context->overdrive_table);<br>
         table_context->overdrive_table = NULL;<br>
 <br>
-       kfree(smu->irq_source);<br>
-       smu->irq_source = NULL;<br>
-<br>
         ret = smu_fini_fb_allocations(smu);<br>
         if (ret)<br>
                 return ret;<br>
-- <br>
2.22.0<br>
<br>
_______________________________________________<br>
amd-gfx mailing list<br>
<a href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a><o:p></o:p></p>
</div>
</div>
</div>
</div>
</body>
</html>