[M] Change in coreboot[main]: drivers/wifi: Support 320Mhz Bandwidth Enablement per MCC

[Jérémy Compostella (Code Review)]

Attention is currently required from: Bob Moragues, Bora Guvendik, Cliff Huang, Eric Lai, Hannah Williams, Poornima Tom, Ravishankar Sarawadi, Subrata Banik, YH Lin, [e-mail].

Jérémy Compostella has posted comments on this change by Poornima Tom. ( https://review.coreboot.org/c/coreboot/+/81790?usp=email )

Change subject: drivers/wifi: Support 320Mhz Bandwidth Enablement per MCC
......................................................................

Patch Set 5:

(6 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/81790/comment/05907e49_3710ca55?usp... :
PS2, Line 7: Wifi
> driver/wifi?

Done

File src/drivers/wifi/generic/acpi.c:

https://review.coreboot.org/c/coreboot/+/81790/comment/60555c4e_e479e572?usp... :
PS2, Line 486: WiFi320MHzPerMcc
> avoid camelcase

Done

File src/include/sar.h:

https://review.coreboot.org/c/coreboot/+/81790/comment/bb7ca870_9c68641f?usp... :
PS2, Line 58: enablement_11ax;
> This CL is with respect to 320MHZ wifi bandwidth. So, is this relevant in this CL. […]

Done

https://review.coreboot.org/c/coreboot/+/81790/comment/05a3ef01_b5ff9bde?usp... :
PS2, Line 64: WiFi320MHzPerMcc
> avoid camel case

Done

https://review.coreboot.org/c/coreboot/+/81790/comment/a4f40b9d_c2ba505a?usp... :
PS2, Line 65: __packed
> any reason for using `packed`?

The binary is packed and mixing uint8_t and uint32_t types.

File src/vendorcode/google/chromeos/sar.c:

https://review.coreboot.org/c/coreboot/+/81790/comment/3836c230_c65b94db?usp... :
PS2, Line 219:  *
> don't delete please

Done