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 >
Apparently Analagous 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