Johannes Berg wrote: > On Wed, 2011-01-05 at 21:17 +0530, Sujith wrote: > > From: Sujith Manoharan <Sujith.Manoharan@xxxxxxxxxxx> > > > > Add two new per-BSS attributes to allow configuration of > > HT capabilites and operational parameters by hostapd. > > Looks fine, but > > > @@ -2597,6 +2597,12 @@ static int nl80211_set_bss(struct sk_buff *skb, struct genl_info *info) > > if (info->attrs[NL80211_ATTR_BSS_HT_OPMODE]) > > params.ht_opmode = > > nla_get_u16(info->attrs[NL80211_ATTR_BSS_HT_OPMODE]); > > + if (info->attrs[NL80211_ATTR_BSS_HT_CAPAB]) > > + params.ht_capab = > > + nla_get_u16(info->attrs[NL80211_ATTR_BSS_HT_CAPAB]); > > + if (info->attrs[NL80211_ATTR_BSS_HT_PARAM]) > > + params.ht_param = > > + nla_get_u8(info->attrs[NL80211_ATTR_BSS_HT_PARAM]); > > > > if (!rdev->ops->change_bss) > > return -EOPNOTSUPP; > > This appears to be missing a -1 initialisation? Will fix. Sujith -- 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