From: Peter Foley <pefoley2@verizon.net>
To: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Linux Kbuild Mailing List <linux-kbuild@vger.kernel.org>,
mmarek@suse.cz, rdunlap@xenotime.net, lacombar@gmail.com
Subject: [PATCH RFC] kconfig.h fix extraneous dependencies
Date: Sun, 31 Jul 2011 20:36:03 -0400 (EDT) [thread overview]
Message-ID: <alpine.LNX.2.00.1107311812460.62769@linux> (raw)
The introduction of kconfig.h caused fixdep to create three extraneous
dependencies.
include/config/.h
include/config/h.h
include/config/foo.h
Remove them by eliminating instances of CONFIG_
Signed-off-by: Peter Foley <pefoley2@verizon.net>
---
Suggestions for a better replacement for CONFIG_FOO would be appreciated.
diff --git a/include/linux/kconfig.h b/include/linux/kconfig.h
index 067eda0..66a0659 100644
--- a/include/linux/kconfig.h
+++ b/include/linux/kconfig.h
@@ -1,15 +1,15 @@
-#ifndef __LINUX_KCONFIG_H
-#define __LINUX_KCONFIG_H
+#ifndef __LINUX_KCONFIG
+#define __LINUX_KCONFIG
#include <generated/autoconf.h>
/*
- * Helper macros to use CONFIG_ options in C expressions. Note that
+ * Helper macros to use CONFIG options in C expressions. Note that
* these only work with boolean and tristate options.
*/
/*
- * IS_ENABLED(CONFIG_FOO) evaluates to 1 if CONFIG_FOO is set to 'y' or 'm',
+ * IS_ENABLED(CONFIG _FOO) evaluates to 1 if CONFIG _FOO is set to 'y' or 'm',
* 0 otherwise.
*
*/
@@ -17,16 +17,16 @@
(__enabled_ ## option || __enabled_ ## option ## _MODULE)
/*
- * IS_BUILTIN(CONFIG_FOO) evaluates to 1 if CONFIG_FOO is set to 'y', 0
+ * IS_BUILTIN(CONFIG _FOO) evaluates to 1 if CONFIG _FOO is set to 'y', 0
* otherwise. For boolean options, this is equivalent to
- * IS_ENABLED(CONFIG_FOO).
+ * IS_ENABLED(CONFIG _FOO).
*/
#define IS_BUILTIN(option) __enabled_ ## option
/*
- * IS_MODULE(CONFIG_FOO) evaluates to 1 if CONFIG_FOO is set to 'm', 0
+ * IS_MODULE(CONFIG _FOO) evaluates to 1 if CONFIG _FOO is set to 'm', 0
* otherwise.
*/
#define IS_MODULE(option) __enabled_ ## option ## _MODULE
-#endif /* __LINUX_KCONFIG_H */
+#endif /* __LINUX_KCONFIG */
next reply other threads:[~2011-08-01 0:38 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-01 0:36 Peter Foley [this message]
2011-08-01 0:53 ` [PATCH RFC] kconfig.h fix extraneous dependencies Arnaud Lacombe
2011-08-01 1:16 ` Peter Foley
2011-08-01 9:24 ` Michal Marek
2011-08-01 13:51 ` [PATCH v2] " Peter Foley
2011-09-09 9:46 ` Michal Marek
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=alpine.LNX.2.00.1107311812460.62769@linux \
--to=pefoley2@verizon.net \
--cc=lacombar@gmail.com \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mmarek@suse.cz \
--cc=rdunlap@xenotime.net \
/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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox