GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

mb/google/octopus/variants/fleex: Only do LTE power off for LTE sku 

Only do LTE power off for LTE sku in order to save extra 130ms delay
for non-LTE sku.

BUG=b:168075958
BRANCH=octopus
TEST=build image and verify on the DUT with LTE DB.

Signed-off-by: Eric Lai <ericr_lai@compal.corp-partner.google.com>
Change-Id: If983185ff2f09fb1b2553c6ff1a1473d3254de4b
Reviewed-on: https://review.coreboot.org/c/coreboot/+/45687
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Marco Chen <marcochen@google.com>
This commit is contained in:
Eric Lai 2020-09-24 20:38:40 +08:00 committed by Patrick Georgi
parent 40ed6f2f78
commit 5f43369bec
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/mainboard/google/octopus/variants/fleex/variant.c
View File

@ -2,27 +2,31 @@
#include <acpi/acpi.h> #include <acpi/acpi.h>
#include <baseboard/variants.h> #include <baseboard/variants.h>
#include <delay.h>
#include <ec/google/chromeec/ec.h> #include <ec/google/chromeec/ec.h>
#include <gpio.h>
#include <sar.h> #include <sar.h>
#define LTE_SKU 4
static bool is_lte_sku(void)
{
return (google_chromeec_get_board_sku() == LTE_SKU);
}
void variant_smi_sleep(u8 slp_typ) void variant_smi_sleep(u8 slp_typ)
{ {
/* Currently use cases here all target to S5 therefore we do early return /* Currently use cases here all target to S5 therefore we do early return
* here for saving one transaction to the EC for getting SKU ID. */ * here for saving one transaction to the EC for getting SKU ID. */
if (slp_typ != ACPI_S5) if (slp_typ != ACPI_S5)
return; return;
if (is_lte_sku())
power_off_lte_module(); power_off_lte_module();
} }
const char *get_wifi_sar_cbfs_filename(void) const char *get_wifi_sar_cbfs_filename(void)
{ {
const char *filename = NULL; const char *filename = NULL;
uint32_t sku_id = google_chromeec_get_board_sku();
if (sku_id != 4) if (!is_lte_sku())
filename = "wifi_sar-fleex.hex"; filename = "wifi_sar-fleex.hex";
return filename; return filename;