<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 30 August 2016 at 00:01, Sean Paul <span dir="ltr"><<a href="mailto:seanpaul@google.com" target="_blank">seanpaul@google.com</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="">On Mon, Aug 29, 2016 at 9:02 AM, Baoyou Xie <<a href="mailto:baoyou.xie@linaro.org">baoyou.xie@linaro.org</a>> wrote:<br>
> We get 1 warning when build kernel with W=1:<br>
> drivers/gpu/drm/nouveau/nvkm/<wbr>engine/gr/ctxgm107.c:937:1: warning: no previous prototype for 'gm107_grctx_generate_tpcid' [-Wmissing-prototypes]<br>
><br>
> In fact, this function is only used in the file in which it is<br>
> declared and don't need a declaration, but can be made static.<br>
> so this patch marks this function with 'static'.<br>
><br>
<br>
</span>It would make things a lot easier if you either consolidated all of<br>
these static changes into one patch, or released them as a series.<br>
<span class=""><font color="#888888"><br></font></span></blockquote><div> </div><div>Do you mean that I need to consolidate all of them into one patch this time? or next time?<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=""><font color="#888888">
Sean<br>
</font></span><div class=""><div class="h5"><br>
<br>
> Signed-off-by: Baoyou Xie <<a href="mailto:baoyou.xie@linaro.org">baoyou.xie@linaro.org</a>><br>
> ---<br>
>  drivers/gpu/drm/nouveau/nvkm/<wbr>engine/gr/ctxgm107.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/drivers/gpu/drm/nouveau/<wbr>nvkm/engine/gr/ctxgm107.c b/drivers/gpu/drm/nouveau/<wbr>nvkm/engine/gr/ctxgm107.c<br>
> index 6d3c501..4c4b5ab 100644<br>
> --- a/drivers/gpu/drm/nouveau/<wbr>nvkm/engine/gr/ctxgm107.c<br>
> +++ b/drivers/gpu/drm/nouveau/<wbr>nvkm/engine/gr/ctxgm107.c<br>
> @@ -933,7 +933,7 @@ gm107_grctx_generate_attrib(<wbr>struct gf100_grctx *info)<br>
>         }<br>
>  }<br>
><br>
> -void<br>
> +static void<br>
>  gm107_grctx_generate_tpcid(<wbr>struct gf100_gr *gr)<br>
>  {<br>
>         struct nvkm_device *device = gr->base.engine.subdev.device;<br>
> --<br>
> 2.7.4<br>
><br>
</div></div></blockquote></div><br></div></div>