From cdc59faa85f342a4c90ef784a755a74066543ea8 Mon Sep 17 00:00:00 2001 From: Scott Date: Mon, 6 Feb 2017 18:38:11 -0800 Subject: [PATCH] cr50: Remove usleep() call from ina_enable function Since the INAs are now enabled based on rdd_attach, there is no need to have a a delay following enabling the 3.3V rail and the delay can't be a usleep() because ina_connect() is not being called from rdd_attach which is running in an interrupt context. BUG=chrome-os-partner:62375 BRANCH=none TEST=manual The usleep() call was causing a reboot but was only visible when using UART console. Verified that with this CL the reboot was no longer happening. Change-Id: Id9bbef4e89dee39a2dfa443c3af54c1f9a611ebd Signed-off-by: Scott Reviewed-on: https://chromium-review.googlesource.com/438675 Reviewed-by: Mary Ruthven Commit-Queue: Vadim Bendebury Trybot-Ready: Vadim Bendebury --- board/cr50/usb_i2c.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/board/cr50/usb_i2c.c b/board/cr50/usb_i2c.c index 786a886c7f..24a6fbdea6 100644 --- a/board/cr50/usb_i2c.c +++ b/board/cr50/usb_i2c.c @@ -42,8 +42,6 @@ static void ina_connect(void) /* Apply power to INA chips */ gpio_set_level(GPIO_EN_PP3300_INA_L, 0); - /* Allow enough time for power rail to come up */ - usleep(25); /* * Connect B0/B1 pads to I2C0 input SDA/SCL. Note, that the inputs