Arnd Bergmann
2023-Apr-17  21:03 UTC
[Nouveau] [PATCH] drm/nouveau: dispnv50: fix missing-prototypes warning
From: Arnd Bergmann <arnd at arndb.de>
nv50_display_create() is declared in another header, along with
a couple of declarations that are now outdated:
drivers/gpu/drm/nouveau/dispnv50/disp.c:2517:1: error: no previous prototype for
'nv50_display_create'
Fixes: ba801ef068c1 ("drm/nouveau/kms: display destroy/init/fini hooks can
be static")
Signed-off-by: Arnd Bergmann <arnd at arndb.de>
---
 drivers/gpu/drm/nouveau/dispnv50/disp.c | 1 +
 drivers/gpu/drm/nouveau/nv50_display.h  | 4 +---
 2 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/nouveau/dispnv50/disp.c
b/drivers/gpu/drm/nouveau/dispnv50/disp.c
index 5bb777ff1313..9b6824f6b9e4 100644
--- a/drivers/gpu/drm/nouveau/dispnv50/disp.c
+++ b/drivers/gpu/drm/nouveau/dispnv50/disp.c
@@ -64,6 +64,7 @@
 #include "nouveau_connector.h"
 #include "nouveau_encoder.h"
 #include "nouveau_fence.h"
+#include "nv50_display.h"
 
 #include <subdev/bios/dp.h>
 
diff --git a/drivers/gpu/drm/nouveau/nv50_display.h
b/drivers/gpu/drm/nouveau/nv50_display.h
index fbd3b15583bc..60f77766766e 100644
--- a/drivers/gpu/drm/nouveau/nv50_display.h
+++ b/drivers/gpu/drm/nouveau/nv50_display.h
@@ -31,7 +31,5 @@
 #include "nouveau_reg.h"
 
 int  nv50_display_create(struct drm_device *);
-void nv50_display_destroy(struct drm_device *);
-int  nv50_display_init(struct drm_device *);
-void nv50_display_fini(struct drm_device *);
+
 #endif /* __NV50_DISPLAY_H__ */
-- 
2.39.2
Karol Herbst
2023-May-22  11:05 UTC
[Nouveau] [PATCH] drm/nouveau: dispnv50: fix missing-prototypes warning
Reviewed-by: Karol Herbst <kherbst at redhat.com> will merge soon On Mon, Apr 17, 2023 at 11:03?PM Arnd Bergmann <arnd at kernel.org> wrote:> > From: Arnd Bergmann <arnd at arndb.de> > > nv50_display_create() is declared in another header, along with > a couple of declarations that are now outdated: > > drivers/gpu/drm/nouveau/dispnv50/disp.c:2517:1: error: no previous prototype for 'nv50_display_create' > > Fixes: ba801ef068c1 ("drm/nouveau/kms: display destroy/init/fini hooks can be static") > Signed-off-by: Arnd Bergmann <arnd at arndb.de> > --- > drivers/gpu/drm/nouveau/dispnv50/disp.c | 1 + > drivers/gpu/drm/nouveau/nv50_display.h | 4 +--- > 2 files changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/gpu/drm/nouveau/dispnv50/disp.c b/drivers/gpu/drm/nouveau/dispnv50/disp.c > index 5bb777ff1313..9b6824f6b9e4 100644 > --- a/drivers/gpu/drm/nouveau/dispnv50/disp.c > +++ b/drivers/gpu/drm/nouveau/dispnv50/disp.c > @@ -64,6 +64,7 @@ > #include "nouveau_connector.h" > #include "nouveau_encoder.h" > #include "nouveau_fence.h" > +#include "nv50_display.h" > > #include <subdev/bios/dp.h> > > diff --git a/drivers/gpu/drm/nouveau/nv50_display.h b/drivers/gpu/drm/nouveau/nv50_display.h > index fbd3b15583bc..60f77766766e 100644 > --- a/drivers/gpu/drm/nouveau/nv50_display.h > +++ b/drivers/gpu/drm/nouveau/nv50_display.h > @@ -31,7 +31,5 @@ > #include "nouveau_reg.h" > > int nv50_display_create(struct drm_device *); > -void nv50_display_destroy(struct drm_device *); > -int nv50_display_init(struct drm_device *); > -void nv50_display_fini(struct drm_device *); > + > #endif /* __NV50_DISPLAY_H__ */ > -- > 2.39.2 >
Reasonably Related Threads
- [PATCH 1/3] drm/nouveau/kms/nv50-: make base917c_format static
- [PATCH 3/3] drm/nouveau/kms/nv50-: include n50_display.h for nv50_display_create
- [PATCH] bl: fix backlight regression
- [PATCH] disp/gf119-: avoid creating non-existent heads
- [PATCH 1/4] nouveau: Allow allocating BOs at specific offsets