Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/45640 )
Change subject: mb/google/octopus/var/fleex: Use Wifi SAR table for non-LTE sku only ......................................................................
mb/google/octopus/var/fleex: Use Wifi SAR table for non-LTE sku only
Use Wifi SAR table for non-LTE sku only.
BUG=b:169115341 BRANCH=octopus TEST=Check no SAR table can be loaded with sku id 4.
Signed-off-by: Eric Lai ericr_lai@compal.corp-partner.google.com Change-Id: I086fa14a9f23e4a0fc0ef8085040219c932dbf17 Reviewed-on: https://review.coreboot.org/c/coreboot/+/45640 Reviewed-by: Marco Chen marcochen@google.com Reviewed-by: Angel Pons th3fanbus@gmail.com Reviewed-by: Ivy Jian ivy_jian@compal.corp-partner.google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/mainboard/google/octopus/variants/fleex/variant.c 1 file changed, 13 insertions(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Marco Chen: Looks good to me, approved Ivy Jian: Looks good to me, approved Angel Pons: Looks good to me, but someone else must approve
diff --git a/src/mainboard/google/octopus/variants/fleex/variant.c b/src/mainboard/google/octopus/variants/fleex/variant.c index 52f822a..1b0784c 100644 --- a/src/mainboard/google/octopus/variants/fleex/variant.c +++ b/src/mainboard/google/octopus/variants/fleex/variant.c @@ -1,10 +1,11 @@ /* SPDX-License-Identifier: GPL-2.0-only */
#include <acpi/acpi.h> -#include <ec/google/chromeec/ec.h> #include <baseboard/variants.h> #include <delay.h> +#include <ec/google/chromeec/ec.h> #include <gpio.h> +#include <sar.h>
void variant_smi_sleep(u8 slp_typ) { @@ -15,3 +16,14 @@
power_off_lte_module(); } + +const char *get_wifi_sar_cbfs_filename(void) +{ + const char *filename = NULL; + uint32_t sku_id = google_chromeec_get_board_sku(); + + if (sku_id != 4) + filename = "wifi_sar-fleex.hex"; + + return filename; +}