Re: [PATCH] staging: rtl8723bs: refactor BSS Coexistence channel report logic

[Te-Hsiu Huang]

>
> We only ever set the first element in the class_active[] array.
> #puzzled
>
> > +     bool ch_present[BSS_COEX_MAX_CLASSES][BSS_COEX_MAX_CHANNELS] = {{false}};
>
> Btw, use "= {};" to initialize arrays to zero.  It's not standard C, but
> it's kernel C.
>
> regards,
> dan carpenter
>

Hi Dan,
Thanks for your time!! I understand the confusion. I kept the array
structure because it reflects the IEEE 802.11 Intolerant Channel
Report format, which is organized by Regulatory Classes.

Although the driver currently only reports for the 2.4GHz class (index
0), using an array makes the code semantically aligned with the
specification and easier to extend when other operating classes need
to be supported in the future.

I have submitted a V2 which uses the '= {}' initialization you suggested.