From ea188ffdb7dd75e3bfc50c248436bd29388b90e1 Mon Sep 17 00:00:00 2001 From: Randall Spangler Date: Thu, 31 Oct 2013 09:50:42 -0700 Subject: [PATCH] cleanup: TODOs in ec_commands.h Comment changes only; no code changes. BUG=chrome-os-partner:11223 BRANCH=none TEST=build all boards; pass unit tests Change-Id: I8f6f57514886dd1fda0e44cae1b036c22be481f8 Signed-off-by: Randall Spangler Reviewed-on: https://chromium-review.googlesource.com/175228 Reviewed-by: Bill Richardson --- common/host_command.c | 1 + include/ec_commands.h | 13 +++++++++---- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/common/host_command.c b/common/host_command.c index 34c274001a..a0d703b90a 100644 --- a/common/host_command.c +++ b/common/host_command.c @@ -372,6 +372,7 @@ void host_command_task(void) /*****************************************************************************/ /* Host commands */ +/* TODO(crosbug.com/p/11223): Remove this once the kernel no longer cares */ static int host_command_proto_version(struct host_cmd_handler_args *args) { struct ec_response_proto_version *r = args->response; diff --git a/include/ec_commands.h b/include/ec_commands.h index 9af502ca9a..4ed4890f0b 100644 --- a/include/ec_commands.h +++ b/include/ec_commands.h @@ -8,8 +8,13 @@ #ifndef __CROS_EC_COMMANDS_H #define __CROS_EC_COMMANDS_H -/* Current version of this protocol */ -/* TODO: This is effectively useless; protocol is determined in other ways */ +/* + * Current version of this protocol + * + * TODO(crosbug.com/p/11223): This is effectively useless; protocol is + * determined in other ways. Remove this once the kernel code no longer + * depends on it. + */ #define EC_PROTO_VERSION 0x00000002 /* Command version mask */ @@ -1764,8 +1769,8 @@ struct ec_params_sb_wr_block { /* System commands */ /* - * TODO: this is a confusing name, since it doesn't necessarily reboot the EC. - * Rename to "set image" or something similar. + * TODO(crosbug.com/p/23747): This is a confusing name, since it doesn't + * necessarily reboot the EC. Rename to "image" or something similar? */ #define EC_CMD_REBOOT_EC 0xd2