Return-path: Received: from mail-io0-f176.google.com ([209.85.223.176]:35464 "EHLO mail-io0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752305AbcF1OMQ (ORCPT ); Tue, 28 Jun 2016 10:12:16 -0400 Received: by mail-io0-f176.google.com with SMTP id f30so17697730ioj.2 for ; Tue, 28 Jun 2016 07:12:11 -0700 (PDT) Date: Tue, 28 Jun 2016 10:06:26 -0400 From: Bob Copeland To: Yaniv Machani Cc: linux-kernel@vger.kernel.org, Johannes Berg , "David S . Miller" , linux-wireless@vger.kernel.org, netdev@vger.kernel.org, Meirav Kama Subject: Re: [PATCH 3/4] mac80211: mesh: fixed HT ies in beacon template Message-ID: <20160628140625.GC9984@localhost> (sfid-20160628_161238_775596_AB6EC190) References: <20160628111307.8784-1-yanivma@ti.com> <20160628111307.8784-2-yanivma@ti.com> <20160628111307.8784-3-yanivma@ti.com> <20160628111307.8784-4-yanivma@ti.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20160628111307.8784-4-yanivma@ti.com> Sender: linux-wireless-owner@vger.kernel.org List-ID: On Tue, Jun 28, 2016 at 02:13:06PM +0300, Yaniv Machani wrote: > From: Meirav Kama > > There are several values in HT info elements of mesh beacon (built by the > mac80211) that are incorrect. Would be good to enumerate the problems here. > To fix them: > 1. mac80211 will check configuration from cfg and will build accordingly. > 2. changes made in mesh default values. What is wrong with the defaults? > sband = local->hw.wiphy->bands[band]; > if (!sband->ht_cap.ht_supported || > @@ -431,11 +433,40 @@ int mesh_add_ht_cap_ie(struct ieee80211_sub_if_data *sdata, > sdata->vif.bss_conf.chandef.width == NL80211_CHAN_WIDTH_10) > return 0; > > + /* determine capability flags */ > + cap = sband->ht_cap.cap; There is some weird whitespace here (space instead of tabs for the comment). -- Bob Copeland %% http://bobcopeland.com/