Add host command to control charge state v2

This replaces the obsolete and temporary (ha!) EC_CMD_CHARGE_DUMP host
command with EC_CMD_CHARGE_STATE. This is used to monitor and adjust the new
charge state implementation, including any board-specific customizations.

This command is a single catch-all command with multiple subcommands
(similar to EC_CMD_LIGHTBAR_CMD) so that we don't have to keep adding new
top-level host commands just to support incremental changes.

BUG=chrome-os-partner:23776
BRANCH=ToT
TEST=manual

From the AP, try these commands:

  ectool chargestate show
  ectool chargestate param
  ectool chargestate param <NUM>
  ectool chargestate param <NUM> <VALUE>

Watch the EC console and use its "chg" command to verify the effects of
setting various params.

Note: the Samus-specific fast-charging profile override is param 0x10000.
You can check it with the EC console "fastcharge" command.

Change-Id: Iad2f773a085bc25c05073b3eed9866f122ae9d78
Signed-off-by: Bill Richardson <wfrichar@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/193305
Reviewed-by: Randall Spangler <rspangler@chromium.org>

1 files changed, 0 insertions, 17 deletions

diff --git a/common/charge_state_v1.c b/common/charge_state_v1.c
index d08aa2430b..1de56e34fa 100644
--- a/common/charge_state_v1.c
+++ b/common/charge_state_v1.c
@@ -979,23 +979,6 @@ static int charge_command_charge_control(struct host_cmd_handler_args *args)
 DECLARE_HOST_COMMAND(EC_CMD_CHARGE_CONTROL, charge_command_charge_control,
 		     EC_VER_MASK(0) | EC_VER_MASK(1));
 
-static int charge_command_dump(struct host_cmd_handler_args *args)
-{
-	char *dest = (char *)args->response;
-
-	if (system_is_locked())
-		return EC_RES_ACCESS_DENIED;
-
-	ASSERT(sizeof(task_ctx) <= args->response_max);
-
-	memcpy(dest, &task_ctx, sizeof(task_ctx));
-	args->response_size = sizeof(task_ctx);
-
-	return EC_RES_SUCCESS;
-}
-DECLARE_HOST_COMMAND(EC_CMD_CHARGE_DUMP, charge_command_dump,
-		     EC_VER_MASK(0));
-
 static void reset_current_limit(void)
 {
 	user_current_limit = -1;