<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 25 October 2016 at 04:51, Arnd Bergmann <span dir="ltr"><<a href="mailto:arnd@arndb.de" target="_blank">arnd@arndb.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-">On Saturday, October 22, 2016 4:56:22 PM CEST Baoyou Xie wrote:<br>
> @@ -1341,7 +1341,7 @@ int smu7_disable_dpm_tasks(struct pp_hwmgr *hwmgr)<br>
>         return result;<br>
>  }<br>
><br>
> -int smu7_reset_asic_tasks(struct pp_hwmgr *hwmgr)<br>
> +static int smu7_reset_asic_tasks(struct pp_hwmgr *hwmgr)<br>
>  {<br>
><br>
>         return 0;<br>
><br>
<br>
</span>This needs to be done differently:<br>
<br>
../drivers/gpu/drm/amd/amdgpu/<wbr>../powerplay/hwmgr/smu7_hwmgr.<wbr>c:1344:12: error: 'smu7_reset_asic_tasks' defined but not used [-Werror=unused-function]<br>
<br>
The function has no callers, so the easiest way would be to remove it<br>
entirely, but it's possible that there are plans to add users soon.<br>
<span class="gmail-HOEnZb"><font color="#888888"><br></font></span></blockquote><div>It was assumed that this function will be used soon, so this patch remains it.<br></div><div>if it still not be used in 4.10, then we can remove it.<br></div><div>is it right?<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-HOEnZb"><font color="#888888">
        Arnd<br>
</font></span></blockquote></div><br></div></div>