summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSameer Nanda <snanda@chromium.org>2014-10-10 08:57:24 -0700
committerSameer Nanda <snanda@chromium.org>2014-10-15 02:53:45 +0000
commitb82a3406f4c0fad43946e84ab69cb424700fcef6 (patch)
tree6586f58a7a3a426d0ed45d90ad504ea31b9fd779
parent41b927442e770d6f2303d32f88a96b859775b27f (diff)
downloadchrome-ec-b82a3406f4c0fad43946e84ab69cb424700fcef6.tar.gz
temp_metrics: add "--" when invoking ectool tmp006cal
ectool has switched over to using getopt for command line options parsing. This breaks temp_metrics' invocation of "ectool tmp006cal" command since some of the coefficients are negative and therefore start with "-". getopt treats that as another command line option causing ectool to barf. BUG=chromium:422160 TEST=On a Pixel issue "sudo start temp_metrics" and check /var/log/messages to ensure that no new messages such as "init: temp_metrics main process ended, respawning" show up. BRANCH=none Original-Change-Id: I42232b3027ec6339814d226f1d8ab493e3420eea Signed-off-by: Sameer Nanda <snanda@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/222845 Reviewed-by: Randall Spangler <rspangler@chromium.org> (cherry picked from commit 63c41f239223cb343978635c7409ee341eeb08d8) Change-Id: Idd8e1653a5c77b0e03adb5306b81bcb3b8b97472 Reviewed-on: https://chromium-review.googlesource.com/223176 Reviewed-by: Randall Spangler <rspangler@chromium.org> Commit-Queue: Sameer Nanda <snanda@chromium.org> Tested-by: Sameer Nanda <snanda@chromium.org>
-rw-r--r--util/temp_metrics.conf4
1 files changed, 3 insertions, 1 deletions
diff --git a/util/temp_metrics.conf b/util/temp_metrics.conf
index 307938d019..bf10ee0907 100644
--- a/util/temp_metrics.conf
+++ b/util/temp_metrics.conf
@@ -107,7 +107,9 @@ script
CHARGER_D="3 $CHARGER_D_S0 $B0 $B1 $B2"
for i in "$USB_C" "$PCH_D" "$HINGE_C" "$CHARGER_D"; do
- ectool tmp006cal $i
+ # Add "--" otherwise ectool will barf when trying to parse negative
+ # coefficients.
+ ectool tmp006cal -- $i
done
}