From bcb9428312f2f053b2f9da642d658d01ecec085a Mon Sep 17 00:00:00 2001 From: Alec Berg Date: Fri, 6 Jun 2014 17:23:04 -0700 Subject: samus: enable PP3300_ACCEL_EN by default Keep accelerometer power enabled all the time. It consumes very little power, we will need it to be on all the time for tap for battery, and when it is off, it can cause problems because the PP3300_EC rail leaks through the i2c pull-ups into the accelerometer. With this change, we should never see the bug in which i2c bus 1 is getting a lot of errors on boot. BUG=chrome-os-partner:29003 BRANCH=none TEST=tested on multiple samus units. can talk to accel using i2cxfer console command, and never saw any bus problems. Change-Id: I2034e217fbb1157cc0f9b867ef50e7932d75c761 Signed-off-by: Alec Berg Reviewed-on: https://chromium-review.googlesource.com/202988 Reviewed-by: Vincent Palatin --- board/samus/board.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/board/samus/board.c b/board/samus/board.c index fbee2bc72c..000c1f0db7 100644 --- a/board/samus/board.c +++ b/board/samus/board.c @@ -110,7 +110,7 @@ const struct gpio_info gpio_list[] = { {"PP3300_DSW_GATED_EN", LM4_GPIO_J, (1<<3), GPIO_OUT_LOW, NULL}, {"PP3300_LTE_EN", LM4_GPIO_D, (1<<2), GPIO_OUT_LOW, NULL}, {"PP3300_WLAN_EN", LM4_GPIO_J, (1<<0), GPIO_OUT_LOW, NULL}, - {"PP3300_ACCEL_EN", LM4_GPIO_J, (1<<1), GPIO_OUT_LOW, NULL}, + {"PP3300_ACCEL_EN", LM4_GPIO_J, (1<<1), GPIO_OUT_HIGH, NULL}, {"PP1050_EN", LM4_GPIO_C, (1<<7), GPIO_OUT_LOW, NULL}, {"PP5000_USB_EN", LM4_GPIO_C, (1<<5), GPIO_OUT_LOW, NULL}, {"PP5000_EN", LM4_GPIO_H, (1<<7), GPIO_OUT_LOW, NULL}, -- cgit v1.2.1