8ef2f7c77c
coreboot adds a patch on top of upstream Kconfig which allows the generated Kconfig dependency files to be placed in a separate directory than the autoconfig files based on the KCONFIG_SPLITCONFIG variable. Add a comment to explain this difference. Change-Id: Ief38ab84f852ff24f896ec8bbf094aa737a172d9 Signed-off-by: Nicholas Chin <nic.c3.14@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/69952 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com> Reviewed-by: Paul Menzel <paulepanter@mailbox.org> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Reviewed-by: Martin L Roth <gaumless@gmail.com>
66 lines
2.1 KiB
Diff
66 lines
2.1 KiB
Diff
From 2a44aa33d6506a29b14d9c429410ad582d1d6636 Mon Sep 17 00:00:00 2001
|
|
From: Martin Roth <gaumless@gmail.com>
|
|
Date: Sun, 13 Nov 2022 15:20:57 -0700
|
|
Subject: [PATCH] util/kconfig: Move Kconfig deps back into build/config
|
|
|
|
revert commit 1b9e740a8 (kconfig: fix failing to generate auto.conf) [1]
|
|
|
|
The above change caused all of the enabled kconfig options to be written
|
|
into the top level build directory. We don't want that, so go back to
|
|
the old behavior for the coreboot tree.
|
|
|
|
[1] https://web.archive.org/web/20220316120807/https://github.com/torvalds/linux/commit/1b9e740a81f91ae338b29ed70455719804957b80
|
|
|
|
Signed-off-by: Martin Roth <gaumless@gmail.com>
|
|
Change-Id: I2904f69a5d85337ad0a6b48590ccd4b4a6e38b70
|
|
---
|
|
util/kconfig/confdata.c | 20 +++++++++++---------
|
|
1 file changed, 11 insertions(+), 9 deletions(-)
|
|
|
|
Index: kconfig/confdata.c
|
|
===================================================================
|
|
--- kconfig.orig/confdata.c
|
|
+++ kconfig/confdata.c
|
|
@@ -230,6 +230,13 @@ static const char *conf_get_autoheader_n
|
|
return name ? name : "include/generated/autoconf.h";
|
|
}
|
|
|
|
+static const char *conf_get_autobase_name(void)
|
|
+{
|
|
+ char *name = getenv("KCONFIG_SPLITCONFIG");
|
|
+
|
|
+ return name ? name : "include/config/";
|
|
+}
|
|
+
|
|
static int conf_set_sym_val(struct symbol *sym, int def, int def_flags, char *p)
|
|
{
|
|
char *p2;
|
|
@@ -1024,19 +1031,19 @@ static int conf_write_autoconf_cmd(const
|
|
|
|
static int conf_touch_deps(void)
|
|
{
|
|
- const char *name, *tmp;
|
|
+ const char *name;
|
|
struct symbol *sym;
|
|
int res, i;
|
|
|
|
- name = conf_get_autoconfig_name();
|
|
- tmp = strrchr(name, '/');
|
|
- depfile_prefix_len = tmp ? tmp - name + 1 : 0;
|
|
- if (depfile_prefix_len + 1 > sizeof(depfile_path))
|
|
- return -1;
|
|
-
|
|
- strncpy(depfile_path, name, depfile_prefix_len);
|
|
- depfile_path[depfile_prefix_len] = 0;
|
|
+ /*
|
|
+ * Upstream Kconfig sets depfile_path based on the directory
|
|
+ * prefix of the autoconfig path, but coreboot overrides this
|
|
+ * using the KCONFIG_SPLITCONFIG environment variable
|
|
+ */
|
|
+ strcpy(depfile_path, conf_get_autobase_name());
|
|
+ depfile_prefix_len = strlen(depfile_path);
|
|
|
|
+ name = conf_get_autoconfig_name();
|
|
conf_read_simple(name, S_DEF_AUTO);
|
|
sym_calc_value(modules_sym);
|
|
|