Skip to contentSkip to conversation

cfg80211: fix compilation warning for cfg80211_leave_all()

PATCH Applied Arend van Spriel 2 participants 2 messages Jun 23, 2013 – Jun 24, 2013 802.11 (including cfg80211/NL80211)

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 cfg80211: fix compilation warning for cfg80211_leave_all()

The following compilation issue popped up moving from v3.10-rc1 to v3.10-rc6 after merging wireless-testing.

net/wireless/sysfs.c:86:13: error: 'cfg80211_leave_all' defined but not used [-Werror=unused-function]

The function is only called when CONFIG_PM is enabled. Moving the function under CONFIG_PM as well.

---
 net/wireless/sysfs.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
net/wireless/sysfs.c +1 -1
index 360a42c..a23253e 100644
@@ -83,6 +83,7 @@ static int wiphy_uevent(struct device *dev, struct kobj_uevent_env *env)
 	return 0;
 }
 
+#ifdef CONFIG_PM
 static void cfg80211_leave_all(struct cfg80211_registered_device *rdev)
 {
 	struct wireless_dev *wdev;
@@ -91,7 +92,6 @@ static void cfg80211_leave_all(struct cfg80211_registered_device *rdev)
 		cfg80211_leave(rdev, wdev);
 }
 
-#ifdef CONFIG_PM
 static int wiphy_suspend(struct device *dev, pm_message_t state)
 {
 	struct cfg80211_registered_device *rdev = dev_to_rdev(dev);
Signature · 1 line
1.7.10.4
maintainer Applied
#1 lore raw
Quoted 8 lines · Arend van Spriel

The following compilation issue popped up moving from v3.10-rc1 to v3.10-rc6 after merging wireless-testing.

net/wireless/sysfs.c:86:13: error: 'cfg80211_leave_all' defined but not used [-Werror=unused-function]

The function is only called when CONFIG_PM is enabled. Moving the function under CONFIG_PM as well.

Applied.

johannes