Re: [PATCH] create net/wireless/Kconfig and make nl80211 optional

[Johannes Berg <johannes@sipsolutions.net>]

On Fri, 2007-02-23 at 08:33 -0800, James Ketrenos wrote:
> Johannes Berg wrote:
> > On Thu, 2007-02-22 at 11:42 -0800, James Ketrenos wrote:
> >> Move the wireless Kconfig options into their own wireless/Kconfig and
> >> add a CONFIG_NL80211 configuration option to allow nl80211 support to be
> >> optionally included (default =y)
> > 
> > Ack, but we'll need to modify core.h to stub out nl80211_init() and
> > nl80211_exit() in case nl80211 isn't compiled in. The reason this worked
> > for you is that I forgot to hook up nl80211 completely ;)
> > 
> > Do you want me to take care of that?`
> 
> Something like this?

Yup. I'd prefer this instead, and then rolled into your original patch
so it applies after my hookup patch.

johannes

--- wireless-dev.orig/net/wireless/nl80211.h	2007-02-23 21:08:45.353476868 +0100
+++ wireless-dev/net/wireless/nl80211.h	2007-02-23 21:09:58.543476868 +0100
@@ -1,7 +1,17 @@
 #ifndef __NET_WIRELESS_NL80211_H
 #define __NET_WIRELESS_NL80211_H
 
+#ifdef CONFIG_CFG80211_NL80211
 extern int nl80211_init(void);
 extern void nl80211_exit(void);
+#else
+static inline int nl80211_init(void)
+{
+	return 0;
+}
+static inline void nl80211_exit(void)
+{
+}
+#endif /* CONFIG_CFG80211_NL80211 */
 
 #endif /* __NET_WIRELESS_NL80211_H */