Re: [U-Boot] [PATCH v2] kbuild: do not hide clang warnings during build with W=1

2014-11-04 Thread Tom Rini
On Fri, Oct 31, 2014 at 11:00:34AM +0900, Masahiro Yamada wrote:

 These lines originate in Linux;  it looks like Linux folks chose to
 hide bunch of warnings reported by clang rather than fixing C code.
 
 In U-boot, warnings are being fixed thanks to Jeroen's great efforts.
 Let's stop suppressing clang warnings.
 
 Signed-off-by: Masahiro Yamada yamad...@jp.panasonic.com
 Cc: Jeroen Hofstee jer...@myspectrum.nl

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v2] kbuild: do not hide clang warnings during build with W=1

2014-10-30 Thread Masahiro Yamada
These lines originate in Linux;  it looks like Linux folks chose to
hide bunch of warnings reported by clang rather than fixing C code.

In U-boot, warnings are being fixed thanks to Jeroen's great efforts.
Let's stop suppressing clang warnings.

Signed-off-by: Masahiro Yamada yamad...@jp.panasonic.com
Cc: Jeroen Hofstee jer...@myspectrum.nl
---

Changes in v2:
  - Rephrase commit summary and description for clarification

 scripts/Makefile.extrawarn | 10 --
 1 file changed, 10 deletions(-)

diff --git a/scripts/Makefile.extrawarn b/scripts/Makefile.extrawarn
index 6564350..0ec0d24 100644
--- a/scripts/Makefile.extrawarn
+++ b/scripts/Makefile.extrawarn
@@ -26,16 +26,6 @@ warning-1 += $(call cc-option, -Wmissing-include-dirs)
 warning-1 += $(call cc-option, -Wunused-but-set-variable)
 warning-1 += $(call cc-disable-warning, missing-field-initializers)
 
-# Clang
-warning-1 += $(call cc-disable-warning, initializer-overrides)
-warning-1 += $(call cc-disable-warning, unused-value)
-warning-1 += $(call cc-disable-warning, format)
-warning-1 += $(call cc-disable-warning, unknown-warning-option)
-warning-1 += $(call cc-disable-warning, sign-compare)
-warning-1 += $(call cc-disable-warning, format-zero-length)
-warning-1 += $(call cc-disable-warning, uninitialized)
-warning-1 += $(call cc-option, -fcatch-undefined-behavior)
-
 warning-2 := -Waggregate-return
 warning-2 += -Wcast-align
 warning-2 += -Wdisabled-optimization
-- 
1.9.1

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot