[coreboot-gerrit] Change in coreboot[master]: mb/google/octopus/var/fleex: Uses Wifi SAR table for non-LTE sku

[Karthik Ramasubramanian (Code Review)]

Karthik Ramasubramanian 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
> will fixed in this CL. […]

Sorry for a little back and forth on this. I dont know if there is a different SAR for LTE SKU.

Returning NULL will cause src/vendorcode/google/chromeos/sar.c to pick up wifi_sar_defaults.hex file.

wifi_sar_defaults.hex is built from CONFIG_WIFI_SAR_CBFS_FILEPATH which is wifi_sar-fleex.hex. So the SAR file will end up to be the same for both LTE and non-LTE SKUs.

Can you please double-confirm and correct me if I am wrong.