Re: [PATCH] wifi: move action code from per-type frame structs

[Jeff Johnson <jeff.johnson@oss.qualcomm.com>]

On 2/25/2026 9:08 AM, Johannes Berg wrote:
> 
> And a couple of other notes:
> 
>>  				struct{
>> -					u8 action_code;
>> +					u8 no_fixed_fields[0];
>>  					u8 variable[];
>>  				} __packed chan_switch;
> 
> This thing in a couple of places isn't ideal, but I think it doesn't
> outweigh the benefits of this structural approach.

My fear is that "helpful" members of the community will refer to the
following, believing this is an old style variable array definition:

https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays

So I personally feel that a /* no fixed fields */ comment would provide the
same benefit without bringing unwanted noise (unless the code just doesn't
build without the member being present).

But you're the boss :)

/jeff