Thread (6 messages) 6 messages, 4 authors, 2016-11-16

Re: [GIT PULL] fbdev fixes for 4.9

flat view

From: Michael Ellerman <mpe@ellerman.id.au>
Date: 2016-11-16 06:15:49
Also in: lkml
Subsystem: framebuffer layer, the rest · Maintainers: Helge Deller, Linus Torvalds

Tomi Valkeinen [off-list ref] writes:
[ Unknown signature status ]
On 14/11/16 18:25, Linus Torvalds wrote:
quoted
On Mon, Nov 14, 2016 at 3:44 AM, Tomi Valkeinen [off-list ref] wrote:
quoted
Please pull two fbdev fixes for 4.9.
No.

This has obviously never even been test-compiled. It introduces two
new annoying warnings.
Sorry about that. I dropped the patch for now and here's a new pull request.

I did test compile, and I would have noticed errors but I missed the warnings
(I blame the flu...). I need to improve my testing methods, but I couldn't find
a way to add -Werror to kernel builds.
You can turn it on per-directory, eg:
diff --git a/drivers/video/fbdev/Makefile b/drivers/video/fbdev/Makefile
index ee8c81405a7f..2ee96810d26d 100644
--- a/drivers/video/fbdev/Makefile
+++ b/drivers/video/fbdev/Makefile
@@ -4,6 +4,8 @@
 
 # Each configuration option enables a list of files.
 
+subdir-ccflags-y := -Werror
+
 obj-y				+= core/
 
 obj-$(CONFIG_FB_MACMODES)      += macmodes.o

cheers
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help