netjsonconfig icon indicating copy to clipboard operation
netjsonconfig copied to clipboard
verified publisher icon openwisp

[feature] Add support for new wireless radio syntax `band` #235

Open codesankalp opened this issue 3 years ago • 4 comments

Closes #235

codesankalp avatar Jun 14 '22 15:06 codesankalp

Coverage Status

coverage: 98.665% (-0.06%) from 98.728% when pulling 5d43bcdc66cf255bd60abf010288ba6d1c07b30e on issues/235-band-support into 3238df8837e0b9d457c02914810d0cbfba0fda81 on master.

coveralls avatar Jun 14 '22 16:06 coveralls

Do we have to show the hwmode property in netjson config if we get only the band property from uci ?

Suppose the config only contains band '5g' then in the netjson object should we show this band property? Earlier (in legacy) we are showing hwmode in netjson object.

codesankalp avatar Jun 17 '22 19:06 codesankalp

Do we have to show the hwmode property in netjson config if we get only the band property from uci ?

Suppose the config only contains band '5g' then in the netjson object should we show this band property? Earlier (in legacy) we are showing hwmode in netjson object.

hwmode is needed only on OpenWrt < 21, so on newer systems is not needed I think.

nemesifier avatar Jun 21 '22 14:06 nemesifier

I used the below references:

  1. https://openwrt.org/docs/techref/wireless.standards
  2. https://openwrt.org/docs/guide-user/network/wifi/basic
  3. https://en.wikipedia.org/wiki/List_of_WLAN_channels
  4. https://forum.openwrt.org/t/wi-fi-hwmode-values-seems-to-be-out-dated/96212/2

codesankalp avatar Jun 22 '22 17:06 codesankalp