On 24 October 2016 at 23:43, Karol Herbst <karolher...@gmail.com> wrote:
> 2016-10-24 9:13 GMT+02:00 Baoyou Xie <baoyou@linaro.org>:
> >
> >
> > On 23 October 2016 at 01:32, Karol Herbst <karolher...@gmail.com> wrote:
> >>
> >> I
On 23 October 2016 at 01:32, Karol Herbst <karolher...@gmail.com> 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+
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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nouveau_display.c | 2 +-
drivers/gpu/drm/n
dependencies.
Signed-off-by: Baoyou Xie <baoyou@linaro.org>
---
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 +
drivers/g
-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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nvkm/core/firmware.c | 1 +
1 file changed, 1 insertion(+)
-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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nvkm/engine/pm/
' [-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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nouveau_dis
dependencies.
Signed-off-by: Baoyou Xie <baoyou@linaro.org>
---
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
marks these functions with 'static'.
Signed-off-by: Baoyou Xie <baoyou@linaro.org>
---
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 deletions(-)
diff --git a/drivers/g
' [-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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxnv50.c | 1 +
1 file changed, 1 ins
with 'static'.
Signed-off-by: Baoyou Xie <baoyou@linaro.org>
---
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/gpu/drm/nouveau/nvkm/engine/gr/g
be made static.
So this patch marks this function with 'static'.
Signed-off-by: Baoyou Xie <baoyou@linaro.org>
---
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/drive
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 <baoyou@linaro.org>
---
drivers/g
be made static.
So this patch marks this function with 'static'.
Signed-off-by: Baoyou Xie <baoyou@linaro.org>
---
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/gm20
-off-by: Baoyou Xie <baoyou@linaro.org>
---
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/g
-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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nvkm/subdev/fb/ramgt215
-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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nvkm/subdev/fb/sddr2.c | 1 +
drivers/gpu/drm/nouveau/nvkm/sub
be made static.
So this patch marks this function with 'static'.
Signed-off-by: Baoyou Xie <baoyou@linaro.org>
---
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/drive
-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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nvkm/subdev/volt/g
static.
So this patch marks this function with 'static'.
Signed-off-by: Baoyou Xie <baoyou@linaro.org>
---
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/g
-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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nvkm/subdev/bios
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: Baoyou Xie
, 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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nouveau_display.c | 2 +-
drive
/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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/dispnv04/overlay.c | 1 +
drivers/gpu/drm/nouveau/nvkm/core/firm
-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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nvkm/core/firmware.c | 1 +
1 file changed, 1 insertion(+)
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: Baoyou Xie
, but can be made static.
so this patch marks this function with 'static'.
Signed-off-by: Baoyou Xie <baoyou@linaro.org>
---
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/ctxgf11
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: Baoyou Xie
static.
so this patch marks this function with 'static'.
Signed-off-by: Baoyou Xie <baoyou@linaro.org>
---
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/n
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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nv10_fence.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/n
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 <a...@arndb.de> wrote:
> On Tuesday 30 August 2016, Baoyou X
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: Baoyou Xie
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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nouveau_display.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/d
, but can be made static.
so this patch marks this function with 'static'.
Signed-off-by: Baoyou Xie <baoyou@linaro.org>
---
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/ctxgf11
On 30 August 2016 at 00:01, Sean Paul <seanp...@google.com> wrote:
> On Mon, Aug 29, 2016 at 9:02 AM, Baoyou Xie <baoyou@linaro.org> wrote:
> > We get 1 warning when build kernel with W=1:
> > drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxgm107.c:937:1: warni
' [-Wmissing-prototypes]
In fact, these functions are declared in gr/nv50.h, so this patch
add missing header dependencies
Signed-off-by: Baoyou Xie <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nvkm/engine/gr/ctxnv50.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/nouvea
.
and others are declared in the header files, but need to add
missing header dependencies.
Signed-off-by: Baoyou Xie <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/dispnv04/overlay.c | 2 +-
drivers/gpu/drm/nouveau/nouveau_display.c | 2 +-
drivers/gpu/drm/n
-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 <baoyou@linaro.org>
---
drivers/gpu/drm/nouveau/nvkm/engine/pm/base
38 matches
Mail list logo