All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] MIPS: Disable Werror when W= is set
@ 2017-03-17  1:06 Florian Fainelli
  2017-03-21 19:27 ` Ralf Baechle
  0 siblings, 1 reply; 2+ messages in thread
From: Florian Fainelli @ 2017-03-17  1:06 UTC (permalink / raw)
  To: linux-mips; +Cc: ralf, james.hogan, Florian Fainelli

Using any value for W= will lead to a ton of warnings which are turned
into fatal errors because MIPS adds -Werror to arch/mips/*.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
---
 arch/mips/Kbuild | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/mips/Kbuild b/arch/mips/Kbuild
index 5c3f688a5232..5cef58651db0 100644
--- a/arch/mips/Kbuild
+++ b/arch/mips/Kbuild
@@ -1,7 +1,9 @@
 # Fail on warnings - also for files referenced in subdirs
 # -Werror can be disabled for specific files using:
 # CFLAGS_<file.o> := -Wno-error
+ifeq ($(W),)
 subdir-ccflags-y := -Werror
+endif
 
 # platform specific definitions
 include arch/mips/Kbuild.platforms
-- 
2.9.3

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] MIPS: Disable Werror when W= is set
  2017-03-17  1:06 [PATCH] MIPS: Disable Werror when W= is set Florian Fainelli
@ 2017-03-21 19:27 ` Ralf Baechle
  0 siblings, 0 replies; 2+ messages in thread
From: Ralf Baechle @ 2017-03-21 19:27 UTC (permalink / raw)
  To: Florian Fainelli; +Cc: linux-mips, james.hogan

On Thu, Mar 16, 2017 at 06:06:12PM -0700, Florian Fainelli wrote:

> Using any value for W= will lead to a ton of warnings which are turned
> into fatal errors because MIPS adds -Werror to arch/mips/*.

Imho a fairly reasonable way of dealing with the complicatios of by -Werror.

Applied,

  Ralf

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-03-21 19:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-17  1:06 [PATCH] MIPS: Disable Werror when W= is set Florian Fainelli
2017-03-21 19:27 ` Ralf Baechle

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.