New patch to review for coreboot: Kconfig: Add KCONFIG_STRICT mode

[Stefan Reinauer]

Stefan Reinauer ([e-mail]) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10977

-gerrit

commit 5161802f974c95a20ded682a9d9c984043ff7f52
Author: Stefan Reinauer 
Date:   Fri Jul 17 17:26:48 2015 -0700

Kconfig: Add KCONFIG_STRICT mode

This is basically a -Werror mode for Kconfig. When exporting
    KCONFIG_STRICT in the Makefile, warnings in Kconfig will produce
    errors instead.

This will make it easier to spot unclean Kconfig files, settings
    and dependencies.

Change-Id: I941af24c3ccb10b8b9ddc5c98327154749ebbbc6
    Signed-off-by: Stefan Reinauer 
---
 util/kconfig/confdata.c | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/util/kconfig/confdata.c b/util/kconfig/confdata.c
index 235debf..5804b29 100644
--- a/util/kconfig/confdata.c
+++ b/util/kconfig/confdata.c
@@ -358,6 +358,7 @@ load:
    		if (def == S_DEF_USER) {
    			sym = sym_find(line + strlen(CONFIG_));
    			if (!sym) {
+					conf_warning("trying to assign non-existent symbol %s", line + strlen(CONFIG_));
    				sym_add_change_count(1);
    				goto setsym;
    			}
@@ -402,6 +403,13 @@ setsym:

if (modules_sym)
    	sym_calc_value(modules_sym);
+
+	name = getenv("KCONFIG_STRICT");
+	if (name && *name && conf_warnings) {
+		fprintf(stderr, "\nERROR: %d warnings encountered, and warnings are errors.\n\n", conf_warnings);
+		return 1;
+	}
+
    return 0;
 }