[PATCH] drm/nouveau: silence warnings reported during builds with W=1

[Ben Skeggs]

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>