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 | # ========================================================================== |
| 3 | # |
| 4 | # make W=... settings |
| 5 | # |
| 6 | # W=1 - warnings that may be relevant and does not occur too often |
| 7 | # W=2 - warnings that occur quite often but may still be relevant |
| 8 | # W=3 - the more obscure warnings, can most likely be ignored |
| 9 | # |
| 10 | # $(call cc-option, -W...) handles gcc -W.. options which |
| 11 | # are not supported by all versions of the compiler |
| 12 | # ========================================================================== |
| 13 | |
Xiongfeng Wang | 321cb03 | 2018-01-11 17:22:29 +0800 | [diff] [blame] | 14 | KBUILD_CFLAGS += $(call cc-disable-warning, packed-not-aligned) |
| 15 | |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 16 | ifeq ("$(origin W)", "command line") |
| 17 | export KBUILD_ENABLE_EXTRA_GCC_CHECKS := $(W) |
| 18 | endif |
| 19 | |
| 20 | ifdef KBUILD_ENABLE_EXTRA_GCC_CHECKS |
| 21 | warning- := $(empty) |
| 22 | |
| 23 | warning-1 := -Wextra -Wunused -Wno-unused-parameter |
| 24 | warning-1 += -Wmissing-declarations |
| 25 | warning-1 += -Wmissing-format-attribute |
| 26 | warning-1 += $(call cc-option, -Wmissing-prototypes) |
| 27 | warning-1 += -Wold-style-definition |
| 28 | warning-1 += $(call cc-option, -Wmissing-include-dirs) |
| 29 | warning-1 += $(call cc-option, -Wunused-but-set-variable) |
Arnd Bergmann | c9c6837 | 2016-05-10 23:30:01 +0200 | [diff] [blame] | 30 | warning-1 += $(call cc-option, -Wunused-const-variable) |
Xiongfeng Wang | 321cb03 | 2018-01-11 17:22:29 +0800 | [diff] [blame] | 31 | warning-1 += $(call cc-option, -Wpacked-not-aligned) |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 32 | warning-1 += $(call cc-disable-warning, missing-field-initializers) |
Lee Jones | 7599ea8 | 2016-01-12 14:24:18 +0000 | [diff] [blame] | 33 | warning-1 += $(call cc-disable-warning, sign-compare) |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 34 | |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 35 | warning-2 := -Waggregate-return |
| 36 | warning-2 += -Wcast-align |
| 37 | warning-2 += -Wdisabled-optimization |
| 38 | warning-2 += -Wnested-externs |
| 39 | warning-2 += -Wshadow |
| 40 | warning-2 += $(call cc-option, -Wlogical-op) |
| 41 | warning-2 += $(call cc-option, -Wmissing-field-initializers) |
Lee Jones | 7599ea8 | 2016-01-12 14:24:18 +0000 | [diff] [blame] | 42 | warning-2 += $(call cc-option, -Wsign-compare) |
Arnd Bergmann | a76bcf5 | 2016-11-10 17:44:44 +0100 | [diff] [blame] | 43 | warning-2 += $(call cc-option, -Wmaybe-uninitialized) |
Johannes Thumshirn | de8cf95 | 2017-08-30 16:04:13 +0200 | [diff] [blame] | 44 | warning-2 += $(call cc-option, -Wunused-macros) |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 45 | |
| 46 | warning-3 := -Wbad-function-cast |
| 47 | warning-3 += -Wcast-qual |
| 48 | warning-3 += -Wconversion |
| 49 | warning-3 += -Wpacked |
| 50 | warning-3 += -Wpadded |
| 51 | warning-3 += -Wpointer-arith |
| 52 | warning-3 += -Wredundant-decls |
| 53 | warning-3 += -Wswitch-default |
| 54 | warning-3 += $(call cc-option, -Wpacked-bitfield-compat) |
| 55 | warning-3 += $(call cc-option, -Wvla) |
| 56 | |
| 57 | warning := $(warning-$(findstring 1, $(KBUILD_ENABLE_EXTRA_GCC_CHECKS))) |
| 58 | warning += $(warning-$(findstring 2, $(KBUILD_ENABLE_EXTRA_GCC_CHECKS))) |
| 59 | warning += $(warning-$(findstring 3, $(KBUILD_ENABLE_EXTRA_GCC_CHECKS))) |
| 60 | |
| 61 | ifeq ("$(strip $(warning))","") |
| 62 | $(error W=$(KBUILD_ENABLE_EXTRA_GCC_CHECKS) is unknown) |
| 63 | endif |
| 64 | |
| 65 | KBUILD_CFLAGS += $(warning) |
Behan Webster | 26ea6bb | 2014-07-31 21:08:25 -0700 | [diff] [blame] | 66 | else |
| 67 | |
Michal Marek | 5631d9c | 2015-08-19 17:36:41 +0200 | [diff] [blame] | 68 | ifeq ($(cc-name),clang) |
Behan Webster | 26ea6bb | 2014-07-31 21:08:25 -0700 | [diff] [blame] | 69 | KBUILD_CFLAGS += $(call cc-disable-warning, initializer-overrides) |
| 70 | KBUILD_CFLAGS += $(call cc-disable-warning, unused-value) |
| 71 | KBUILD_CFLAGS += $(call cc-disable-warning, format) |
Behan Webster | 26ea6bb | 2014-07-31 21:08:25 -0700 | [diff] [blame] | 72 | KBUILD_CFLAGS += $(call cc-disable-warning, sign-compare) |
| 73 | KBUILD_CFLAGS += $(call cc-disable-warning, format-zero-length) |
| 74 | KBUILD_CFLAGS += $(call cc-disable-warning, uninitialized) |
| 75 | endif |
Masahiro Yamada | a86fe35 | 2014-04-14 18:27:10 +0900 | [diff] [blame] | 76 | endif |