Greg Kroah-Hartman | b244131 | 2017-11-01 15:07:57 +0100 | [diff] [blame] | 1 | # SPDX-License-Identifier: GPL-2.0 |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 2 | # ========================================================================== |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 3 | # make W=... settings |
| 4 | # |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 5 | # There are three warning groups enabled by W=1, W=2, W=3. |
| 6 | # They are independent, and can be combined like W=12 or W=123. |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 7 | # ========================================================================== |
| 8 | |
Xiongfeng Wang | 321cb03 | 2018-01-11 17:22:29 +0800 | [diff] [blame] | 9 | KBUILD_CFLAGS += $(call cc-disable-warning, packed-not-aligned) |
| 10 | |
Masahiro Yamada | e27128d | 2019-09-01 01:25:55 +0900 | [diff] [blame] | 11 | # backward compatibility |
| 12 | KBUILD_EXTRA_WARN ?= $(KBUILD_ENABLE_EXTRA_GCC_CHECKS) |
| 13 | |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 14 | ifeq ("$(origin W)", "command line") |
Masahiro Yamada | e27128d | 2019-09-01 01:25:55 +0900 | [diff] [blame] | 15 | KBUILD_EXTRA_WARN := $(W) |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 16 | endif |
| 17 | |
Masahiro Yamada | e27128d | 2019-09-01 01:25:55 +0900 | [diff] [blame] | 18 | export KBUILD_EXTRA_WARN |
| 19 | |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 20 | # |
| 21 | # W=1 - warnings which may be relevant and do not occur too often |
| 22 | # |
Masahiro Yamada | e27128d | 2019-09-01 01:25:55 +0900 | [diff] [blame] | 23 | ifneq ($(findstring 1, $(KBUILD_EXTRA_WARN)),) |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 24 | |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 25 | KBUILD_CFLAGS += -Wextra -Wunused -Wno-unused-parameter |
| 26 | KBUILD_CFLAGS += -Wmissing-declarations |
| 27 | KBUILD_CFLAGS += -Wmissing-format-attribute |
| 28 | KBUILD_CFLAGS += -Wmissing-prototypes |
| 29 | KBUILD_CFLAGS += -Wold-style-definition |
| 30 | KBUILD_CFLAGS += -Wmissing-include-dirs |
| 31 | KBUILD_CFLAGS += $(call cc-option, -Wunused-but-set-variable) |
| 32 | KBUILD_CFLAGS += $(call cc-option, -Wunused-const-variable) |
| 33 | KBUILD_CFLAGS += $(call cc-option, -Wpacked-not-aligned) |
| 34 | KBUILD_CFLAGS += $(call cc-option, -Wstringop-truncation) |
Masahiro Yamada | 4c8dd95 | 2019-05-09 15:46:35 +0900 | [diff] [blame] | 35 | # The following turn off the warnings enabled by -Wextra |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 36 | KBUILD_CFLAGS += -Wno-missing-field-initializers |
| 37 | KBUILD_CFLAGS += -Wno-sign-compare |
Rikard Falkeborn | 355a358 | 2020-07-08 21:07:56 +0200 | [diff] [blame] | 38 | KBUILD_CFLAGS += -Wno-type-limits |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 39 | |
Masahiro Yamada | 6863f56 | 2019-09-07 11:52:36 +0900 | [diff] [blame] | 40 | KBUILD_CPPFLAGS += -DKBUILD_EXTRA_WARN1 |
| 41 | |
Behan Webster | 26ea6bb | 2014-07-31 21:08:25 -0700 | [diff] [blame] | 42 | else |
| 43 | |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 44 | # Some diagnostics enabled by default are noisy. |
| 45 | # Suppress them by using -Wno... except for W=1. |
| 46 | |
Masahiro Yamada | 076f421 | 2018-10-30 22:26:33 +0900 | [diff] [blame] | 47 | ifdef CONFIG_CC_IS_CLANG |
Masahiro Yamada | a149430 | 2019-05-10 23:10:09 +0900 | [diff] [blame] | 48 | KBUILD_CFLAGS += -Wno-initializer-overrides |
Masahiro Yamada | a149430 | 2019-05-10 23:10:09 +0900 | [diff] [blame] | 49 | KBUILD_CFLAGS += -Wno-format |
| 50 | KBUILD_CFLAGS += -Wno-sign-compare |
| 51 | KBUILD_CFLAGS += -Wno-format-zero-length |
Nathan Chancellor | 82f2bc2 | 2020-03-11 12:41:21 -0700 | [diff] [blame] | 52 | KBUILD_CFLAGS += $(call cc-disable-warning, pointer-to-enum-cast) |
Nathan Chancellor | afe956c | 2020-03-26 12:41:55 -0700 | [diff] [blame] | 53 | KBUILD_CFLAGS += -Wno-tautological-constant-out-of-range-compare |
Behan Webster | 26ea6bb | 2014-07-31 21:08:25 -0700 | [diff] [blame] | 54 | endif |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 55 | |
| 56 | endif |
| 57 | |
| 58 | # |
| 59 | # W=2 - warnings which occur quite often but may still be relevant |
| 60 | # |
Masahiro Yamada | e27128d | 2019-09-01 01:25:55 +0900 | [diff] [blame] | 61 | ifneq ($(findstring 2, $(KBUILD_EXTRA_WARN)),) |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 62 | |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 63 | KBUILD_CFLAGS += -Wdisabled-optimization |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 64 | KBUILD_CFLAGS += -Wshadow |
| 65 | KBUILD_CFLAGS += $(call cc-option, -Wlogical-op) |
| 66 | KBUILD_CFLAGS += -Wmissing-field-initializers |
Rikard Falkeborn | 355a358 | 2020-07-08 21:07:56 +0200 | [diff] [blame] | 67 | KBUILD_CFLAGS += -Wtype-limits |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 68 | KBUILD_CFLAGS += $(call cc-option, -Wmaybe-uninitialized) |
| 69 | KBUILD_CFLAGS += $(call cc-option, -Wunused-macros) |
| 70 | |
Masahiro Yamada | 6863f56 | 2019-09-07 11:52:36 +0900 | [diff] [blame] | 71 | KBUILD_CPPFLAGS += -DKBUILD_EXTRA_WARN2 |
| 72 | |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 73 | endif |
| 74 | |
| 75 | # |
| 76 | # W=3 - more obscure warnings, can most likely be ignored |
| 77 | # |
Masahiro Yamada | e27128d | 2019-09-01 01:25:55 +0900 | [diff] [blame] | 78 | ifneq ($(findstring 3, $(KBUILD_EXTRA_WARN)),) |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 79 | |
| 80 | KBUILD_CFLAGS += -Wbad-function-cast |
Arnd Bergmann | 095fbca | 2020-10-26 23:03:13 +0100 | [diff] [blame] | 81 | KBUILD_CFLAGS += -Wcast-align |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 82 | KBUILD_CFLAGS += -Wcast-qual |
| 83 | KBUILD_CFLAGS += -Wconversion |
| 84 | KBUILD_CFLAGS += -Wpacked |
| 85 | KBUILD_CFLAGS += -Wpadded |
| 86 | KBUILD_CFLAGS += -Wpointer-arith |
| 87 | KBUILD_CFLAGS += -Wredundant-decls |
Joe Perches | a97ea93 | 2020-07-21 21:57:43 -0700 | [diff] [blame] | 88 | KBUILD_CFLAGS += -Wsign-compare |
Masahiro Yamada | 64a9190 | 2019-09-01 01:25:54 +0900 | [diff] [blame] | 89 | KBUILD_CFLAGS += -Wswitch-default |
| 90 | KBUILD_CFLAGS += $(call cc-option, -Wpacked-bitfield-compat) |
| 91 | |
Masahiro Yamada | 6863f56 | 2019-09-07 11:52:36 +0900 | [diff] [blame] | 92 | KBUILD_CPPFLAGS += -DKBUILD_EXTRA_WARN3 |
| 93 | |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 94 | endif |