diff options
author | Andrew Bresticker <abrestic@chromium.org> | 2013-08-22 14:15:01 -0700 |
---|---|---|
committer | ChromeBot <chrome-bot@google.com> | 2013-08-22 15:00:00 -0700 |
commit | 67efd2c100f9de6fe34d381f6145e2d795d8549f (patch) | |
tree | a18dccf94e3f415f26cfb8b5b099119a83a7228a | |
parent | 10655b03fd66020bc7d44fe4d44ad415cb0480a0 (diff) | |
download | chrome-ec-67efd2c100f9de6fe34d381f6145e2d795d8549f.tar.gz |
Don't send IN_PROGRESS response if GET_COMMS_STATUS is unsupported
If the EC responds to the AP with IN_PROGRESS, the AP will then use
GET_COMMS_STATUS to poll for completion of the previous command.
If the EC doesn't support GET_COMMS_STATUS, it will return an error,
confusing the AP.
BUG=chrome-os-partner:20978
TEST=flashrom on Pit
BRANCH=pit
Change-Id: I7c911ebc047042450b1eefc992ba2250d35fa078
Signed-off-by: Andrew Bresticker <abrestic@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/66702
Reviewed-by: Randall Spangler <rspangler@chromium.org>
-rw-r--r-- | common/flash_common.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/common/flash_common.c b/common/flash_common.c index 082f64fb75..3dc4e2b824 100644 --- a/common/flash_common.c +++ b/common/flash_common.c @@ -578,7 +578,7 @@ static int flash_command_erase(struct host_cmd_handler_args *args) return EC_RES_ACCESS_DENIED; /* Indicate that we might be a while */ -#ifdef HAS_TASK_HOSTCMD +#if defined(HAS_TASK_HOSTCMD) && defined(CONFIG_HOST_COMMAND_STATUS) args->result = EC_RES_IN_PROGRESS; host_send_response(args); #endif |