2022-02-08 03:57:00

by Veerendranath Jakkam

[permalink] [raw]
Subject: Re: [PATCH 04/19] ieee80211: Add EHT (802.11be) definitions

On 2/7/22 5:40 PM, Johannes Berg wrote:
>
>> Append _MASK for the macros representing multi bit fields like above?
>>
> I guess that'd make some sense.
>
>> Or we can use GENMASK() to represent values
>>
> I don't like GENMASK() that much to be honest, it always feels confusing
> which way around the arguments should be and whether the edges are
> included or not, and then I don't feel we gain much from it?
>
> johannes


Ok, I think we can go with appending _MASK


veeru