On 10/20/2023 11:46 PM, Jeff Johnson wrote: > On 10/20/2023 2:57 AM, Arend van Spriel wrote: >> On 10/19/2023 3:42 AM, Daniel Berlin wrote: >>> This patch adds support for 6G chanspecs, as part of adding 6G and >>> 802.11ax support. > > In ath drivers we are actively discouraging the use of "G" since 1) that > is not the SI unit for gigahertz, and 2) it conflicts with the use of > that term in cellular communication where it refers to the generation of > technology. So I'd recommend using GHz in descriptions/comments and > GHZ/ghz in identifiers. I understand, but I am not going to push back on it. The identifiers using this are defined and used in the firmware API so at least those I prefer to keep similar/same. For comments in the driver code we should consider using GHz instead. > Side note: in USA, Comcast have started promoting their network as > "10G". In their case they're referring to 10 Gbps speeds. They are being > called out for misleading consumers with that terminology. > If consumers are that stupid they can mislead by anything ;-) Thanks, Arend