Thread (4 messages) 4 messages, 2 authors, 2012-05-03

Re: [PATCH 2/2] mac80211: don't transmit 40MHz frames to 20MHz peer

From: Thomas Pedersen <hidden>
Date: 2012-05-03 19:02:40

On Thu, May 3, 2012 at 11:55 AM, Johannes Berg
[off-list ref] wrote:
On Thu, 2012-04-26 at 15:01 -0700, Thomas Pedersen wrote:
quoted
If a mesh peer indicates it is operating as 20MHz-only in its HT
operation IE, have the rate control algorithm respect this by disabling
the equivalent bit in the ieee80211_sta HT capabilities.

Signed-off-by: Thomas Pedersen <redacted>
---
 net/mac80211/mesh_plink.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/net/mac80211/mesh_plink.c b/net/mac80211/mesh_plink.c
index f4124d7..6209327 100644
--- a/net/mac80211/mesh_plink.c
+++ b/net/mac80211/mesh_plink.c
@@ -302,6 +302,12 @@ static struct sta_info *mesh_peer_init(struct ieee80211_sub_if_data *sdata,
      else
              memset(&sta->sta.ht_cap, 0, sizeof(sta->sta.ht_cap));

+     if (elems->ht_operation)
+             if (!(elems->ht_operation->ht_param &
+                   IEEE80211_HT_PARAM_CHAN_WIDTH_ANY))
+                     sta->sta.ht_cap.cap &=
+                                         ~IEEE80211_HT_CAP_SUP_WIDTH_20_40;
The double if is a bit odd, why not use && to clean up all the
formatting?
Good point, but later Ashok adds a patch which takes the same branch anyway.

Thomas
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help