From 1e1b5d4463c2f273c593f90fd1f60b7c06139f86 Mon Sep 17 00:00:00 2001 From: Philip Chen Date: Thu, 11 Jan 2018 12:16:21 -0800 Subject: power/rk3399: Change power-off sequence for KD panel We should turn off PP3300_S0 and then PP1800_S0 to meet KD panel spec. PP3300_S0 has to be on in S3_WoUSB, so PP1800_S0 also has to be on - let's move PP1800_S0_EN to s0s3_usb_wake_power_seq. BUG=b:71057948 BRANCH=none TEST='suspend_stress_test' for 10+ cycles without seeing things go wrong Change-Id: Ic44411062b4c9e857b9f8ca6565550ba8bd2f950 Signed-off-by: Philip Chen Reviewed-on: https://chromium-review.googlesource.com/862254 Commit-Ready: Philip Chen Tested-by: Philip Chen Reviewed-by: Philip Chen --- power/rk3399.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/power/rk3399.c b/power/rk3399.c index b429556d5c..b264816f72 100644 --- a/power/rk3399.c +++ b/power/rk3399.c @@ -150,7 +150,6 @@ static const struct power_seq_op s3s0_usb_wake_power_seq[] = { /* The power sequence for POWER_S0S3 */ #if CONFIG_CHIPSET_POWER_SEQ_VERSION == 2 static const struct power_seq_op s0s3_power_seq[] = { - { GPIO_PP1800_S0_EN, 0, 1 }, { GPIO_AP_CORE_EN, 0, 20 }, }; #else @@ -168,6 +167,7 @@ static const struct power_seq_op s0s3_power_seq[] = { /* Sigs that need to be left on in S3, if we need to wake-on-USB */ static const struct power_seq_op s0s3_usb_wake_power_seq[] = { { GPIO_PP3300_S0_EN, 0, 20 }, + { GPIO_PP1800_S0_EN, 0, 1 }, { GPIO_PP1800_USB_EN, 0, 1 }, }; #endif -- cgit v1.2.1