Add host command to set battery vendor parametersstabilize-zako-5712.88.Bstabilize-5712.89.Bstabilize-5712.61.Bstabilize-5712.49.Brelease-R35-5712.B

Also adds 'battparam' console command.

BUG=chrome-os-partner:25145
BRANCH=ToT
TEST=Run 'ectool batteryparam set 0 0x1234'
         'ectool batteryparam get 0'
and on the console:
         'battparam 0'
         'battparam 0 0x1234'
on a board that implements parameter 0.

Change-Id: I9cc54d001631f53dd39ae64cfdeececaa1747181
Original-Change-Id: Ib2812f57f2484309d613b23dab12ad43e0417bd2
Signed-off-by: Dave Parker <dparker@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/195824
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/197162
(cherry picked from commit 4e1472e3037602beec0eb5849ab76fb055d5a657)
Reviewed-on: https://chromium-review.googlesource.com/197590
Commit-Queue: Bernie Thompson <bhthompson@chromium.org>
Tested-by: Bernie Thompson <bhthompson@chromium.org>

1 files changed, 57 insertions, 0 deletions

diff --git a/util/ectool.c b/util/ectool.c
index abe05e7dcb..f8d93ec25d 100644
--- a/util/ectool.c
+++ b/util/ectool.c
@@ -36,6 +36,8 @@ const char help_str[] =
 	"      Prints battery info\n"
 	"  batterycutoff\n"
 	"      Cut off battery output power\n"
+	"  batteryparam\n"
+	"      Read or write board-specific battery parameter\n"
 	"  boardversion\n"
 	"      Prints the board version\n"
 	"  chargecurrentlimit\n"
@@ -2944,6 +2946,60 @@ int cmd_battery_cut_off(int argc, char *argv[])
 	return rv;
 }
 
+int cmd_battery_vendor_param(int argc, char *argv[])
+{
+	struct ec_params_battery_vendor_param p;
+	struct ec_response_battery_vendor_param r;
+	char *e;
+	int rv;
+
+	if (argc < 3)
+		goto cmd_battery_vendor_param_usage;
+
+	if (!strcasecmp(argv[1], "get"))
+		p.mode = BATTERY_VENDOR_PARAM_MODE_GET;
+	else if (!strcasecmp(argv[1], "set"))
+		p.mode = BATTERY_VENDOR_PARAM_MODE_SET;
+	else
+		goto cmd_battery_vendor_param_usage;
+
+	p.param = strtol(argv[2], &e, 0);
+	if (e && *e) {
+		fprintf(stderr, "Invalid param.\n");
+		goto cmd_battery_vendor_param_usage;
+	}
+
+	if (p.mode == BATTERY_VENDOR_PARAM_MODE_SET) {
+		if (argc != 4) {
+			fprintf(stderr, "Missing value.\n");
+			goto cmd_battery_vendor_param_usage;
+		}
+
+		p.value = strtol(argv[3], &e, 0);
+		if (e && *e) {
+			fprintf(stderr, "Invalid value.\n");
+			goto cmd_battery_vendor_param_usage;
+		}
+	}
+
+	rv = ec_command(EC_CMD_BATTERY_VENDOR_PARAM, 0, &p, sizeof(p),
+			&r, sizeof(r));
+
+	if (rv < 0)
+		return rv;
+
+	printf("0x%08x\n", r.value);
+
+	return 0;
+
+cmd_battery_vendor_param_usage:
+	fprintf(stderr,
+		"Usage:\t %s get <param>\n"
+		"\t %s set <param> <value>\n",
+		argv[0], argv[0]);
+	return -1;
+}
+
 int cmd_board_version(int argc, char *argv[])
 {
 	struct ec_response_board_version response;
@@ -3541,6 +3597,7 @@ const struct command commands[] = {
 	{"backlight", cmd_lcd_backlight},
 	{"battery", cmd_battery},
 	{"batterycutoff", cmd_battery_cut_off},
+	{"batteryparam", cmd_battery_vendor_param},
 	{"boardversion", cmd_board_version},
 	{"chargecurrentlimit", cmd_charge_current_limit},
 	{"chargedump", cmd_charge_dump},