openconfig/public

missing channels in channels-type?

precla opened this issue · 4 comments

range "1..14 | 17 | 21 | 25 | 29 | 33 | 36 | 37 | 40 | 41 | 44 | 45 | 48 | 49 | 52 | 53 | 56 | 57 | 60 | 61 | 64 | 65 | 69 | 73 | 77 | 81 | 85 | 89 | 93 | 97 | 100 | 101 | 104 | 105 | 108 | 109 | 112 | 113 | 116 | 117 | 120 | 121 | 124 | 125 | 128 | 129 | 132 | 133 | 136 | 137 | 140 | 141 | 144 | 145 | 149 | 153 | 157 | 161 | 165 | 169 | 173 | 177 | 181 | 185 | 189 | 193 | 197 | 201 | 205 | 209 | 213 | 217 | 221 | 225 | 229 | 233";

For example, channels 171, 172 are missing. iw phy phy0 channels shows it for my device (mac80211 hwsim):

* 5855 MHz [171] 
          Maximum TX power: 13.0 dBm
          Channel widths: 20MHz VHT80
* 5860 MHz [172] 
          Maximum TX power: 13.0 dBm
          Channel widths: 20MHz VHT80

There seems to be a few more missing when compared to this wikipedia article. For example 155, 138 -> don't have those in my iw phy phy0 channels output tho.

https://en.wikipedia.org/wiki/List_of_WLAN_channels#5_GHz_(802.11a/h/j/n/ac/ax)

Is this on purpose?

@precla
I'm not sure on 172, this is an 802.11p frequency and i'm not sure enterprise infrastructure will be used for this band for C-V2X, this requires some research. HWSIM can be used by those other industries as well, so it's not unusual for them to have new bands before actual hardware.

171 is a UNII-4, which we should get added as i think OC hardware from one vendor supports this today.

In terms of terminology, these are the 20MHz primary channels. For 155, 138, these are 80MHz channels in 5GHz and 40MHz in 6GHz. So 155 is inclusive of (149, 153, 157, 161) in 5GHz and (153, 157) in 6GHz. 138 is inclusive of (132, 136, 140, 144) in 5GHz.

precla commented

Considering the comment and commit, I can say that it can be closed as completed unless you want to further investigate.
Thanks!

@precla I think we are ok with the current implementation unless we need to add those .11p channels. The remaining are all bonded center frequency number, but we include the primary 20MHz members. For example:
171 = 165 @ 80MHz.

The same goes for things like 38 (which is 36 + 40). Configuration would be the primary 20MHz + channel width.

precla commented

Thanks again for the investigation and feedback!