mac80211: mark local variable __maybe_unused
authorJohannes Berg <johannes.berg@intel.com>
Wed, 9 Apr 2014 19:31:13 +0000 (21:31 +0200)
committerJohannes Berg <johannes.berg@intel.com>
Mon, 5 May 2014 14:03:42 +0000 (16:03 +0200)
The 'local' variable in __ieee80211_vif_copy_chanctx_to_vlans()
is only used/needed when lockdep is compiled in, mark it as such
to avoid compile warnings in the other case.

While at it, fix some indentation where it's used.

Reviewed-by: Luciano Coelho <luciano.coelho@intel.com>
Reviewed-by: Emmanuel Grumbach <emmanuel.grumbach@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/mac80211/chan.c

index d8b1b861484259a426e91ad12d5dbe0ac17f784b..3702d642cd8f77bad58e23863efcfc4275db0418 100644 (file)
@@ -848,7 +848,7 @@ static void
 __ieee80211_vif_copy_chanctx_to_vlans(struct ieee80211_sub_if_data *sdata,
                                      bool clear)
 {
-       struct ieee80211_local *local = sdata->local;
+       struct ieee80211_local *local __maybe_unused = sdata->local;
        struct ieee80211_sub_if_data *vlan;
        struct ieee80211_chanctx_conf *conf;
 
@@ -864,7 +864,7 @@ __ieee80211_vif_copy_chanctx_to_vlans(struct ieee80211_sub_if_data *sdata,
         * to a channel context that has already been freed.
         */
        conf = rcu_dereference_protected(sdata->vif.chanctx_conf,
-                               lockdep_is_held(&local->chanctx_mtx));
+                                        lockdep_is_held(&local->chanctx_mtx));
        WARN_ON(!conf);
 
        if (clear)