diff options
| author | Vadim Bendebury <vbendeb@chromium.org> | 2013-06-19 16:48:32 -0700 |
|---|---|---|
| committer | ChromeBot <chrome-bot@google.com> | 2013-06-20 15:51:35 -0700 |
| commit | b839e6524193b61bad0f5ce935b45f48e26d23c0 (patch) | |
| tree | f86626542f3dd154bbecdf5a5de9aa39a50ec155 | |
| parent | e74e60c4651111a66380d99683ecd5cf9d7dbfb2 (diff) | |
| download | chrome-ec-b839e6524193b61bad0f5ce935b45f48e26d23c0.tar.gz | |
drop: Ignore command version for VBNV CONTEXT command
The ec driver does not yet pass the command version properly from app
to the EC. This causes a failure when trying to use the vbnvram
context command (23). The thing is that the command version is not
even important for this command, as it was never implemented
differently.
As a quick fix - mark the command descriptor as supporting both
versions. I wonder if we should implement a "don't care" option for
situations like this.
BRANCH=none
BUG=chromium:239197
TEST=manual
. on peach_pit:
localhost ~ # mosys nvram vboot read
70000000000000000000000000000020
localhost ~ #
Change-Id: I16fcc0d6752d9e778a026717208d8d6487d5dc77
Signed-off-by: Vadim Bendebury <vbendeb@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/59348
| -rw-r--r-- | common/system_common.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/common/system_common.c b/common/system_common.c index 03febba603..176977930e 100644 --- a/common/system_common.c +++ b/common/system_common.c @@ -840,9 +840,15 @@ int host_command_vbnvcontext(struct host_cmd_handler_args *args) return EC_RES_SUCCESS; } + +/* + * TODO(crbug.com/239197) : Adding both versions to the version mask is a + * temporary workaround for a problem in the cros_ec driver. Drop + * EC_VER_MASK(0) once cros_ec driver can send the correct version. + */ DECLARE_HOST_COMMAND(EC_CMD_VBNV_CONTEXT, host_command_vbnvcontext, - EC_VER_MASK(EC_VER_VBNV_CONTEXT)); + EC_VER_MASK(EC_VER_VBNV_CONTEXT) | EC_VER_MASK(0)); int host_command_reboot(struct host_cmd_handler_args *args) { |