diff options
| author | Randall Spangler <rspangler@chromium.org> | 2013-11-21 15:52:22 -0800 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2013-12-03 22:15:25 +0000 |
| commit | 804b24cc81ca2f954eef45695d8084d45827c061 (patch) | |
| tree | c7ee325076729c59da948528312da1e9038b0e38 /power/baytrail.c | |
| parent | 3b0a4055d17c17e4ee8df474858b40ffda518c31 (diff) | |
| download | chrome-ec-804b24cc81ca2f954eef45695d8084d45827c061.tar.gz | |
rambi: Remove proto1 workaround which leaves PP5000_EN on
proto1.5 boards should not need this workaround.
BUG=chrome-os-partner:23673
BRANCH=none
TEST=boot a proto1.5 board -> boots
apshutdown -> off
gpioget PP5000_EN -> 0
powerbtn -> boots
gpioget PP5000_EN -> 1
Change-Id: Ie7bb962a9be0934506a6a5d0aefda0282ebb15ec
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/177668
Reviewed-by: Aaron Durbin <adurbin@chromium.org>
Diffstat (limited to 'power/baytrail.c')
| -rw-r--r-- | power/baytrail.c | 17 |
1 files changed, 3 insertions, 14 deletions
diff --git a/power/baytrail.c b/power/baytrail.c index 258a2ea414..3cf166c8ad 100644 --- a/power/baytrail.c +++ b/power/baytrail.c @@ -126,12 +126,7 @@ enum x86_state x86_chipset_init(void) gpio_set_level(GPIO_SUSP_VR_EN, 0); gpio_set_level(GPIO_PP1350_EN, 0); gpio_set_level(GPIO_PP3300_DX_EN, 0); - /* - * TODO(crosbug.com/p/23673): turn off PP5000 after - * Rev.1 hardware is obsolete: - * - * gpio_set_level(GPIO_PP5000_EN, 0); - */ + gpio_set_level(GPIO_PP5000_EN, 0); gpio_set_level(GPIO_PCH_RSMRST_L, 0); gpio_set_level(GPIO_PCH_SYS_PWROK, 0); wireless_enable(0); @@ -250,12 +245,7 @@ enum x86_state x86_handle_state(enum x86_state state) chipset_force_shutdown(); wireless_enable(0); gpio_set_level(GPIO_PP3300_DX_EN, 0); - /* - * TODO(crosbug.com/p/23673): turn off PP5000 after - * Rev.1 hardware is obsolete: - * - * gpio_set_level(GPIO_PP5000_EN, 0); - */ + gpio_set_level(GPIO_PP5000_EN, 0); gpio_set_level(GPIO_TOUCHSCREEN_RESET_L, 0); return X86_S3; } @@ -320,8 +310,7 @@ enum x86_state x86_handle_state(enum x86_state state) /* Turn off power rails */ gpio_set_level(GPIO_PP3300_DX_EN, 0); - /* TODO(crosbug.com/p/23673): turn off PP5000 after rev.1 */ - /* gpio_set_level(GPIO_PP5000_EN, 0); */ + gpio_set_level(GPIO_PP5000_EN, 0); return X86_S3; case X86_S3S5: |