[coreboot-gerrit] Change in coreboot[master]: vc/google/chromeos: load wifi_sar_defaults.hex as the main WiFi SAR C...

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44672 )

Change subject: vc/google/chromeos: load wifi_sar_defaults.hex as the main WiFi SAR CBFS source
......................................................................

Patch Set 3: Code-Review+2

(1 comment)

https://review.coreboot.org/c/coreboot/+/44672/2/src/vendorcode/google/chrom... 
File src/vendorcode/google/chromeos/sar.c:

https://review.coreboot.org/c/coreboot/+/44672/2/src/vendorcode/google/chrom... 
PS2, Line 17: if (filename == NULL)
            : 		filename = WIFI_SAR_CBFS_FILENAME;
> Ya. This code is weird. […]

I think it is probably to allow mainboards to also return NULL to use the default filename.