From: Steven Rostedt <rostedt@goodmis.org> To: linux-kernel@vger.kernel.org Cc: Ingo Molnar <mingo@elte.hu>, Andrew Morton <akpm@linux-foundation.org>, Linus Torvalds <torvalds@linux-foundation.org>, Peter Zijlstra <peterz@infradead.org>, Thomas Gleixner <tglx@linutronix.de>, Theodore Tso <tytso@mit.edu>, Arnaldo Carvalho de Melo <acme@redhat.com>, zippel@linux-m68k.org, linux-kbuild@vger.kernel.org, Sam Ravnborg <sam@ravnborg.org> Subject: [PATCH 1/7] kconfig: streamline_config.pl do not stop with no depends Date: Thu, 30 Apr 2009 14:50:23 -0400 [thread overview] Message-ID: <20090430185217.661759804@goodmis.org> (raw) In-Reply-To: <20090430185022.122124349@goodmis.org> [-- Attachment #1: 0001-kconfig-streamline_config.pl-do-not-stop-with-no-de.patch --] [-- Type: text/plain, Size: 1670 bytes --] From: Steven Rostedt <srostedt@redhat.com> If a config is does not have a prompt, it must be selected. streamline_config.pl keeps track of all configs that select other configs. If a config that does not have a prompt needs to be set to enable a current module, it will add include all configs that select it. Note, streamline_config.pl does not enable modules that are not already enabled. It only keeps enabled those that were enabled and might be needed to compile the current modules. The code to find the selects of a config is after the code that adds the depends. But if a config needed selects but had no dependencies, it would not be set. Because the code would stop before getting to the select. [ Impact: fix to enable configs that need to be selected ] Signed-off-by: Steven Rostedt <rostedt@goodmis.org> --- scripts/kconfig/streamline_config.pl | 8 +++----- 1 files changed, 3 insertions(+), 5 deletions(-) diff --git a/scripts/kconfig/streamline_config.pl b/scripts/kconfig/streamline_config.pl index 79d8557..1774905 100644 --- a/scripts/kconfig/streamline_config.pl +++ b/scripts/kconfig/streamline_config.pl @@ -237,13 +237,11 @@ while ($repeat) { foreach my $config (keys %configs) { $config =~ s/^CONFIG_//; - if (!defined($depends{$config})) { - next; + if (defined($depends{$config})) { + # This config has dependencies. Make sure they are also included + parse_config_dep_select $depends{$config}; } - # This config has dependencies. Make sure they are also included - parse_config_dep_select $depends{$config}; - if (defined($prompt{$config}) || !defined($selects{$config})) { next; } -- 1.6.2.1 --
next prev parent reply other threads:[~2009-04-30 18:52 UTC|newest] Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top 2009-04-30 18:50 [PATCH 0/7] kconfig: more featured minimum module configs Steven Rostedt 2009-04-30 18:50 ` Steven Rostedt [this message] 2009-04-30 18:50 ` [PATCH 2/7] kconfig: do not warn about modules built in Steven Rostedt 2009-04-30 18:50 ` [PATCH 3/7] kconfig: enable CONFIG_IKCONFIG from streamline_config.pl Steven Rostedt 2009-04-30 21:51 ` Alan Jenkins 2009-04-30 21:51 ` Alan Jenkins 2009-04-30 22:53 ` Steven Rostedt 2009-04-30 22:53 ` Steven Rostedt 2009-04-30 18:50 ` [PATCH 4/7] kconfig: add check if end exists in extract-ikconfig Steven Rostedt 2009-04-30 18:50 ` [PATCH 5/7] kconfig: have extract-ikconfig read ELF files Steven Rostedt 2009-04-30 18:50 ` [PATCH 6/7] kconfig: keep config.gz around even if CONFIG_IKCONFIG_PROC is not set Steven Rostedt 2009-04-30 18:50 ` [PATCH 7/7] kconfig: search for a config to base the local(mod|yes)config on Steven Rostedt 2009-04-30 22:04 ` Alan Jenkins 2009-04-30 22:04 ` Alan Jenkins 2009-04-30 22:54 ` Steven Rostedt 2009-04-30 22:54 ` Steven Rostedt 2009-05-04 12:15 ` Andi Kleen 2009-05-04 12:28 ` Peter Zijlstra 2009-05-04 14:46 ` Steven Rostedt 2009-05-04 14:59 ` Andi Kleen 2009-04-30 19:10 ` [PATCH 0/7] kconfig: more featured minimum module configs Ingo Molnar 2009-04-30 22:55 ` Steven Rostedt 2009-05-01 12:01 ` Ingo Molnar 2009-05-01 23:27 ` Steven Rostedt 2009-05-06 12:15 ` Ingo Molnar
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20090430185217.661759804@goodmis.org \ --to=rostedt@goodmis.org \ --cc=acme@redhat.com \ --cc=akpm@linux-foundation.org \ --cc=linux-kbuild@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mingo@elte.hu \ --cc=peterz@infradead.org \ --cc=sam@ravnborg.org \ --cc=tglx@linutronix.de \ --cc=torvalds@linux-foundation.org \ --cc=tytso@mit.edu \ --cc=zippel@linux-m68k.org \ --subject='Re: [PATCH 1/7] kconfig: streamline_config.pl do not stop with no depends' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.