cros_ec_commands.h in v3.18 kernel is out-of-sync with EC's ec_commands.h |
|||||
Issue descriptionWhat steps will reproduce the problem? (1)Get to a console on a Caroline running top-of-tree software. (2)execute command "ectool motionsense fifo_info" (3)note the failure response : "ioctl -1, errno 90 (Message too long), EC result 0 (success)" What is the expected result? The ectool motionsense command should work. What happens instead? Command failed due to mismatch in parameters structure sizes.
,
Apr 21 2017
Fix up for review : https://chromium-review.googlesource.com/#/c/484746/
,
Apr 25 2017
,
May 18 2017
,
May 19 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/kernel/+/2002b8cac2d411a091547cf563997984dc48c503 commit 2002b8cac2d411a091547cf563997984dc48c503 Author: Nick Vaccaro <nvaccaro@chromium.org> Date: Fri May 19 23:19:39 2017 CHROMIUM: mfd: cros-ec: bring in EC command definitions We are adding support for new sensor api, so we need to bring in new EC command definitions. To do that we are fetching ec_command.h at <c5332f6fe7d4d5197ac2154319f5e7dd4436d62a> from the EC repository, which contains canonical version of the file. The new alignment changes brought in are not meant to change the existing layout, but rather document it/make it explicit. BUG= chromium:713814 TEST=Built/boot on caroline Change-Id: I2161aa2c444d13420029515188e1dd82bdd1c52c Signed-off-by: Nick Vaccaro <nvaccaro@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/484746 Reviewed-by: Dmitry Torokhov <dtor@chromium.org> [modify] https://crrev.com/2002b8cac2d411a091547cf563997984dc48c503/include/linux/mfd/cros_ec_commands.h
,
May 20 2017
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 Deleted