EricR Lai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45640 )
Change subject: mb/google/octopus/var/fleex: Uses Wifi SAR table for non-LTE sku ......................................................................
Patch Set 4:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45640/4/src/mainboard/google/octopu... File src/mainboard/google/octopus/variants/fleex/variant.c:
https://review.coreboot.org/c/coreboot/+/45640/4/src/mainboard/google/octopu... PS4, Line 25: 4
If you can define a macro instead of magic number, it will be more self-explanatory.
sounds good.