diff options
author | Randall Spangler <rspangler@chromium.org> | 2014-02-06 13:14:16 -0800 |
---|---|---|
committer | Randall Spangler <rspangler@chromium.org> | 2014-02-07 17:24:37 +0000 |
commit | 93cbb026fddf926056532e01e154167205055cde (patch) | |
tree | 03584a29149bdf6c99d9ec6424cc8f87044ace68 /chip | |
parent | 3652dc7ab670ab918c9eca5c1855609c69008fa6 (diff) | |
download | chrome-ec-93cbb026fddf926056532e01e154167205055cde.tar.gz |
lm4: Remove 500k clock delay in clock_init()
We copied that delay because it seemed to be necessary on early LM4
chips to avoid glitching the UART. But on current boards (e.g. rambi)
this does not seem to be necessary, and delays boot by 31ms. So,
remove the delay.
BUG=chrome-os-partner:23794
BRANCH=rambi
TEST=boot system; see little to no glitching on EC uart, and system boots ok
hibernate 1; see little to no glitching on EC uart, and system boots ok
Orig-Change-Id: I9d4b5927da4282e47e1b09be838104c64f25268c
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/185232
Reviewed-by: Bill Richardson <wfrichar@chromium.org>
(cherry picked from commit 36d4ecb153518623945f0a8e89e4413938d38869)
Change-Id: Ica9447e527da9d2098c17890ee63deacb362c166
Reviewed-on: https://chromium-review.googlesource.com/185346
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Commit-Queue: Randall Spangler <rspangler@chromium.org>
Tested-by: Randall Spangler <rspangler@chromium.org>
Diffstat (limited to 'chip')
-rw-r--r-- | chip/lm4/clock.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/chip/lm4/clock.c b/chip/lm4/clock.c index 4fa72c5ca1..04b86fae92 100644 --- a/chip/lm4/clock.c +++ b/chip/lm4/clock.c @@ -167,16 +167,6 @@ void clock_init(void) LM4_SYSTEM_MOSCCTL = 0x04; #endif - /* - * TODO(crosbug.com/p/23794): UART seems to glitch unless we wait 500k - * cycles before enabling the PLL, but only if this is a cold boot. - * Why? UART doesn't even use the PLL'd system clock. I've heard - * rumors the Stellaris ROM library does this too, but why? Revisit on - * current systems to see if this is is still needed. - */ - if (!system_jumped_to_this_image()) - clock_wait_cycles(500000); - /* Make sure PLL is disabled */ disable_pll(); } |