Re: [PATCH] Kbuild: enable -Wunused-macros warning for "make W=1"

2017-08-28 Thread Johannes Thumshirn
On Fri, Aug 25, 2017 at 01:19:39AM +0900, Masahiro Yamada wrote: > This makes W=1 too noisy. > > For example, drivers often define unused register macros > for completeness. I do not think it is too bad in my opinion. > > Perhaps, should it be moved to warning-2 ? > Sure, I'll send a v2 in a

Re: [PATCH] Kbuild: enable -Wunused-macros warning for "make W=1"

2017-08-28 Thread Johannes Thumshirn
On Fri, Aug 25, 2017 at 01:19:39AM +0900, Masahiro Yamada wrote: > This makes W=1 too noisy. > > For example, drivers often define unused register macros > for completeness. I do not think it is too bad in my opinion. > > Perhaps, should it be moved to warning-2 ? > Sure, I'll send a v2 in a

Re: [PATCH] Kbuild: enable -Wunused-macros warning for "make W=1"

2017-08-24 Thread Masahiro Yamada
Hi Johannes, 2017-08-03 19:09 GMT+09:00 Johannes Thumshirn : > We have lots of dead defines and macros in drivers, lets offer users a way > to detect and eventually remove them. > > Signed-off-by: Johannes Thumshirn > --- > scripts/Makefile.extrawarn | 1

Re: [PATCH] Kbuild: enable -Wunused-macros warning for "make W=1"

2017-08-24 Thread Masahiro Yamada
Hi Johannes, 2017-08-03 19:09 GMT+09:00 Johannes Thumshirn : > We have lots of dead defines and macros in drivers, lets offer users a way > to detect and eventually remove them. > > Signed-off-by: Johannes Thumshirn > --- > scripts/Makefile.extrawarn | 1 + > 1 file changed, 1 insertion(+) > >

[PATCH] Kbuild: enable -Wunused-macros warning for "make W=1"

2017-08-03 Thread Johannes Thumshirn
We have lots of dead defines and macros in drivers, lets offer users a way to detect and eventually remove them. Signed-off-by: Johannes Thumshirn --- scripts/Makefile.extrawarn | 1 + 1 file changed, 1 insertion(+) diff --git a/scripts/Makefile.extrawarn

[PATCH] Kbuild: enable -Wunused-macros warning for "make W=1"

2017-08-03 Thread Johannes Thumshirn
We have lots of dead defines and macros in drivers, lets offer users a way to detect and eventually remove them. Signed-off-by: Johannes Thumshirn --- scripts/Makefile.extrawarn | 1 + 1 file changed, 1 insertion(+) diff --git a/scripts/Makefile.extrawarn b/scripts/Makefile.extrawarn index