commit 8127309516ed6de177929ec0dd129ea07105f207
author Vic Yang <victoryang@chromium.org> Tue Aug 21 15:57:09 2012 +0800
committer Gerrit <chrome-bot@google.com> Tue Aug 21 03:31:28 2012 -0700
tree 72e4e2c49653ab4652344d4c27c337a85c0115a9
parent 19959bce233fc6438285f1e72bb4068146e818d6

FAFT: Fix keyboard developer mode support

Sometimes the EC_IN_RW signal is already set, and we need a cold reset
to clear it. Also change EC reboot from hard reboot to soft reboot.

BUG=chrome-os-partner:12944
TEST=Successfully switch to developer mode

Change-Id: I7e73e7d92d94f549f87ed3a2b67653c3cf869afb
Signed-off-by: Vic Yang <victoryang@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/30983
Reviewed-by: Tom Wai-Hong Tam <waihong@chromium.org>

server/cros/faftsequence.py

diff --git a/server/cros/faftsequence.py b/server/cros/faftsequence.py
index a20582a..242f285 100644
--- a/server/cros/faftsequence.py
+++ b/server/cros/faftsequence.py
@@ -789,7 +789,9 @@
             logging.info('running the customized rec reboot command')
             os.system(self._customized_rec_reboot_command)
         elif self.client_attr.chrome_ec:
-            self.send_uart_command("reboot hard ap-off")
+            # Cold reset to clear EC_IN_RW signal
+            self.servo.cold_reset()
+            self.send_uart_command("reboot ap-off")
             time.sleep(self.EC_BOOT_DELAY)
             self.send_uart_command("hostevent set 0x4000")
             self.servo.power_short_press()