You-Cheng Syu has uploaded this change for review.

View Change

google/kukui: Configure AP_IN_SLEEP_L correctly.

This pin should be set to its alternative function SRCLKENA0 instead of
GPIO, so that SPM can control it.

BUG=b:113367227
BRANCH=none
TEST=1. Boot. Run 'powerinfo' in EC console and see power state in S0.
2. Run 'powerd_dbus_suspend --wakeup_timeout=10', and then
run 'powerinfo' in EC console and see power state in S3.
3. Wait until AP resume.
4. Run 'powerinfo' in EC console and see power state back to S0.

Change-Id: I0a7e34f95381dec17eb6d166d6552c12e021bd9a
Signed-off-by: You-Cheng Syu <youcheng@google.com>
---
M src/mainboard/google/kukui/early_init.c
1 file changed, 1 insertion(+), 2 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/20/32120/1
diff --git a/src/mainboard/google/kukui/early_init.c b/src/mainboard/google/kukui/early_init.c
index a16a335..1193bb3 100644
--- a/src/mainboard/google/kukui/early_init.c
+++ b/src/mainboard/google/kukui/early_init.c
@@ -32,8 +32,7 @@

setup_chromeos_gpios();

- /* Declare we are in S0 */
- gpio_output(AP_IN_SLEEP_L, 1);
+ gpio_set_mode(AP_IN_SLEEP_L, PAD_SRCLKENA0_FUNC_SRCLKENA0);

mtk_spi_init(CONFIG_DRIVER_TPM_SPI_BUS, SPI_PAD0_MASK, 1 * MHz);
gpio_eint_configure(CR50_IRQ, IRQ_TYPE_EDGE_RISING);

To view, visit change 32120. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I0a7e34f95381dec17eb6d166d6552c12e021bd9a
Gerrit-Change-Number: 32120
Gerrit-PatchSet: 1
Gerrit-Owner: You-Cheng Syu <youcheng@google.com>
Gerrit-MessageType: newchange