<div dir="ltr"><div><div>dear all:<br></div>    I have resend a patch, which consolidate the 'nouveau' changes into it, its subject is "drm/nouveau: silence warnings reported during builds with W=1".<br></div>   FYI.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 30 August 2016 at 20:40, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tuesday 30 August 2016, Baoyou Xie wrote:<br>
> On 30 August 2016 at 00:01, Sean Paul <<a href="mailto:seanpaul@google.com">seanpaul@google.com</a>> wrote:<br>
><br>
> > 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<br>
> > 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>
> > 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>
> ><br>
> ><br>
> Do you mean that I need to consolidate all of them into one patch this<br>
> time? or next time?<br>
<br>
</span>I'd suggest consolidating the 'nouveau' changes into a single patch,<br>
as this is one (very big) driver and resend that one.<br>
<span class="HOEnZb"><font color="#888888"><br>
        Arnd<br>
</font></span></blockquote></div><br></div>