aboutsummaryrefslogtreecommitdiff
path: root/src/mainboard/google/poppy
diff options
context:
space:
mode:
authorIvy Jian <ivy_jian@compal.corp-partner.google.com>2019-05-27 11:19:44 +0800
committerFurquan Shaikh <furquan@google.com>2019-05-29 16:40:03 +0000
commitb80d1324d3e86c0b8c72fd7cd09df07794931059 (patch)
treec4255e9b732b167b0f55361849333b8b8a485e20 /src/mainboard/google/poppy
parent23fbd052b90be2e9a6c3858bb9b314069e70ea6d (diff)
mb/google/poppy/variants/nami: Disable FPMCU for non-fingerprint variants
Even fingerprint device probe failed on non-fingerpint boards,the CRFP driver still register the device that cause the GPE#1 as wake source every time. Override devicetree for non-fingerpirnt variants to avoid unexpected wake event(GPE#1). BUG=b:129650040 BRANCH=firmware-nami-10775.108.B TEST=Boots to OS and check no GPE#1 wake event from eventlog when S0ix exit. Signed-off-by: Ivy Jian <ivy_jian@compal.corp-partner.google.com> Change-Id: I6fa96e04a34e296889414b96a8c604fc61b8a236 Reviewed-on: https://review.coreboot.org/c/coreboot/+/33017 Reviewed-by: EricR Lai <ericr_lai@compal.corp-partner.google.com> Reviewed-by: John Su <john_su@compal.corp-partner.google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/mainboard/google/poppy')
-rw-r--r--src/mainboard/google/poppy/variants/nami/mainboard.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/mainboard/google/poppy/variants/nami/mainboard.c b/src/mainboard/google/poppy/variants/nami/mainboard.c
index dff62c217c..cc87136520 100644
--- a/src/mainboard/google/poppy/variants/nami/mainboard.c
+++ b/src/mainboard/google/poppy/variants/nami/mainboard.c
@@ -238,6 +238,7 @@ void variant_devtree_update(void)
struct device *root = SA_DEV_ROOT;
config_t *cfg = root->chip_info;
uint8_t pl2_id = PL2_ID_DEFAULT;
+ struct device *spi_fpmcu = PCH_DEV_GSPI1;
switch (sku_id) {
case SKU_0_SONA:
@@ -260,6 +261,7 @@ void variant_devtree_update(void)
case SKU_3_PANTHEON:
case SKU_4_PANTHEON:
cfg->usb2_ports[5].enable = 0;
+ spi_fpmcu->enabled = 0;
break;
case SKU_0_BARD:
case SKU_1_BARD: