Linux wireless drivers development
 help / color / mirror / Atom feed
* [PATCH] mac80211: fix confusing parentheses
@ 2011-10-18 11:39 Johannes Berg
  0 siblings, 0 replies; only message in thread
From: Johannes Berg @ 2011-10-18 11:39 UTC (permalink / raw)
  To: John Linville; +Cc: linux-wireless

From: Johannes Berg <johannes.berg@intel.com>

There's an extra pair of parentheses here that
is simply confusing because it implies a nesting
that doesn't actually exist. Just remove it.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
---
 net/mac80211/cfg.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- a/net/mac80211/cfg.c	2011-10-18 13:36:05.000000000 +0200
+++ b/net/mac80211/cfg.c	2011-10-18 13:38:05.000000000 +0200
@@ -1413,7 +1413,7 @@ static int ieee80211_set_channel(struct
 	    (old_oper_type != local->_oper_channel_type))
 		ieee80211_hw_config(local, IEEE80211_CONF_CHANGE_CHANNEL);
 
-	if ((sdata && sdata->vif.type != NL80211_IFTYPE_MONITOR) &&
+	if (sdata && sdata->vif.type != NL80211_IFTYPE_MONITOR &&
 	    old_vif_oper_type != sdata->vif.bss_conf.channel_type)
 		ieee80211_bss_info_change_notify(sdata, BSS_CHANGED_HT);
 



^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2011-10-18 11:39 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-10-18 11:39 [PATCH] mac80211: fix confusing parentheses Johannes Berg

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox