On 08/31/2016 06:04 AM, Arnd Bergmann wrote: > On Tuesday 30 August 2016, Baoyou Xie wrote: >> We get some warnings when building kernel with W=1: >> drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgf117.c:222:1: warning: no >> previous prototype for 'gf117_grctx_generate_main' [-Wmissing-prototypes] >> drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxnv50.c:255:1: warning: no previous >> prototype for 'nv50_grctx_fill' [-Wmissing-prototypes] >> drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxnv50.c:265:1: warning: no previous >> prototype for 'nv50_grctx_init' [-Wmissing-prototypes] >> .... >> >> In fact, some functions are only used in the file in which they >> are declared and don't need a declaration, but can be made static. >> and others are declared in the header files, but need to add >> missing header dependencies. >> >> Signed-off-by: Baoyou Xie <baoyou.xie at linaro.org> > > The patch looks very good, and this is better than one patch per file. I > would personally > do this as two patches, as you are touching a number of files and are doing > two separate > things here. As a rule of thumb, when the changelog text mentions two or more > things > that the patch does, the patch should be split up so that each part does one > thing. > After you do that, it also becomes easier to write a good subject line like > "mark > function as static" and "add missing includes", which tells you more about > the patch > than "silence warnings". > > It's quite likely that the nouveau maintainers are less picky than I am > though, so > if they want to apply the patch as-is > > Acked-by: Arnd Bergmann <arnd at arndb.de> > > Otherwise please split and resend in a few days, in case there are other > comments This patch does indeed look a lot better than the previous series. I would also prefer the patch to be split into two as suggested by Arnd.
Other than that, I'm very happy to take the patch. Thank you for your work! Ben. > that need to be addressed. > > Arnd > _______________________________________________ > dri-devel mailing list > dri-devel at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel > -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 801 bytes Desc: OpenPGP digital signature URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160904/4beb8c4e/attachment-0001.sig>
