diff options
author | Randall Spangler <rspangler@chromium.org> | 2012-07-12 10:36:18 -0700 |
---|---|---|
committer | Gerrit <chrome-bot@google.com> | 2012-07-12 15:50:14 -0700 |
commit | c304ff7d81a2a3072e5388584b232413a76f3343 (patch) | |
tree | 0966fab51e692912cd4c30fa07df3aaa8960c405 /common/host_command.c | |
parent | c171fbe3dcc2c9982b7de5ff7d2b5df6dcee542c (diff) | |
download | chrome-ec-c304ff7d81a2a3072e5388584b232413a76f3343.tar.gz |
Use response pointer and size from host command args struct
Saves 2 params being passed around needlessly.
BUG=chrome-os-partner:11275
TEST=mkbp hash from u-boot console should still work
Change-Id: I958e4a09f16413e4d051e278dc0384aa9b791aa4
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/27312
Diffstat (limited to 'common/host_command.c')
-rw-r--r-- | common/host_command.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/common/host_command.c b/common/host_command.c index 3e0ea06d5d..158aa79726 100644 --- a/common/host_command.c +++ b/common/host_command.c @@ -49,7 +49,7 @@ void host_command_received(struct host_cmd_handler_args *args) if (args->command == EC_CMD_REBOOT) { system_reset(1); /* Reset should never return; if it does, post an error */ - host_send_response(EC_RES_ERROR, NULL, 0); + host_send_response(EC_RES_ERROR); return; } @@ -225,8 +225,7 @@ void host_command_task(void) /* process it */ if ((evt & TASK_EVENT_CMD_PENDING) && pending_args) { enum ec_status res = host_command_process(pending_args); - host_send_response(res, pending_args->response, - pending_args->response_size); + host_send_response(res); } } } |