commit 7334feb3e2c5aab69ac1dd9a03e24b9e6caf91e9
author Chun-Ta Lin <itspeter@google.com> Mon Oct 02 12:05:53 2017 +0800
committer chrome-bot <chrome-bot@chromium.org> Mon Oct 02 23:28:27 2017 -0700
tree 453e4cdbe70db7ea3856f5682ef63ff823b32525
parent f8c21bfdbc6e43bc0951f37b1753e1b3f6cfdf2e

hammerd: Align the output of TouchpadInfo to sysfs

In the standard Elan sysfs output, drivers/input/mouse/elan_i2c_core.c,
elan_sysfs_read_product_id() use ETP_PRODUCT_ID_FORMAT_STRING, which is
"%d.0". In addition, elan_sysfs_read_fw_ver() use "%d.0\n" as well.

BUG=b:65534217
TEST=FEATURES=test emerge-poppy hammerd

Change-Id: Ib3302624f1333cd7c3411d536915c7564ee29bfb
Reviewed-on: https://chromium-review.googlesource.com/694943
Commit-Ready: Chun-ta Lin <itspeter@chromium.org>
Tested-by: Chun-ta Lin <itspeter@chromium.org>
Reviewed-by: Chih-Yu Huang <akahuang@chromium.org>
Reviewed-by: Nicolas Boichat <drinkcat@chromium.org>

hammerd/hammer_updater.cc

diff --git a/hammerd/hammer_updater.cc b/hammerd/hammer_updater.cc
index dc93266..cd83ab0 100644
--- a/hammerd/hammer_updater.cc
+++ b/hammerd/hammer_updater.cc
@@ -414,11 +414,11 @@
   LOG(INFO) << "status: 0x" << std::hex << static_cast<int>(response.status);
   LOG(INFO) << "vendor: 0x" << std::hex << response.vendor;
   LOG(INFO) << "fw_address: 0x" << std::hex << response.fw_address;
-  LOG(INFO) << "fw_size: 0x" << std::hex << response.fw_size;
+  LOG(INFO) << "fw_size: " << response.fw_size << " bytes";
   LOG(INFO) << "allowed_fw_hash: 0x" <<
       base::HexEncode(response.allowed_fw_hash, SHA256_DIGEST_LENGTH);
-  LOG(INFO) << "id: 0x" << std::hex << response.elan.id;
-  LOG(INFO) << "fw_version: 0x" << std::hex << response.elan.fw_version;
+  LOG(INFO) << "product_id: " << response.elan.id << ".0";
+  LOG(INFO) << "fw_ver: " << response.elan.fw_version << ".0";
   LOG(INFO) << "fw_checksum: 0x" << std::hex << response.elan.fw_checksum;
 
   // Check if the image size matches IC size.