On Tue, 2015-01-13 at 09:35 +0100, Janusz Dziedzic wrote:
> + * @IBSS_PARAMS_DISABLE_HT: Disable HT (802.11n)
> + * @IBSS_PARAMS_DISABLE_VHT: Disable VHT
If you list 11n (don't think you need to) maybe also ist 11ac? :)
Or just remove 11n.
> + * @flags See &enum cfg80211_ibss_params_flags
missing :
> * @ht_capa: HT Capabilities over-rides. Values set in ht_capa_mask
> * will be used in ht_capa. Un-supported values will be ignored.
> * @ht_capa_mask: The bits of ht_capa which are to be used.
> + * @vht_capa: VHT Capability overrides
why two spaces? :)
> /**
> * enum nl80211_ext_feature_index - bit index of extended features.
> + * @NL80211_FEATURE_VHT_IBSS: This driver supports IBSS with VHT datarates.
> *
> * @NUM_NL80211_EXT_FEATURES: number of extended features.
> * @MAX_NL80211_EXT_FEATURES: highest extended feature index.
> */
> enum nl80211_ext_feature_index {
> + NL80211_FEATURE_VHT_IBSS,
Yay, first bit :)
> @@ -7258,6 +7268,9 @@ static int nl80211_join_ibss(struct sk_buff *skb, struct genl_info *info)
> return err;
> }
>
> + if (nla_get_flag(info->attrs[NL80211_ATTR_DISABLE_HT]))
> + ibss.flags |= IBSS_PARAMS_DISABLE_HT;
That should perhaps be a separate patch? Also - it seems you need to
check somewhere that people aren't configuring 80 MHz with disable-VHT?
And similar for HT actually - is this new?
johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
