diff options
| author | Tim Wawrzynczak <twawrzynczak@chromium.org> | 2020-01-17 13:48:01 -0700 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2020-01-21 04:41:28 +0000 |
| commit | 042c8774bf063eb01f5b836490b6e0961ea2e072 (patch) | |
| tree | e9008e3339c169ede7d6636fa62fe7d0d3bd7711 | |
| parent | bbac44fb762a7ac366e68a738fb3efb61acd4daf (diff) | |
| download | chrome-ec-042c8774bf063eb01f5b836490b6e0961ea2e072.tar.gz | |
ectool: Fix ectool motionsense info command
Commit 267da3cfc increased the response size of the motionsense "info"
command implicitly by adding a new version of the command. ectool
explicitly gets the latest version of the command to use, and the kernel
was complaining because it only requested 16 bytes, but the new version
sends back 19. Hence, update the expected size of the response to
compensate for this.
BUG=b:147472780, b:147720232
BRANCH=firmware-hatch-12672.B
TEST=ectool motionsense info X now returns data instead of failing
Change-Id: Ib297c02f55406b4dc3b8bb0b6b0843165b350995
Signed-off-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2007897
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Reviewed-by: Evan Green <evgreen@chromium.org>
(cherry picked from commit c520344abe270967b1bd96d43559fb00c0d431d4)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/2009524
Reviewed-by: Philip Chen <philipchen@chromium.org>
Commit-Queue: Philip Chen <philipchen@chromium.org>
Tested-by: Philip Chen <philipchen@chromium.org>
| -rw-r--r-- | util/ectool.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/util/ectool.c b/util/ectool.c index 32ebe478e9..90ed0ae435 100644 --- a/util/ectool.c +++ b/util/ectool.c @@ -4469,7 +4469,7 @@ static const struct { sizeof(struct ec_response_motion_sensor_data) * ECTOOL_MAX_SENSOR }, - ST_BOTH_SIZES(info_3), + { ST_PRM_SIZE(info_3), ST_RSP_SIZE(info_4) }, ST_BOTH_SIZES(ec_rate), ST_BOTH_SIZES(sensor_odr), ST_BOTH_SIZES(sensor_range), |