* [PATCH 1/2] MIPS: Kbuild: remove -Werror
@ 2012-04-27 13:48 Artem Bityutskiy
2012-04-27 13:48 ` [PATCH 2/2] MIPS: bcm63xx: kbuild: " Artem Bityutskiy
2012-04-27 17:03 ` [PATCH 1/2] MIPS: Kbuild: " David Daney
0 siblings, 2 replies; 20+ messages in thread
From: Artem Bityutskiy @ 2012-04-27 13:48 UTC (permalink / raw)
To: Ralf Baechle; +Cc: MIPS Mailing List, MTD Maling List
From: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
MIPS build fails with the standard W=1 Kbuild switch with because of the
-Werror gcc switch.
This patch removes the gcc switch to make W=1 work. Mips is the only
architecture I know which does not build with W=1 and this upsets my aiaiai
scripts. And in general, you never know which warnings newer versions of gcc
will start emiting so having -Werror by default is not the best idea.
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
---
arch/mips/Kbuild | 5 -----
1 files changed, 0 insertions(+), 5 deletions(-)
diff --git a/arch/mips/Kbuild b/arch/mips/Kbuild
index 7dd65cf..0d37730 100644
--- a/arch/mips/Kbuild
+++ b/arch/mips/Kbuild
@@ -1,8 +1,3 @@
-# Fail on warnings - also for files referenced in subdirs
-# -Werror can be disabled for specific files using:
-# CFLAGS_<file.o> := -Wno-error
-subdir-ccflags-y := -Werror
-
# platform specific definitions
include arch/mips/Kbuild.platforms
obj-y := $(platform-y)
--
1.7.9.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* [PATCH 2/2] MIPS: bcm63xx: kbuild: remove -Werror
2012-04-27 13:48 [PATCH 1/2] MIPS: Kbuild: remove -Werror Artem Bityutskiy
@ 2012-04-27 13:48 ` Artem Bityutskiy
2012-05-06 7:06 ` Maciej W. Rozycki
2012-05-21 18:15 ` Ralf Baechle
2012-04-27 17:03 ` [PATCH 1/2] MIPS: Kbuild: " David Daney
1 sibling, 2 replies; 20+ messages in thread
From: Artem Bityutskiy @ 2012-04-27 13:48 UTC (permalink / raw)
To: Ralf Baechle; +Cc: MIPS Mailing List, MTD Maling List
From: Artem Bityutskiy <Artem.Bityutskiy@linux.intel.com>
I cannot build bcm963xx with the standard Kbuild W=1 switch:
arch/mips/bcm63xx/boards/board_bcm963xx.c: At top level:
arch/mips/bcm63xx/boards/board_bcm963xx.c:647:5: error: no previous prototype for 'bcm63xx_get_fallback_sprom' [-Werror=missing-prototypes]
cc1: all warnings being treated as errors
This patch removes the gcc switch to make W=1 work. Mips is the only
architecture I know which does not build with W=1 and this upsets my aiaiai
scripts. And in general, you never know which warnings newer versions of gcc
will start emiting so having -Werror by default is not the best idea.
Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
---
arch/mips/bcm63xx/boards/Makefile | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/arch/mips/bcm63xx/boards/Makefile b/arch/mips/bcm63xx/boards/Makefile
index 9f64fb4..af07c1a 100644
--- a/arch/mips/bcm63xx/boards/Makefile
+++ b/arch/mips/bcm63xx/boards/Makefile
@@ -1,3 +1 @@
obj-$(CONFIG_BOARD_BCM963XX) += board_bcm963xx.o
-
-ccflags-y := -Werror
--
1.7.9.1
^ permalink raw reply related [flat|nested] 20+ messages in thread
* Re: [PATCH 1/2] MIPS: Kbuild: remove -Werror
2012-04-27 13:48 [PATCH 1/2] MIPS: Kbuild: remove -Werror Artem Bityutskiy
2012-04-27 13:48 ` [PATCH 2/2] MIPS: bcm63xx: kbuild: " Artem Bityutskiy
@ 2012-04-27 17:03 ` David Daney
2012-05-06 7:04 ` Maciej W. Rozycki
1 sibling, 1 reply; 20+ messages in thread
From: David Daney @ 2012-04-27 17:03 UTC (permalink / raw)
To: Artem Bityutskiy, Ralf Baechle; +Cc: MIPS Mailing List, MTD Maling List
On 04/27/2012 06:48 AM, Artem Bityutskiy wrote:
> From: Artem Bityutskiy<artem.bityutskiy@linux.intel.com>
>
> MIPS build fails with the standard W=1 Kbuild switch with because of the
> -Werror gcc switch.
>
> This patch removes the gcc switch to make W=1 work. Mips is the only
> architecture I know which does not build with W=1 and this upsets my aiaiai
> scripts. And in general, you never know which warnings newer versions of gcc
> will start emiting so having -Werror by default is not the best idea.
>
> Signed-off-by: Artem Bityutskiy<artem.bityutskiy@linux.intel.com>
I think the warning messages are enough, we don't need to break things.
Acked-by: David Daney <david.daney@cavium.com>
> ---
> arch/mips/Kbuild | 5 -----
> 1 files changed, 0 insertions(+), 5 deletions(-)
>
> diff --git a/arch/mips/Kbuild b/arch/mips/Kbuild
> index 7dd65cf..0d37730 100644
> --- a/arch/mips/Kbuild
> +++ b/arch/mips/Kbuild
> @@ -1,8 +1,3 @@
> -# Fail on warnings - also for files referenced in subdirs
> -# -Werror can be disabled for specific files using:
> -# CFLAGS_<file.o> := -Wno-error
> -subdir-ccflags-y := -Werror
> -
> # platform specific definitions
> include arch/mips/Kbuild.platforms
> obj-y := $(platform-y)
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 1/2] MIPS: Kbuild: remove -Werror
2012-04-27 17:03 ` [PATCH 1/2] MIPS: Kbuild: " David Daney
@ 2012-05-06 7:04 ` Maciej W. Rozycki
0 siblings, 0 replies; 20+ messages in thread
From: Maciej W. Rozycki @ 2012-05-06 7:04 UTC (permalink / raw)
To: David Daney
Cc: Artem Bityutskiy, Ralf Baechle, MIPS Mailing List, MTD Maling List
On Fri, 27 Apr 2012, David Daney wrote:
> > MIPS build fails with the standard W=1 Kbuild switch with because of the
> > -Werror gcc switch.
> >
> > This patch removes the gcc switch to make W=1 work. Mips is the only
> > architecture I know which does not build with W=1 and this upsets my aiaiai
> > scripts. And in general, you never know which warnings newer versions of gcc
> > will start emiting so having -Werror by default is not the best idea.
> >
> > Signed-off-by: Artem Bityutskiy<artem.bityutskiy@linux.intel.com>
>
> I think the warning messages are enough, we don't need to break things.
I disagree. People generally don't fix their broken code just because it
triggers warnings. The cases where GCC is genuinely confused are the
minority -- and even if so, chances are the human reader of that code will
also be.
Maciej
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 1/2] MIPS: Kbuild: remove -Werror
@ 2012-05-06 7:04 ` Maciej W. Rozycki
0 siblings, 0 replies; 20+ messages in thread
From: Maciej W. Rozycki @ 2012-05-06 7:04 UTC (permalink / raw)
To: David Daney
Cc: MIPS Mailing List, MTD Maling List, Ralf Baechle, Artem Bityutskiy
On Fri, 27 Apr 2012, David Daney wrote:
> > MIPS build fails with the standard W=1 Kbuild switch with because of the
> > -Werror gcc switch.
> >
> > This patch removes the gcc switch to make W=1 work. Mips is the only
> > architecture I know which does not build with W=1 and this upsets my aiaiai
> > scripts. And in general, you never know which warnings newer versions of gcc
> > will start emiting so having -Werror by default is not the best idea.
> >
> > Signed-off-by: Artem Bityutskiy<artem.bityutskiy@linux.intel.com>
>
> I think the warning messages are enough, we don't need to break things.
I disagree. People generally don't fix their broken code just because it
triggers warnings. The cases where GCC is genuinely confused are the
minority -- and even if so, chances are the human reader of that code will
also be.
Maciej
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/2] MIPS: bcm63xx: kbuild: remove -Werror
2012-04-27 13:48 ` [PATCH 2/2] MIPS: bcm63xx: kbuild: " Artem Bityutskiy
@ 2012-05-06 7:06 ` Maciej W. Rozycki
2012-05-21 18:15 ` Ralf Baechle
1 sibling, 0 replies; 20+ messages in thread
From: Maciej W. Rozycki @ 2012-05-06 7:06 UTC (permalink / raw)
To: Artem Bityutskiy; +Cc: Ralf Baechle, MIPS Mailing List, MTD Maling List
On Fri, 27 Apr 2012, Artem Bityutskiy wrote:
> From: Artem Bityutskiy <Artem.Bityutskiy@linux.intel.com>
>
> I cannot build bcm963xx with the standard Kbuild W=1 switch:
>
> arch/mips/bcm63xx/boards/board_bcm963xx.c: At top level:
> arch/mips/bcm63xx/boards/board_bcm963xx.c:647:5: error: no previous prototype for 'bcm63xx_get_fallback_sprom' [-Werror=missing-prototypes]
> cc1: all warnings being treated as errors
>
> This patch removes the gcc switch to make W=1 work. Mips is the only
> architecture I know which does not build with W=1 and this upsets my aiaiai
> scripts. And in general, you never know which warnings newer versions of gcc
> will start emiting so having -Werror by default is not the best idea.
If the function has no prototype, then it cannot be reasonably used from
outside -- perhaps you meant to mark it static instead?
Maciej
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/2] MIPS: bcm63xx: kbuild: remove -Werror
@ 2012-05-06 7:06 ` Maciej W. Rozycki
0 siblings, 0 replies; 20+ messages in thread
From: Maciej W. Rozycki @ 2012-05-06 7:06 UTC (permalink / raw)
To: Artem Bityutskiy; +Cc: MIPS Mailing List, MTD Maling List, Ralf Baechle
On Fri, 27 Apr 2012, Artem Bityutskiy wrote:
> From: Artem Bityutskiy <Artem.Bityutskiy@linux.intel.com>
>
> I cannot build bcm963xx with the standard Kbuild W=1 switch:
>
> arch/mips/bcm63xx/boards/board_bcm963xx.c: At top level:
> arch/mips/bcm63xx/boards/board_bcm963xx.c:647:5: error: no previous prototype for 'bcm63xx_get_fallback_sprom' [-Werror=missing-prototypes]
> cc1: all warnings being treated as errors
>
> This patch removes the gcc switch to make W=1 work. Mips is the only
> architecture I know which does not build with W=1 and this upsets my aiaiai
> scripts. And in general, you never know which warnings newer versions of gcc
> will start emiting so having -Werror by default is not the best idea.
If the function has no prototype, then it cannot be reasonably used from
outside -- perhaps you meant to mark it static instead?
Maciej
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 1/2] MIPS: Kbuild: remove -Werror
2012-05-06 7:04 ` Maciej W. Rozycki
@ 2012-05-06 7:34 ` Artem Bityutskiy
-1 siblings, 0 replies; 20+ messages in thread
From: Artem Bityutskiy @ 2012-05-06 7:34 UTC (permalink / raw)
To: Maciej W. Rozycki
Cc: David Daney, Ralf Baechle, MIPS Mailing List, MTD Maling List
[-- Attachment #1: Type: text/plain, Size: 1111 bytes --]
On Sun, 2012-05-06 at 08:04 +0100, Maciej W. Rozycki wrote:
> On Fri, 27 Apr 2012, David Daney wrote:
>
> > > MIPS build fails with the standard W=1 Kbuild switch with because of the
> > > -Werror gcc switch.
> > >
> > > This patch removes the gcc switch to make W=1 work. Mips is the only
> > > architecture I know which does not build with W=1 and this upsets my aiaiai
> > > scripts. And in general, you never know which warnings newer versions of gcc
> > > will start emiting so having -Werror by default is not the best idea.
> > >
> > > Signed-off-by: Artem Bityutskiy<artem.bityutskiy@linux.intel.com>
> >
> > I think the warning messages are enough, we don't need to break things.
>
> I disagree. People generally don't fix their broken code just because it
> triggers warnings. The cases where GCC is genuinely confused are the
> minority -- and even if so, chances are the human reader of that code will
> also be.
Aggressive opinion, nothing more. A patch which fixes the real issue a
better way would be way more respectful.
--
Best Regards,
Artem Bityutskiy
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 1/2] MIPS: Kbuild: remove -Werror
@ 2012-05-06 7:34 ` Artem Bityutskiy
0 siblings, 0 replies; 20+ messages in thread
From: Artem Bityutskiy @ 2012-05-06 7:34 UTC (permalink / raw)
To: Maciej W. Rozycki
Cc: MIPS Mailing List, MTD Maling List, Ralf Baechle, David Daney
[-- Attachment #1: Type: text/plain, Size: 1111 bytes --]
On Sun, 2012-05-06 at 08:04 +0100, Maciej W. Rozycki wrote:
> On Fri, 27 Apr 2012, David Daney wrote:
>
> > > MIPS build fails with the standard W=1 Kbuild switch with because of the
> > > -Werror gcc switch.
> > >
> > > This patch removes the gcc switch to make W=1 work. Mips is the only
> > > architecture I know which does not build with W=1 and this upsets my aiaiai
> > > scripts. And in general, you never know which warnings newer versions of gcc
> > > will start emiting so having -Werror by default is not the best idea.
> > >
> > > Signed-off-by: Artem Bityutskiy<artem.bityutskiy@linux.intel.com>
> >
> > I think the warning messages are enough, we don't need to break things.
>
> I disagree. People generally don't fix their broken code just because it
> triggers warnings. The cases where GCC is genuinely confused are the
> minority -- and even if so, chances are the human reader of that code will
> also be.
Aggressive opinion, nothing more. A patch which fixes the real issue a
better way would be way more respectful.
--
Best Regards,
Artem Bityutskiy
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/2] MIPS: bcm63xx: kbuild: remove -Werror
2012-05-06 7:06 ` Maciej W. Rozycki
@ 2012-05-06 7:36 ` Artem Bityutskiy
-1 siblings, 0 replies; 20+ messages in thread
From: Artem Bityutskiy @ 2012-05-06 7:36 UTC (permalink / raw)
To: Maciej W. Rozycki; +Cc: Ralf Baechle, MIPS Mailing List, MTD Maling List
[-- Attachment #1: Type: text/plain, Size: 1067 bytes --]
On Sun, 2012-05-06 at 08:06 +0100, Maciej W. Rozycki wrote:
> On Fri, 27 Apr 2012, Artem Bityutskiy wrote:
>
> > From: Artem Bityutskiy <Artem.Bityutskiy@linux.intel.com>
> >
> > I cannot build bcm963xx with the standard Kbuild W=1 switch:
> >
> > arch/mips/bcm63xx/boards/board_bcm963xx.c: At top level:
> > arch/mips/bcm63xx/boards/board_bcm963xx.c:647:5: error: no previous prototype for 'bcm63xx_get_fallback_sprom' [-Werror=missing-prototypes]
> > cc1: all warnings being treated as errors
> >
> > This patch removes the gcc switch to make W=1 work. Mips is the only
> > architecture I know which does not build with W=1 and this upsets my aiaiai
> > scripts. And in general, you never know which warnings newer versions of gcc
> > will start emiting so having -Werror by default is not the best idea.
>
> If the function has no prototype, then it cannot be reasonably used from
> outside -- perhaps you meant to mark it static instead?
No, I meant what I meant - MIPS does not build with W=1.
--
Best Regards,
Artem Bityutskiy
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/2] MIPS: bcm63xx: kbuild: remove -Werror
@ 2012-05-06 7:36 ` Artem Bityutskiy
0 siblings, 0 replies; 20+ messages in thread
From: Artem Bityutskiy @ 2012-05-06 7:36 UTC (permalink / raw)
To: Maciej W. Rozycki; +Cc: MIPS Mailing List, MTD Maling List, Ralf Baechle
[-- Attachment #1: Type: text/plain, Size: 1067 bytes --]
On Sun, 2012-05-06 at 08:06 +0100, Maciej W. Rozycki wrote:
> On Fri, 27 Apr 2012, Artem Bityutskiy wrote:
>
> > From: Artem Bityutskiy <Artem.Bityutskiy@linux.intel.com>
> >
> > I cannot build bcm963xx with the standard Kbuild W=1 switch:
> >
> > arch/mips/bcm63xx/boards/board_bcm963xx.c: At top level:
> > arch/mips/bcm63xx/boards/board_bcm963xx.c:647:5: error: no previous prototype for 'bcm63xx_get_fallback_sprom' [-Werror=missing-prototypes]
> > cc1: all warnings being treated as errors
> >
> > This patch removes the gcc switch to make W=1 work. Mips is the only
> > architecture I know which does not build with W=1 and this upsets my aiaiai
> > scripts. And in general, you never know which warnings newer versions of gcc
> > will start emiting so having -Werror by default is not the best idea.
>
> If the function has no prototype, then it cannot be reasonably used from
> outside -- perhaps you meant to mark it static instead?
No, I meant what I meant - MIPS does not build with W=1.
--
Best Regards,
Artem Bityutskiy
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 1/2] MIPS: Kbuild: remove -Werror
2012-05-06 7:34 ` Artem Bityutskiy
@ 2012-05-06 8:32 ` Maciej W. Rozycki
-1 siblings, 0 replies; 20+ messages in thread
From: Maciej W. Rozycki @ 2012-05-06 8:32 UTC (permalink / raw)
To: Artem Bityutskiy
Cc: David Daney, Ralf Baechle, MIPS Mailing List, MTD Maling List
On Sun, 6 May 2012, Artem Bityutskiy wrote:
> > I disagree. People generally don't fix their broken code just because it
> > triggers warnings. The cases where GCC is genuinely confused are the
> > minority -- and even if so, chances are the human reader of that code will
> > also be.
>
> Aggressive opinion, nothing more. A patch which fixes the real issue a
> better way would be way more respectful.
I didn't trigger the issue, so I have nothing to fix. I would if it was
me who did.
And my opinion is based on experience. Please check the LMO archives for
why Ralf added this option in the first place -- many years ago. It's
probably recorded in the git repository too (I'm not sure if the option
was added before or after we moved away from CVS, but in any case old
change logs have been imported when our current repo was created).
Maciej
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 1/2] MIPS: Kbuild: remove -Werror
@ 2012-05-06 8:32 ` Maciej W. Rozycki
0 siblings, 0 replies; 20+ messages in thread
From: Maciej W. Rozycki @ 2012-05-06 8:32 UTC (permalink / raw)
To: Artem Bityutskiy
Cc: MIPS Mailing List, MTD Maling List, Ralf Baechle, David Daney
On Sun, 6 May 2012, Artem Bityutskiy wrote:
> > I disagree. People generally don't fix their broken code just because it
> > triggers warnings. The cases where GCC is genuinely confused are the
> > minority -- and even if so, chances are the human reader of that code will
> > also be.
>
> Aggressive opinion, nothing more. A patch which fixes the real issue a
> better way would be way more respectful.
I didn't trigger the issue, so I have nothing to fix. I would if it was
me who did.
And my opinion is based on experience. Please check the LMO archives for
why Ralf added this option in the first place -- many years ago. It's
probably recorded in the git repository too (I'm not sure if the option
was added before or after we moved away from CVS, but in any case old
change logs have been imported when our current repo was created).
Maciej
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 1/2] MIPS: Kbuild: remove -Werror
2012-05-06 8:32 ` Maciej W. Rozycki
@ 2012-05-06 8:37 ` Artem Bityutskiy
-1 siblings, 0 replies; 20+ messages in thread
From: Artem Bityutskiy @ 2012-05-06 8:37 UTC (permalink / raw)
To: Maciej W. Rozycki
Cc: David Daney, Ralf Baechle, MIPS Mailing List, MTD Maling List
[-- Attachment #1: Type: text/plain, Size: 566 bytes --]
On Sun, 2012-05-06 at 09:32 +0100, Maciej W. Rozycki wrote:
> And my opinion is based on experience. Please check the LMO archives for
> why Ralf added this option in the first place -- many years ago. It's
> probably recorded in the git repository too (I'm not sure if the option
> was added before or after we moved away from CVS, but in any case old
> change logs have been imported when our current repo was created).
We need to figure out how to make -Werror be applied only when we do not
have W=[123].
--
Best Regards,
Artem Bityutskiy
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 1/2] MIPS: Kbuild: remove -Werror
@ 2012-05-06 8:37 ` Artem Bityutskiy
0 siblings, 0 replies; 20+ messages in thread
From: Artem Bityutskiy @ 2012-05-06 8:37 UTC (permalink / raw)
To: Maciej W. Rozycki
Cc: MIPS Mailing List, MTD Maling List, Ralf Baechle, David Daney
[-- Attachment #1: Type: text/plain, Size: 566 bytes --]
On Sun, 2012-05-06 at 09:32 +0100, Maciej W. Rozycki wrote:
> And my opinion is based on experience. Please check the LMO archives for
> why Ralf added this option in the first place -- many years ago. It's
> probably recorded in the git repository too (I'm not sure if the option
> was added before or after we moved away from CVS, but in any case old
> change logs have been imported when our current repo was created).
We need to figure out how to make -Werror be applied only when we do not
have W=[123].
--
Best Regards,
Artem Bityutskiy
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 1/2] MIPS: Kbuild: remove -Werror
2012-05-06 8:37 ` Artem Bityutskiy
@ 2012-05-06 9:14 ` Maciej W. Rozycki
-1 siblings, 0 replies; 20+ messages in thread
From: Maciej W. Rozycki @ 2012-05-06 9:14 UTC (permalink / raw)
To: Artem Bityutskiy
Cc: David Daney, Ralf Baechle, MIPS Mailing List, MTD Maling List
On Sun, 6 May 2012, Artem Bityutskiy wrote:
> > And my opinion is based on experience. Please check the LMO archives for
> > why Ralf added this option in the first place -- many years ago. It's
> > probably recorded in the git repository too (I'm not sure if the option
> > was added before or after we moved away from CVS, but in any case old
> > change logs have been imported when our current repo was created).
>
> We need to figure out how to make -Werror be applied only when we do not
> have W=[123].
Hmm, that sounds better, however has the counter-intuitive side-effect of
lowering the severity of the warnings that are enabled even without
W=[123].
Modern versions of GCC have that selective -Wno-error=foo option and I
think it should be possible to build the precise list of warnings not to
choke on locally in arch/mips/Kbuild with little Makefile magic, falling
back to something sane for older GCC versions (I'm not sure exactly when
these selective options were added, certainly sometime between 4.1 and
4.3).
This will be a bit imperfect if any of these additional -Wfoo options
duplicate ones already added to KBUILD_CFLAGS in our top-level Makefile
(either explicitly or via -Wall), but that's about the best we can do.
I'll see if I can cook up something quickly.
Maciej
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 1/2] MIPS: Kbuild: remove -Werror
@ 2012-05-06 9:14 ` Maciej W. Rozycki
0 siblings, 0 replies; 20+ messages in thread
From: Maciej W. Rozycki @ 2012-05-06 9:14 UTC (permalink / raw)
To: Artem Bityutskiy
Cc: MIPS Mailing List, MTD Maling List, Ralf Baechle, David Daney
On Sun, 6 May 2012, Artem Bityutskiy wrote:
> > And my opinion is based on experience. Please check the LMO archives for
> > why Ralf added this option in the first place -- many years ago. It's
> > probably recorded in the git repository too (I'm not sure if the option
> > was added before or after we moved away from CVS, but in any case old
> > change logs have been imported when our current repo was created).
>
> We need to figure out how to make -Werror be applied only when we do not
> have W=[123].
Hmm, that sounds better, however has the counter-intuitive side-effect of
lowering the severity of the warnings that are enabled even without
W=[123].
Modern versions of GCC have that selective -Wno-error=foo option and I
think it should be possible to build the precise list of warnings not to
choke on locally in arch/mips/Kbuild with little Makefile magic, falling
back to something sane for older GCC versions (I'm not sure exactly when
these selective options were added, certainly sometime between 4.1 and
4.3).
This will be a bit imperfect if any of these additional -Wfoo options
duplicate ones already added to KBUILD_CFLAGS in our top-level Makefile
(either explicitly or via -Wall), but that's about the best we can do.
I'll see if I can cook up something quickly.
Maciej
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 1/2] MIPS: Kbuild: remove -Werror
2012-05-06 9:14 ` Maciej W. Rozycki
@ 2012-05-13 12:08 ` Jonas Gorski
-1 siblings, 0 replies; 20+ messages in thread
From: Jonas Gorski @ 2012-05-13 12:08 UTC (permalink / raw)
To: Maciej W. Rozycki
Cc: Artem Bityutskiy, David Daney, Ralf Baechle, MIPS Mailing List,
MTD Maling List
Hi,
On 6 May 2012 11:14, Maciej W. Rozycki <macro@linux-mips.org> wrote:
> On Sun, 6 May 2012, Artem Bityutskiy wrote:
>
>> > And my opinion is based on experience. Please check the LMO archives for
>> > why Ralf added this option in the first place -- many years ago. It's
>> > probably recorded in the git repository too (I'm not sure if the option
>> > was added before or after we moved away from CVS, but in any case old
>> > change logs have been imported when our current repo was created).
>>
>> We need to figure out how to make -Werror be applied only when we do not
>> have W=[123].
>
> Hmm, that sounds better, however has the counter-intuitive side-effect of
> lowering the severity of the warnings that are enabled even without
> W=[123].
>
> Modern versions of GCC have that selective -Wno-error=foo option and I
> think it should be possible to build the precise list of warnings not to
> choke on locally in arch/mips/Kbuild with little Makefile magic, falling
> back to something sane for older GCC versions (I'm not sure exactly when
> these selective options were added, certainly sometime between 4.1 and
> 4.3).
>
> This will be a bit imperfect if any of these additional -Wfoo options
> duplicate ones already added to KBUILD_CFLAGS in our top-level Makefile
> (either explicitly or via -Wall), but that's about the best we can do.
> I'll see if I can cook up something quickly.
Hm, how about doing it the other way round, i.e. explicitly enable
which warnings we want to treat as errors with -Werror=foo? That way
we don't lower the severity when W=[123] and new default enabled
warnings in gcc can't break the build any more, just better (or worse
;-) heuristics can.
Jonas
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 1/2] MIPS: Kbuild: remove -Werror
@ 2012-05-13 12:08 ` Jonas Gorski
0 siblings, 0 replies; 20+ messages in thread
From: Jonas Gorski @ 2012-05-13 12:08 UTC (permalink / raw)
To: Maciej W. Rozycki
Cc: MIPS Mailing List, MTD Maling List, Ralf Baechle, David Daney,
Artem Bityutskiy
Hi,
On 6 May 2012 11:14, Maciej W. Rozycki <macro@linux-mips.org> wrote:
> On Sun, 6 May 2012, Artem Bityutskiy wrote:
>
>> > And my opinion is based on experience. Please check the LMO archives for
>> > why Ralf added this option in the first place -- many years ago. It's
>> > probably recorded in the git repository too (I'm not sure if the option
>> > was added before or after we moved away from CVS, but in any case old
>> > change logs have been imported when our current repo was created).
>>
>> We need to figure out how to make -Werror be applied only when we do not
>> have W=[123].
>
> Hmm, that sounds better, however has the counter-intuitive side-effect of
> lowering the severity of the warnings that are enabled even without
> W=[123].
>
> Modern versions of GCC have that selective -Wno-error=foo option and I
> think it should be possible to build the precise list of warnings not to
> choke on locally in arch/mips/Kbuild with little Makefile magic, falling
> back to something sane for older GCC versions (I'm not sure exactly when
> these selective options were added, certainly sometime between 4.1 and
> 4.3).
>
> This will be a bit imperfect if any of these additional -Wfoo options
> duplicate ones already added to KBUILD_CFLAGS in our top-level Makefile
> (either explicitly or via -Wall), but that's about the best we can do.
> I'll see if I can cook up something quickly.
Hm, how about doing it the other way round, i.e. explicitly enable
which warnings we want to treat as errors with -Werror=foo? That way
we don't lower the severity when W=[123] and new default enabled
warnings in gcc can't break the build any more, just better (or worse
;-) heuristics can.
Jonas
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH 2/2] MIPS: bcm63xx: kbuild: remove -Werror
2012-04-27 13:48 ` [PATCH 2/2] MIPS: bcm63xx: kbuild: " Artem Bityutskiy
2012-05-06 7:06 ` Maciej W. Rozycki
@ 2012-05-21 18:15 ` Ralf Baechle
1 sibling, 0 replies; 20+ messages in thread
From: Ralf Baechle @ 2012-05-21 18:15 UTC (permalink / raw)
To: Artem Bityutskiy; +Cc: MIPS Mailing List, MTD Maling List
On Fri, Apr 27, 2012 at 04:48:30PM +0300, Artem Bityutskiy wrote:
> From: Artem Bityutskiy <Artem.Bityutskiy@linux.intel.com>
>
> I cannot build bcm963xx with the standard Kbuild W=1 switch:
>
> arch/mips/bcm63xx/boards/board_bcm963xx.c: At top level:
> arch/mips/bcm63xx/boards/board_bcm963xx.c:647:5: error: no previous prototype for 'bcm63xx_get_fallback_sprom' [-Werror=missing-prototypes]
> cc1: all warnings being treated as errors
>
> This patch removes the gcc switch to make W=1 work. Mips is the only
> architecture I know which does not build with W=1 and this upsets my aiaiai
> scripts. And in general, you never know which warnings newer versions of gcc
> will start emiting so having -Werror by default is not the best idea.
>
> Signed-off-by: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
> ---
> arch/mips/bcm63xx/boards/Makefile | 2 --
> 1 files changed, 0 insertions(+), 2 deletions(-)
>
> diff --git a/arch/mips/bcm63xx/boards/Makefile b/arch/mips/bcm63xx/boards/Makefile
> index 9f64fb4..af07c1a 100644
> --- a/arch/mips/bcm63xx/boards/Makefile
> +++ b/arch/mips/bcm63xx/boards/Makefile
> @@ -1,3 +1 @@
> obj-$(CONFIG_BOARD_BCM963XX) += board_bcm963xx.o
> -
> -ccflags-y := -Werror
There's been a whole bunch of other -Werrors below arch/mips some of
which even were combined with -Wall. I removed all off them except the
central -Werror in arch/mips/Kbuild. I'm pondering a better solution
for that one now.
Ralf
^ permalink raw reply [flat|nested] 20+ messages in thread
end of thread, other threads:[~2012-05-21 18:15 UTC | newest]
Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-27 13:48 [PATCH 1/2] MIPS: Kbuild: remove -Werror Artem Bityutskiy
2012-04-27 13:48 ` [PATCH 2/2] MIPS: bcm63xx: kbuild: " Artem Bityutskiy
2012-05-06 7:06 ` Maciej W. Rozycki
2012-05-06 7:06 ` Maciej W. Rozycki
2012-05-06 7:36 ` Artem Bityutskiy
2012-05-06 7:36 ` Artem Bityutskiy
2012-05-21 18:15 ` Ralf Baechle
2012-04-27 17:03 ` [PATCH 1/2] MIPS: Kbuild: " David Daney
2012-05-06 7:04 ` Maciej W. Rozycki
2012-05-06 7:04 ` Maciej W. Rozycki
2012-05-06 7:34 ` Artem Bityutskiy
2012-05-06 7:34 ` Artem Bityutskiy
2012-05-06 8:32 ` Maciej W. Rozycki
2012-05-06 8:32 ` Maciej W. Rozycki
2012-05-06 8:37 ` Artem Bityutskiy
2012-05-06 8:37 ` Artem Bityutskiy
2012-05-06 9:14 ` Maciej W. Rozycki
2012-05-06 9:14 ` Maciej W. Rozycki
2012-05-13 12:08 ` Jonas Gorski
2012-05-13 12:08 ` Jonas Gorski
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.