2023-10-21 18:29:11

by Arend van Spriel

[permalink] [raw]
Subject: Re: [PATCH 1/5] [brcmfmac] Add support for encoding/decoding 6g chanspecs

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.
> <https://arstechnica.com/tech-policy/2023/10/comcast-should-stop-advertising-slower-speeds-as-10g-industry-group-says/>

If consumers are that stupid they can mislead by anything ;-)

Thanks,
Arend


Attachments:
smime.p7s (4.12 kB)
S/MIME Cryptographic Signature