Skip site navigation (1)Skip section navigation (2) 
Date:      Thu, 5 Nov 2020 12:10:25 +0000 (UTC)
From:      "Bjoern A. Zeeb" <bz@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-12@freebsd.org
Subject:   svn commit: r367374 - in stable/12: lib/lib80211 sbin/ifconfig sys/net80211
Message-ID:  <202011051210.0A5CAPH7024713@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help 
Author: bz
Date: Thu Nov  5 12:10:24 2020
New Revision: 367374
URL: https://svnweb.freebsd.org/changeset/base/367374

Log:
  MFC r366522:
  
    80211: non-functional changes
  
    Sort a few VHT160 and 80+80 lines, update some comments, and remove
    a superfluous ','.
  
    No functional changes intended.

Modified:
  stable/12/lib/lib80211/lib80211_regdomain.c
  stable/12/sbin/ifconfig/ifieee80211.c
  stable/12/sys/net80211/ieee80211.c
  stable/12/sys/net80211/ieee80211_node.h
Directory Properties:
  stable/12/   (props changed)

Modified: stable/12/lib/lib80211/lib80211_regdomain.c
==============================================================================
--- stable/12/lib/lib80211/lib80211_regdomain.c	Thu Nov  5 12:08:04 2020	(r367373)
+++ stable/12/lib/lib80211/lib80211_regdomain.c	Thu Nov  5 12:10:24 2020	(r367374)
@@ -191,11 +191,11 @@ decode_flag(struct mystate *mt, const char *p, int len
 		FLAG(IEEE80211_CHAN_VHT20),
 		FLAG(IEEE80211_CHAN_VHT40),
 		FLAG(IEEE80211_CHAN_VHT80),
+		FLAG(IEEE80211_CHAN_VHT160),
 		/*
 		 * XXX VHT80P80? This likely should be done by
 		 * 80MHz chan logic in net80211 / ifconfig.
 		 */
-		FLAG(IEEE80211_CHAN_VHT160),
 		FLAG(IEEE80211_CHAN_ST),
 		FLAG(IEEE80211_CHAN_TURBO),
 		FLAG(IEEE80211_CHAN_PASSIVE),

Modified: stable/12/sbin/ifconfig/ifieee80211.c
==============================================================================
--- stable/12/sbin/ifconfig/ifieee80211.c	Thu Nov  5 12:08:04 2020	(r367373)
+++ stable/12/sbin/ifconfig/ifieee80211.c	Thu Nov  5 12:10:24 2020	(r367374)
@@ -2385,8 +2385,7 @@ regdomain_makechannels(
 				    &dc->dc_chaninfo);
 			}
 
-			/* VHT80 */
-			/* XXX dc_vhtcap? */
+			/* VHT80 is mandatory (and so should be VHT40 above). */
 			if (1) {
 				regdomain_addchans(ci, &rd->bands_11ac, reg,
 				    IEEE80211_CHAN_A | IEEE80211_CHAN_HT40U |

Modified: stable/12/sys/net80211/ieee80211.c
==============================================================================
--- stable/12/sys/net80211/ieee80211.c	Thu Nov  5 12:08:04 2020	(r367373)
+++ stable/12/sys/net80211/ieee80211.c	Thu Nov  5 12:10:24 2020	(r367374)
@@ -158,7 +158,7 @@ ieee80211_chan_init(struct ieee80211com *ic)
 
 		/*
 		 * Setup the HT40/VHT40 upper/lower bits.
-		 * The VHT80 math is done elsewhere.
+		 * The VHT80/... math is done elsewhere.
 		 */
 		if (IEEE80211_IS_CHAN_HT40(c) && c->ic_extieee == 0)
 			c->ic_extieee = ieee80211_mhz2ieee(c->ic_freq +
@@ -167,8 +167,8 @@ ieee80211_chan_init(struct ieee80211com *ic)
 
 		/* Update VHT math */
 		/*
-		 * XXX VHT again, note that this assumes VHT80 channels
-		 * are legit already
+		 * XXX VHT again, note that this assumes VHT80/... channels
+		 * are legit already.
 		 */
 		set_vht_extchan(c);
 
@@ -712,8 +712,8 @@ ieee80211_vap_attach(struct ieee80211vap *vap, ifm_cha
 	ieee80211_syncflag_vht_locked(ic, IEEE80211_FVHT_VHT);
 	ieee80211_syncflag_vht_locked(ic, IEEE80211_FVHT_USEVHT40);
 	ieee80211_syncflag_vht_locked(ic, IEEE80211_FVHT_USEVHT80);
-	ieee80211_syncflag_vht_locked(ic, IEEE80211_FVHT_USEVHT80P80);
 	ieee80211_syncflag_vht_locked(ic, IEEE80211_FVHT_USEVHT160);
+	ieee80211_syncflag_vht_locked(ic, IEEE80211_FVHT_USEVHT80P80);
 	IEEE80211_UNLOCK(ic);
 
 	return 1;
@@ -767,8 +767,8 @@ ieee80211_vap_detach(struct ieee80211vap *vap)
 	ieee80211_syncflag_vht_locked(ic, IEEE80211_FVHT_VHT);
 	ieee80211_syncflag_vht_locked(ic, IEEE80211_FVHT_USEVHT40);
 	ieee80211_syncflag_vht_locked(ic, IEEE80211_FVHT_USEVHT80);
-	ieee80211_syncflag_vht_locked(ic, IEEE80211_FVHT_USEVHT80P80);
 	ieee80211_syncflag_vht_locked(ic, IEEE80211_FVHT_USEVHT160);
+	ieee80211_syncflag_vht_locked(ic, IEEE80211_FVHT_USEVHT80P80);
 
 	/* NB: this handles the bpfdetach done below */
 	ieee80211_syncflag_ext_locked(ic, IEEE80211_FEXT_BPF);
@@ -1160,7 +1160,7 @@ struct vht_chan_range vht80_chan_ranges[] = {
 	{ 5570, 5650 },
 	{ 5650, 5730 },
 	{ 5735, 5815 },
-	{ 0, 0, }
+	{ 0, 0 }
 };
 
 static int

Modified: stable/12/sys/net80211/ieee80211_node.h
==============================================================================
--- stable/12/sys/net80211/ieee80211_node.h	Thu Nov  5 12:08:04 2020	(r367373)
+++ stable/12/sys/net80211/ieee80211_node.h	Thu Nov  5 12:10:24 2020	(r367374)
@@ -70,7 +70,7 @@ struct ieee80211vap;
 struct ieee80211_scanparams;
 
 /*
- * Information element ``blob''.  We use this structure
+ * Information element (IE) ``blob''.  We use this structure
  * to capture management frame payloads that need to be
  * retained.  Information elements within the payload that
  * we need to consult have references recorded.

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202011051210.0A5CAPH7024713>