On Thu, Oct 20, 2011 at 2:46 AM, Christian Lamparter <chunkeey@xxxxxxxxxxxxxx> wrote: > On Thursday, October 20, 2011 03:03:24 AM Thomas Pedersen wrote: >> Signed-off-by: Thomas Pedersen <thomas@xxxxxxxxxxx> >> Signed-off-by: Ashok Nagarajan <anagar6@xxxxxxx> >> --- >> net/mac80211/agg-rx.c | 3 ++- >> net/mac80211/agg-tx.c | 4 +++- >> net/mac80211/ht.c | 3 ++- >> net/mac80211/rx.c | 1 + >> 4 files changed, 8 insertions(+), 3 deletions(-) >> >> diff --git a/net/mac80211/agg-tx.c b/net/mac80211/agg-tx.c >> index 2ac0339..36c2836 100644 >> --- a/net/mac80211/agg-tx.c >> +++ b/net/mac80211/agg-tx.c >> @@ -378,6 +379,7 @@ int ieee80211_start_tx_ba_session(struct ieee80211_sta *pubsta, u16 tid, >> * by drivers or the standard. >> */ >> if (sdata->vif.type != NL80211_IFTYPE_STATION && >> + sdata->vif.type != NL80211_IFTYPE_MESH_POINT && >> sdata->vif.type != NL80211_IFTYPE_AP_VLAN && >> sdata->vif.type != NL80211_IFTYPE_AP) >> return -EINVAL; > > just a nitpick, but it looks like the comment in agg-tx.c > seems to be a little out of date. Hence, would you mind > removing it as well? > No problem :) > /* > * The aggregation code is not prepared to handle > * anything but STA/AP due to the BSSID handling. > * IBSS could work in the code but isn't supported > * by drivers or the standard. > */ > if (sdata->vif.type != NL80211_IFTYPE_STATION && > sdata->vif.type != NL80211_IFTYPE_AP_VLAN && > sdata->vif.type != NL80211_IFTYPE_AP) > return -EINVAL; > > Regards, > Chr > -- 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
