commit f2b1d01801824a9a6fdeb1e222f993778f4372fe
author Todd Broch <tbroch@chromium.org> Thu Jun 14 12:55:21 2012 -0700
committer Gerrit <chrome-bot@google.com> Fri Jun 15 13:18:43 2012 -0700
tree 6110340e67942bc4d547f0e585c298a4326156f9
parent ec4120588dfaed180a33066d726d0dc4f8c6c9f1

autotest: faft_client: fixup for platforms without EC

Some platforms (desktops currently) don't require an embedded
controller (EC).  CL adds check using 'mosys ec info' to confirm
presence (or not) of EC

BUG=chrome-os-partner:10543
TEST=manual,

on dut w/o EC,
  python /usr/local/autotest/cros/faft_client.py

and it works.

Change-Id: I254689a399a27097af2e21cb2e4909fa34022be8
Reviewed-on: https://gerrit.chromium.org/gerrit/25310
Reviewed-by: Todd Broch <tbroch@chromium.org>
Tested-by: Todd Broch <tbroch@chromium.org>

client/cros/faft_client.py

diff --git a/client/cros/faft_client.py b/client/cros/faft_client.py
index ed15d10..3fe37c8 100644
--- a/client/cros/faft_client.py
+++ b/client/cros/faft_client.py
@@ -63,13 +63,16 @@
                                 'bios')
         self._bios_handler.new_image()
 
-        self._ec_handler = flashrom_handler.FlashromHandler()
-        self._ec_handler.init(saft_flashrom_util,
-                              self._chromeos_interface,
-                              'ec_root_key.vbpubk',
-                              '/usr/share/vboot/devkeys',
-                              'ec')
-        self._ec_handler.new_image()
+        self._ec_handler = None
+        if not os.system("mosys ec info"):
+            self._ec_handler = flashrom_handler.FlashromHandler()
+            self._ec_handler.init(saft_flashrom_util,
+                                  self._chromeos_interface,
+                                  'ec_root_key.vbpubk',
+                                  '/usr/share/vboot/devkeys',
+                                  'ec')
+            self._ec_handler.new_image()
+
 
         self._kernel_handler = kernel_handler.KernelHandler()
         # TODO(waihong): The dev_key_path is a new argument. We do that in