diff options
author | Johannes Berg <johannes@sipsolutions.net> | 2008-02-21 05:22:12 -0500 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2008-02-29 15:41:38 -0500 |
commit | f3af89d1aaaf2d2ef3bc6afe5eadb7127c51fe12 (patch) | |
tree | a46cf19d134d66de32a3ccd44dce80aab9722d4a /net/mac80211/debugfs_sta.c | |
parent | 665e8aafb4e0826caec9db25617b186ea3f3ec91 (diff) |
mac80211: fix debugfs_sta print_mac() warning
When print_mac() was marked as __pure to avoid emitting a function
call in pr_debug() scenarios, a warning in this code surfaced since
it relies on the fact that the buffer is modified and doesn't use
the return value. This patch makes it use the return value instead.
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Reported-by: Harvey Harrison <harvey.harrison@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'net/mac80211/debugfs_sta.c')
-rw-r--r-- | net/mac80211/debugfs_sta.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/net/mac80211/debugfs_sta.c b/net/mac80211/debugfs_sta.c index 47db0d4aa433..1a4d2c3775af 100644 --- a/net/mac80211/debugfs_sta.c +++ b/net/mac80211/debugfs_sta.c | |||
@@ -297,12 +297,13 @@ STA_OPS_WR(agg_status); | |||
297 | void ieee80211_sta_debugfs_add(struct sta_info *sta) | 297 | void ieee80211_sta_debugfs_add(struct sta_info *sta) |
298 | { | 298 | { |
299 | struct dentry *stations_dir = sta->local->debugfs.stations; | 299 | struct dentry *stations_dir = sta->local->debugfs.stations; |
300 | DECLARE_MAC_BUF(mac); | 300 | DECLARE_MAC_BUF(mbuf); |
301 | u8 *mac; | ||
301 | 302 | ||
302 | if (!stations_dir) | 303 | if (!stations_dir) |
303 | return; | 304 | return; |
304 | 305 | ||
305 | print_mac(mac, sta->addr); | 306 | mac = print_mac(mbuf, sta->addr); |
306 | 307 | ||
307 | sta->debugfs.dir = debugfs_create_dir(mac, stations_dir); | 308 | sta->debugfs.dir = debugfs_create_dir(mac, stations_dir); |
308 | if (!sta->debugfs.dir) | 309 | if (!sta->debugfs.dir) |