diff options
| author | Aseda Aboagye <aaboagye@google.com> | 2018-08-20 08:28:04 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2018-08-20 21:43:06 -0700 |
| commit | b059ccdead89713485fc031faa622115d0839d57 (patch) | |
| tree | bf1d4d500fbc36b63b0807479d35bbf5c2a77e75 /util/ectool.c | |
| parent | 4b490cbb31aa9cb817864cf525fce21bf2943c41 (diff) | |
| download | chrome-ec-b059ccdead89713485fc031faa622115d0839d57.tar.gz | |
ectool: Fix ADC read return value.
This commit fixes a bug with the ectool adcread command. For successful
reads, it was returning a non-zero return code.
BUG=b:76155036
BRANCH=None
TEST=Use `ectool adcread` on non-existent ADC channel, verify return
code is non-zero. Use it on a valid channel, verify the return code is
0.
Change-Id: Ida8208bf100a3d66bd132770fb352678f408560c
Signed-off-by: Aseda Aboagye <aaboagye@google.com>
Reviewed-on: https://chromium-review.googlesource.com/1181173
Commit-Ready: Aseda Aboagye <aaboagye@chromium.org>
Tested-by: Aseda Aboagye <aaboagye@chromium.org>
Reviewed-by: Edward Hill <ecgh@chromium.org>
Diffstat (limited to 'util/ectool.c')
| -rw-r--r-- | util/ectool.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/util/ectool.c b/util/ectool.c index e6332373c3..c2ce4dc8da 100644 --- a/util/ectool.c +++ b/util/ectool.c @@ -405,8 +405,10 @@ int cmd_adc_read(int argc, char *argv[]) } rv = ec_command(EC_CMD_ADC_READ, 0, &p, sizeof(p), &r, sizeof(r)); - if (rv > 0) + if (rv > 0) { printf("%s: %d\n", argv[1], r.adc_value); + return 0; + } return rv; } |