[Nouveau] [PATCH] drm/nouveau/gr: add missing header dependencies

2016-08-31 Thread Baoyou Xie
nv50_grctx_init' [-Wmissing-prototypes] In fact, these functions are declared in gr/nv50.h, so this patch add missing header dependencies Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxnv50.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/nouveau/nvkm/en

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

2016-08-31 Thread Baoyou Xie
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 --- drivers/gpu/drm/nouveau/dispnv04/overlay.c | 2 +- drivers/gpu/drm/nouveau/nouveau_display.c | 2 +- drivers/gpu/drm/nouv

[Nouveau] [PATCH v2] drm/nouveau/pm: mark nvkm_perfsig_find() static

2016-08-31 Thread Baoyou Xie
new' [-Wmissing-prototypes] In fact, these functions are only used in the file in which they are declared and don't need a declaration, but can be made static. so this patch marks these functions with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/engine/pm

Re: [Nouveau] [PATCH v2] drm/nouveau/gr: mark gm107_grctx_generate_tpcid() static

2016-08-31 Thread Baoyou Xie
On 30 August 2016 at 00:01, Sean Paul wrote: > On Mon, Aug 29, 2016 at 9:02 AM, Baoyou Xie wrote: > > We get 1 warning when build kernel with W=1: > > drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgm107.c:937:1: warning: no > previous prototype for 'gm107_grctx_generate_tpcid

[Nouveau] [PATCH] drm/nouveau/gr/gf117-: mark symbols static where possible

2016-08-31 Thread Baoyou Xie
eclaration, but can be made static. so this patch marks this function with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgf117.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgf117.c b/dri

[Nouveau] [PATCH] drm/nouveau/pm: mark symbols static where possible

2016-08-31 Thread Baoyou Xie
new' [-Wmissing-prototypes] In fact, these functions are only used in the file in which they are declared and don't need a declaration, but can be made static. so this patch marks these functions with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/engine/pm

[Nouveau] [PATCH] fix:nv10_fence: mark symbols static where possible

2016-08-31 Thread Baoyou Xie
he file in which it is declared and don't need a declaration, but can be made static. so this patch marks it 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nv10_fence.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nv

Re: [Nouveau] [PATCH v2] drm/nouveau/gr: mark gm107_grctx_generate_tpcid() static

2016-08-31 Thread Baoyou Xie
dear all: 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". FYI. On 30 August 2016 at 20:40, Arnd Bergmann wrote: > On Tuesday 30 August 2016, Baoyou Xie wrote: &

[Nouveau] [PATCH] fix:overlay: add missing header dependencies

2016-08-31 Thread Baoyou Xie
We get 1 warning when build kernel with W=1: drivers/gpu/drm/nouveau/dispnv04/overlay.c:496:1: warning: no previous prototype for 'nouveau_overlay_init' [-Wmissing-prototypes] In fact, this function is declared in disp.h, so this patch add missing header dependencies Signed-off-by:

[Nouveau] [PATCH] fix:nouveau_display: mark symbols static where possible

2016-08-31 Thread Baoyou Xie
on is only used in the file in which it is declared and don't need a declaration, but can be made static. so this patch marks it 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nouveau_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Nouveau] [PATCH v2] drm/nouveau/gr: mark gm107_grctx_generate_tpcid() static

2016-08-31 Thread Baoyou Xie
eclaration, but can be made static. so this patch marks this function with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgm107.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgm107.c b/dri

[Nouveau] [PATCH] drm/nouveau/gr: mark symbols static where possible

2016-08-31 Thread Baoyou Xie
eclaration, but can be made static. so this patch marks this function with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgm107.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgm107.c b/dri

[Nouveau] [PATCH] drm/nouveau/gr/gm107: mark symbols static where possible

2016-08-31 Thread Baoyou Xie
but can be made static. so this patch marks this function with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/engine/gr/gm107.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/gr/gm107.c b/drivers/gpu/drm/nouve

[Nouveau] [PATCH v2] drm/nouveau: add missing header dependencies

2016-08-31 Thread Baoyou Xie
We get 1 warning when build kernel with W=1: drivers/gpu/drm/nouveau/dispnv04/overlay.c:496:1: warning: no previous prototype for 'nouveau_overlay_init' [-Wmissing-prototypes] In fact, this function is declared in disp.h, so this patch add missing header dependencies Signed-off-by:

[Nouveau] [PATCH v2] drm/nouveau/gr/gf117-: mark symbols static where possible

2016-08-31 Thread Baoyou Xie
eclaration, but can be made static. so this patch marks this function with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgf117.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgf117.c b/dri

[Nouveau] [PATCH] drm/nouveau/disp: remove unused function in sorg94.c

2016-09-15 Thread Baoyou Xie
We get 1 warning when building kernel with W=1: drivers/gpu/drm/nouveau/nvkm/engine/disp/sorg94.c:49:1: warning: no previous prototype for 'g94_sor_output_new' [-Wmissing-prototypes] In fact, this function is called by no one and not exported, so this patch removes it. Signed-off-by:

[Nouveau] [PATCH] drm/nouveau/core: add missing header dependencies

2016-09-18 Thread Baoyou Xie
put' [-Wmissing-prototypes] In fact, both functions are declared in drivers/gpu/drm/nouveau/include/nvkm/core/firmware.h, so this patch adds missing header dependencies. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/core/firmware.c | 1 + 1 file changed, 1 insertion(+) diff --g

[Nouveau] [PATCH 1/3] drm/nouveau: add missing header dependencies

2016-09-25 Thread Baoyou Xie
dev/volt/priv.h, drivers/gpu/drm/nouveau/nvkm/core/firmware.h, drivers/gpu/drm/nouveau/nvkm/engine/gr/nv50.h, drivers/gpu/drm/nouveau/dispnv04/disp.h, so this patch adds missing header dependencies. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/dispnv04/overlay.c | 1 + drivers/gpu/drm/nouve

[Nouveau] [PATCH 2/3] drm/nouveau: mark symbols static where possible

2016-09-25 Thread Baoyou Xie
27; [-Wmissing-prototypes] In fact, these functions are only used in the file in which they are declared and don't need a declaration, but can be made static. So this patch marks these functions with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nouv

[Nouveau] [PATCH 3/3] drm/nouveau: remove unused function

2016-09-25 Thread Baoyou Xie
We get 1 warning when building kernel with W=1: drivers/gpu/drm/nouveau/nvkm/engine/disp/sorg94.c:49:1: warning: no previous prototype for 'g94_sor_output_new' [-Wmissing-prototypes] In fact, this function is called by no one and not exported, so this patch removes it. Signed-off-by:

[Nouveau] [PATCH 02/17] drm/nouveau/bios: mark symbols static where possible

2016-10-22 Thread Baoyou Xie
try' [-Wmissing-prototypes] In fact, both functions are only used in the file in which they are declared and don't need a declaration, but can be made static. So this patch marks these functions with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/subdev/

[Nouveau] [PATCH 03/17] drm/nouveau/clk: mark symbol static where possible

2016-10-22 Thread Baoyou Xie
but can be made static. So this patch marks this function with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/subdev/clk/gt215.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/clk/gt215.c b/drivers/gpu/dr

[Nouveau] [PATCH 06/17] drm/nouveau/gpio: mark symbol static where possible

2016-10-22 Thread Baoyou Xie
ion, but can be made static. So this patch marks this function with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/subdev/gpio/gk104.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/gpio/gk104.c b/drivers/gp

[Nouveau] [PATCH 09/17] drm/nouveau/volt: mark symbols static where possible

2016-10-22 Thread Baoyou Xie
set' [-Wmissing-prototypes] In fact, both functions are only used in the file in which they are declared and don't need a declaration, but can be made static. So this patch marks these functions with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/s

[Nouveau] [PATCH 07/17] drm/nouveau/secboot: mark symbol static where possible

2016-10-22 Thread Baoyou Xie
ration, but can be made static. So this patch marks this function with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/subdev/secboot/gm200.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/secboot/gm200.c

[Nouveau] [PATCH 08/17] drm/nouveau/volt: add missing header dependencies

2016-10-22 Thread Baoyou Xie
g header dependencies. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/subdev/volt/gpio.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/volt/gpio.c b/drivers/gpu/drm/nouveau/nvkm/subdev/volt/gpio.c index d2bac1d..443c031 100644 --- a/drivers/gp

[Nouveau] [PATCH 05/17] drm/nouveau/fb: mark symbols static where possible

2016-10-22 Thread Baoyou Xie
7;gt215_link_train_fini' [-Wmissing-prototypes] In fact, both functions are only used in the file in which they are declared and don't need a declaration, but can be made static. So this patch marks these functions with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau

[Nouveau] [PATCH 04/17] drm/nouveau/fb: add missing header dependencies

2016-10-22 Thread Baoyou Xie
alc' [-Wmissing-prototypes] In fact, these functions are declared in drivers/gpu/drm/nouveau/nvkm/subdev/fb/ram.h. So this patch adds missing header dependencies. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/subdev/fb/sddr2.c | 1 + drivers/gpu/drm/nouveau/nvkm/subdev/fb/sd

[Nouveau] [PATCH 10/17] drm/nouveau/device: mark symbol static where possible

2016-10-22 Thread Baoyou Xie
ion, but can be made static. So this patch marks this function with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/engine/device/user.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/nvkm/engine/device/user.c b/drivers/gp

[Nouveau] [PATCH 12/17] drm/nouveau/fifo: mark symbols static where possible

2016-10-22 Thread Baoyou Xie
totype for 'g84_fifo_chan_object_ctor' [-Wmissing-prototypes] In fact, both functions are only used in the file in which they are declared and don't need a declaration, but can be made static. So this patch marks these functions with 'static'. Signed-off-by: Baoyou Xie --- drive

[Nouveau] [PATCH 13/17] drm/nouveau/gr: add missing header dependencies

2016-10-22 Thread Baoyou Xie
nv50_grctx_init' [-Wmissing-prototypes] In fact, these functions are declared in drivers/gpu/drm/nouveau/nvkm/engine/gr/nv50.h. So this patch adds missing header dependencies. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxnv50.c | 1 + 1 file changed, 1 insertion(+) diff

[Nouveau] [PATCH 14/17] drm/nouveau/gr: mark symbols static where possible

2016-10-22 Thread Baoyou Xie
can be made static. So this patch marks these functions with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgf117.c | 2 +- drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgm107.c | 2 +- drivers/gpu/drm/nouveau/nvkm/engine/gr/gf100.c| 4 ++-- drivers/

[Nouveau] [PATCH 11/17] drm/nouveau/disp: mark symbols static where possible

2016-10-22 Thread Baoyou Xie
ed a declaration, but can be made static. So this patch marks these functions with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/engine/disp/channv50.c | 8 drivers/gpu/drm/nouveau/nvkm/engine/disp/rootnv50.c | 2 +- 2 files changed, 5 insertions(+), 5 de

[Nouveau] [PATCH 16/17] drm/nouveau/dispnv04: add missing header dependencies

2016-10-22 Thread Baoyou Xie
ing header dependencies. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/dispnv04/overlay.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/nouveau/dispnv04/overlay.c b/drivers/gpu/drm/nouveau/dispnv04/overlay.c index ec444ea..a79514d 100644 --- a/drive

[Nouveau] [PATCH 17/17] drm/nouveau: mark symbols static where possible

2016-10-22 Thread Baoyou Xie
nce_context_new' [-Wmissing-prototypes] In fact, both functions are only used in the file in which they are declared and don't need a declaration, but can be made static. So this patch marks these functions with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nouvea

[Nouveau] [PATCH 15/17] drm/nouveau/pm: mark symbols static where possible

2016-10-22 Thread Baoyou Xie
new' [-Wmissing-prototypes] In fact, both functions are only used in the file in which they are declared and don't need a declaration, but can be made static. So this patch marks these functions with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/engine

[Nouveau] [PATCH 01/17] drm/nouveau/core: add missing header dependencies

2016-10-22 Thread Baoyou Xie
put' [-Wmissing-prototypes] In fact, these functions are declared in drivers/gpu/drm/nouveau/include/nvkm/core/firmware.h. So this patch adds missing header dependencies. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nvkm/core/firmware.c | 1 + 1 file changed, 1 insertion(+) diff --g

[Nouveau] [PATCH v2 1/2] drm/nouveau: add missing header dependencies

2016-10-23 Thread Baoyou Xie
isp.h. So this patch adds missing header dependencies. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/dispnv04/overlay.c | 2 +- drivers/gpu/drm/nouveau/nvkm/core/firmware.c | 1 + drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxnv50.c | 1 + drivers/gpu/drm/nouveau/nvkm/subdev/fb/sddr2.c | 1 +

[Nouveau] [PATCH v2 2/2] drm/nouveau: mark symbols static where possible

2016-10-23 Thread Baoyou Xie
-Wmissing-prototypes] In fact, both functions are only used in the file in which they are declared and don't need a declaration, but can be made static. So this patch marks these functions with 'static'. Signed-off-by: Baoyou Xie --- drivers/gpu/drm/nouveau/nouveau_display.

Re: [Nouveau] [PATCH 01/17] drm/nouveau/core: add missing header dependencies

2016-10-24 Thread Baoyou Xie
On 23 October 2016 at 01:32, Karol Herbst wrote: > I think it would be better to squash those commits: > 1. for the includes > 2. for static declerations > > OK, I have resent new patch that squash those commits. > 2016-10-22 11:41 GMT+02:00 Baoyou Xie : > > We get

Re: [Nouveau] [PATCH 01/17] drm/nouveau/core: add missing header dependencies

2016-10-24 Thread Baoyou Xie
On 24 October 2016 at 23:43, Karol Herbst wrote: > 2016-10-24 9:13 GMT+02:00 Baoyou Xie : > > > > > > On 23 October 2016 at 01:32, Karol Herbst wrote: > >> > >> I think it would be better to squash those commits: > >> 1. for the includes >