Re: [RFC 1/3] cfg80211: Add support for HE

2018-05-28 Thread Arend van Spriel
On 5/25/2018 9:51 PM, Luca Coelho wrote: Arend, On Fri, 2018-05-25 at 13:11 +0300, Luca Coelho wrote: On Mon, 2018-05-21 at 21:47 +0200, Arend van Spriel wrote: On 5/18/2018 4:05 PM, Luca Coelho wrote: @@ -781,6 +783,23 @@ int wiphy_register(struct wiphy *wiphy)

Re: [RFC 1/3] cfg80211: Add support for HE

2018-05-25 Thread Luca Coelho
Arend, On Fri, 2018-05-25 at 13:11 +0300, Luca Coelho wrote: > On Mon, 2018-05-21 at 21:47 +0200, Arend van Spriel wrote: > > On 5/18/2018 4:05 PM, Luca Coelho wrote: > > > @@ -781,6 +783,23 @@ int wiphy_register(struct wiphy *wiphy) > > > sband->channels[i].band = band;

Re: [RFC 1/3] cfg80211: Add support for HE

2018-05-25 Thread Arend van Spriel
On 5/25/2018 12:11 PM, Luca Coelho wrote: +static int > >+nl80211_send_ift_data(struct sk_buff *msg, > >+const struct ieee80211_sband_iftype_data *iftdata) > >make it nl80211_send_iftype_data. Okay, I'll replace all ift instances to iftype. My comment is mainly about

Re: [RFC 1/3] cfg80211: Add support for HE

2018-05-25 Thread Luca Coelho
Hi Arend, On Mon, 2018-05-21 at 21:47 +0200, Arend van Spriel wrote: > On 5/18/2018 4:05 PM, Luca Coelho wrote: > > From: Luca Coelho > > > > Add support for the HE in cfg80211 and also add userspace API to > > nl80211 to send rate information out, conforming with > >

Re: [RFC 1/3] cfg80211: Add support for HE

2018-05-21 Thread Arend van Spriel
On 5/18/2018 4:05 PM, Luca Coelho wrote: From: Luca Coelho Add support for the HE in cfg80211 and also add userspace API to nl80211 to send rate information out, conforming with P802.11ax_D1.4. A couple of things changed in D2.0 so does it make sense to introduce

[RFC 1/3] cfg80211: Add support for HE

2018-05-18 Thread Luca Coelho
From: Luca Coelho Add support for the HE in cfg80211 and also add userspace API to nl80211 to send rate information out, conforming with P802.11ax_D1.4. Additionally, remove the IEEE80211_MAX_AMPDU_BUF definition from some realtek drivers in staging because they are