commit 32075c167a2d4252802645c46378b5d5bea91ae6
author Tom Wai-Hong Tam <waihong@chromium.org> Wed Jan 20 07:09:47 2016 +0800
committer chrome-bot <chrome-bot@chromium.org> Wed Jan 20 14:25:25 2016 -0800
tree f729e9e8cee3390f47d10f64cdf42ea69314c8b4
parent e70bdf1f777b309f0fae91a1ba9c248222d47e5a

faft: Record the RO firmware ID

It is common that the RO version is different with the RW version
once the lab FAFT only updates the RW portion. Better to record
both RW and RO firmware IDs.

BUG=chromium:555709
TEST=Ran a FAFT test and checked the following messages:
17:15:40 INFO | autoserv| System info:
17:15:40 INFO | autoserv| {'ec_version': 'lulu_v1.1.2323-8b1aa45',
17:15:40 INFO | autoserv| 'hwid': 'LULU D25-A2F-A4Q-Q5D',
17:15:40 INFO | autoserv| 'ro_fwid': 'Google_Lulu.6301.136.16',
17:15:40 INFO | autoserv| 'rw_fwid': 'Google_Lulu.6301.136.44',
17:15:40 INFO | autoserv| 'servod_version': 'servod 0.0.1-c76375e'}

Change-Id: I9ae9b8812cab9806f0eec6fe1eec1ecf3b642990
Reviewed-on: https://chromium-review.googlesource.com/322486
Commit-Ready: Wai-Hong Tam <waihong@chromium.org>
Tested-by: Wai-Hong Tam <waihong@chromium.org>
Reviewed-by: danny chan <dchan@chromium.org>

server/cros/faft/firmware_test.py

diff --git a/server/cros/faft/firmware_test.py b/server/cros/faft/firmware_test.py
index 063f617..c43acf2 100644
--- a/server/cros/faft/firmware_test.py
+++ b/server/cros/faft/firmware_test.py
@@ -193,9 +193,10 @@
         This info is used by generate_test_report later.
         """
         system_info = {
-            'fw_version': self.faft_client.ec.get_version(),
             'hwid': self.faft_client.system.get_crossystem_value('hwid'),
-            'fwid': self.faft_client.system.get_crossystem_value('fwid'),
+            'ec_version': self.faft_client.ec.get_version(),
+            'ro_fwid': self.faft_client.system.get_crossystem_value('ro_fwid'),
+            'rw_fwid': self.faft_client.system.get_crossystem_value('fwid'),
             'servod_version': self._client._servo_host.run(
                 'servod --version').stdout.strip(),
         }