Baoyou Xie
2016-Aug-28 05:02 UTC
[Nouveau] [PATCH] fix:overlay: add missing header dependencies
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 <baoyou.xie at 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 100644 --- a/drivers/gpu/drm/nouveau/dispnv04/overlay.c +++ b/drivers/gpu/drm/nouveau/dispnv04/overlay.c @@ -33,7 +33,7 @@ #include "nouveau_connector.h" #include "nouveau_display.h" #include "nvreg.h" - +#include "disp.h" struct nouveau_plane { struct drm_plane base; -- 2.7.4
Arnd Bergmann
2016-Aug-29 10:41 UTC
[Nouveau] [PATCH] fix:overlay: add missing header dependencies
On Sunday, August 28, 2016 1:02:52 PM CEST Baoyou Xie wrote:> 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 <baoyou.xie at linaro.org> >Acked-by: Arnd Bergmann <arnd at arndb.de> a few general notes: - please use my arnd at arndb.de address on patch submissions, not the linaro address. - for the email subject lines, have a look at what the typical style is for the file. "overlay:" is not a good name here because it's not unique to the driver, so better use "drm/nouveau", which will also get the attention of the right people. - please drop the "fix:" prefix in the subject. Arnd
Possibly Parallel Threads
- [PATCH v2] drm/nouveau: add missing header dependencies
- [PATCH 1/3] drm/nouveau: add missing header dependencies
- [PATCH] drm/nouveau: silence warnings reported during builds with W=1
- [PATCH] drm/nv10/plane: add plane support for nv10-nv40
- [PATCH v2 1/2] drm/nouveau: add missing header dependencies