summaryrefslogtreecommitdiff
path: root/config/module/coreboot/default/patches/0052-Remove-KCONFIG_WERROR.patch
blob: 1ef4162bf9f68c652ce7ae595cfa2f61ed286748 (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
From a0d7318453e1b4827175de4a848df85bff097f73 Mon Sep 17 00:00:00 2001
From: Leah Rowe <leah@libreboot.org>
Date: Mon, 21 Sep 2026 11:20:54 +0100
Subject: [PATCH 52/56] Remove KCONFIG_WERROR

Signed-off-by: Leah Rowe <leah@libreboot.org>
---
 Documentation/getting_started/kconfig.md | 3 ---
 Makefile                                 | 4 ----
 util/kconfig/Makefile                    | 4 ----
 util/kconfig/confdata.c                  | 2 --
 util/kconfig/symbol.c                    | 2 --
 5 files changed, 15 deletions(-)

diff --git a/Documentation/getting_started/kconfig.md b/Documentation/getting_started/kconfig.md
index 2917c53aae..6c4102ba17 100644
--- a/Documentation/getting_started/kconfig.md
+++ b/Documentation/getting_started/kconfig.md
@@ -99,9 +99,6 @@ included in the Linux version.
 - KCONFIG_SPLITCONFIG=”directory name for individual SYMBOL.h files”.
   coreboot sets this to $(obj)/config.
 
-- KCONFIG_WERROR=value. Define to enable warnings as errors. This is enabled
-  in coreboot, and should not be changed.
-
 #### Used only for ‘make menuconfig’
 - MENUCONFIG_MODE=single_menu.  Set to "single_menu" to enable.  All other
   values disable the option.  This makes submenus appear below the menu option
diff --git a/Makefile b/Makefile
index 6a36b5968f..35576f3734 100644
--- a/Makefile
+++ b/Makefile
@@ -31,7 +31,6 @@ KCONFIG_DEPENDENCIES := $(obj)/auto.conf.cmd
 KCONFIG_SPLITCONFIG := $(obj)/config/
 KCONFIG_TRISTATE := $(obj)/tristate.conf
 KCONFIG_NEGATIVES := 1
-KCONFIG_WERROR := 1
 KCONFIG_WARN_UNKNOWN_SYMBOLS := 1
 KCONFIG_PACKAGE := CB.Config
 KCONFIG_MAKEFILE_REAL ?= $(objk)/Makefile.real
@@ -39,9 +38,6 @@ KCONFIG_MAKEFILE_REAL ?= $(objk)/Makefile.real
 COREBOOT_EXPORTS += KCONFIG_CONFIG KCONFIG_AUTOHEADER KCONFIG_AUTOCONFIG
 COREBOOT_EXPORTS += KCONFIG_DEPENDENCIES KCONFIG_SPLITCONFIG KCONFIG_TRISTATE
 COREBOOT_EXPORTS += KCONFIG_NEGATIVES
-ifeq ($(filter %config,$(MAKECMDGOALS)),)
-COREBOOT_EXPORTS += KCONFIG_WERROR
-endif
 COREBOOT_EXPORTS += KCONFIG_WARN_UNKNOWN_SYMBOLS
 COREBOOT_EXPORTS += KCONFIG_AUTOADS KCONFIG_PACKAGE
 COREBOOT_EXPORTS += KCONFIG_RUSTCCFG
diff --git a/util/kconfig/Makefile b/util/kconfig/Makefile
index 76cc2c5e2c..45366936fc 100644
--- a/util/kconfig/Makefile
+++ b/util/kconfig/Makefile
@@ -31,10 +31,6 @@ ifneq ($(findstring c, $(KBUILD_EXTRA_WARN)),)
 export KCONFIG_WARN_UNKNOWN_SYMBOLS=1
 endif
 
-ifneq ($(findstring e, $(KBUILD_EXTRA_WARN)),)
-export KCONFIG_WERROR=1
-endif
-
 # We need this, in case the user has it in its environment
 unexport CONFIG_
 
diff --git a/util/kconfig/confdata.c b/util/kconfig/confdata.c
index ae6ae2b3d3..fa09647f44 100644
--- a/util/kconfig/confdata.c
+++ b/util/kconfig/confdata.c
@@ -157,8 +157,6 @@ static int conf_lineno, conf_warnings;
 
 bool conf_errors(void)
 {
-	if (conf_warnings)
-		return getenv("KCONFIG_WERROR");
 	return false;
 }
 
diff --git a/util/kconfig/symbol.c b/util/kconfig/symbol.c
index 3afe5be2d4..1a24d458cb 100644
--- a/util/kconfig/symbol.c
+++ b/util/kconfig/symbol.c
@@ -317,8 +317,6 @@ static void sym_warn_unmet_dep(struct symbol *sym)
 
 bool sym_dep_errors(void)
 {
-	if (sym_warnings)
-		return getenv("KCONFIG_WERROR");
 	return false;
 }
 
-- 
2.47.3