From 77ef6189299a6ac752bd45ee77ca810f86ef2fb1 Mon Sep 17 00:00:00 2001 From: David Schneider Date: Mon, 13 Jun 2016 18:34:26 -0700 Subject: [PATCH] gru/kevin: Turn PP1800_PMU on earlier in sequence PP1800_PMU impacts the initial centerlogic voltage due to DVS circuitry. Since there's no other sequencing dependency, turn it on earlier. This fixes centerlogic from initially starting too high (1.5V). BUG=none BRANCH=none TEST=Watch PPVAR_CENTERLOGIC and confirm that it starts at the target voltage Change-Id: Icac076a7e8aef978401452a98d9f6bc8b373d94f Signed-off-by: David Schneider Reviewed-on: https://chromium-review.googlesource.com/352247 Reviewed-by: Shawn N --- power/rk3399.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/power/rk3399.c b/power/rk3399.c index 6f2b0c6cba..0ae4786b51 100644 --- a/power/rk3399.c +++ b/power/rk3399.c @@ -119,9 +119,9 @@ enum power_state power_handle_state(enum power_state state) gpio_set_level(GPIO_PP900_DDRPLL_EN, 1); gpio_set_level(GPIO_PP900_PCIE_EN, 1); msleep(2); + gpio_set_level(GPIO_PP1800_PMU_EN_L, 0); gpio_set_level(GPIO_PPVAR_CLOGIC_EN, 1); msleep(2); - gpio_set_level(GPIO_PP1800_PMU_EN_L, 0); gpio_set_level(GPIO_PP1800_USB_EN_L, 0); gpio_set_level(GPIO_PP1800_AP_AVDD_EN_L, 0); msleep(2); @@ -227,9 +227,9 @@ enum power_state power_handle_state(enum power_state state) msleep(10); gpio_set_level(GPIO_PP1800_AP_AVDD_EN_L, 1); gpio_set_level(GPIO_PP1800_USB_EN_L, 1); - gpio_set_level(GPIO_PP1800_PMU_EN_L, 1); msleep(10); gpio_set_level(GPIO_PPVAR_CLOGIC_EN, 0); + gpio_set_level(GPIO_PP1800_PMU_EN_L, 1); msleep(10); gpio_set_level(GPIO_PP900_PCIE_EN, 0); gpio_set_level(GPIO_PP900_DDRPLL_EN, 0);