All of lore.kernel.org
 help / color / mirror / Atom feed
From: Julian Calaby <julian.calaby@gmail.com>
To: Ben Greear <greearb@candelatech.com>
Cc: linux-wireless <linux-wireless@vger.kernel.org>,
	Johannes Berg <johannes@sipsolutions.net>
Subject: Re: [PATCH] mac80211: Let VHT work on 2.4Ghz
Date: Tue, 23 Feb 2016 09:48:30 +1100	[thread overview]
Message-ID: <CAGRGNgWPv89sWV9wtXPBpothJrebRJ+d6a8-TnDU0o-p6ZkAmA@mail.gmail.com> (raw)
In-Reply-To: <1456180974-21062-1-git-send-email-greearb@candelatech.com>

Hi Ben,

On Tue, Feb 23, 2016 at 9:42 AM,  <greearb@candelatech.com> wrote:
> From: Ben Greear <greearb@candelatech.com>
>
> ath10k supports VHT on 2.4Ghz band.
> If supplicant and hostapd and radio think
> VHT should be allowed, then kernel should let them
> try.
>
> Signed-off-by: Ben Greear <greearb@candelatech.com>
> ---
>  net/mac80211/mlme.c | 16 ----------------
>  net/mac80211/util.c | 13 +------------
>  net/mac80211/vht.c  | 15 ---------------
>  3 files changed, 1 insertion(+), 43 deletions(-)
>
> diff --git a/net/mac80211/util.c b/net/mac80211/util.c
> index f8eb0f8..cabe16f 100644
> --- a/net/mac80211/util.c
> +++ b/net/mac80211/util.c
> @@ -1604,17 +1603,7 @@ static int ieee80211_build_preq_ies_band(struct ieee80211_local *local,
>                 *offset = noffset;
>         }
>
> -       /* Check if any channel in this sband supports at least 80 MHz */
> -       for (i = 0; i < sband->n_channels; i++) {
> -               if (sband->channels[i].flags & (IEEE80211_CHAN_DISABLED |
> -                                               IEEE80211_CHAN_NO_80MHZ))
> -                       continue;
> -
> -               have_80mhz = true;
> -               break;
> -       }
> -
> -       if (sband->vht_cap.vht_supported && have_80mhz && !disable_vht) {
> +       if (sband->vht_cap.vht_supported && /*have_80mhz &&*/ !disable_vht) {

Do you want to actually remove have_80mhz here? We do have git history
if this needs to be put back.

Thanks,

-- 
Julian Calaby

Email: julian.calaby@gmail.com
Profile: http://www.google.com/profiles/julian.calaby/

  reply	other threads:[~2016-02-22 22:48 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-22 22:42 [PATCH] mac80211: Let VHT work on 2.4Ghz greearb
2016-02-22 22:48 ` Julian Calaby [this message]
2016-02-22 22:55   ` Ben Greear
2016-02-23 10:47 ` Arik Nemtsov
2016-02-23 13:46   ` Ben Greear
2016-02-24  8:40     ` Arik Nemtsov
2016-02-24 13:24       ` Ben Greear
2016-02-24 13:32         ` Arik Nemtsov
2016-02-24 13:37           ` Ben Greear
2016-02-24 14:25             ` Arik Nemtsov
2016-02-23 14:10 ` Johannes Berg
2016-02-23 14:18   ` Ben Greear
2016-02-23 14:22     ` Johannes Berg
2016-02-23 14:58       ` Ben Greear
2016-02-23 21:08   ` Jouni Malinen
2016-02-23 21:15     ` Ben Greear

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAGRGNgWPv89sWV9wtXPBpothJrebRJ+d6a8-TnDU0o-p6ZkAmA@mail.gmail.com \
    --to=julian.calaby@gmail.com \
    --cc=greearb@candelatech.com \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.