diff options
| author |  Gwendal Grignou <gwendal@chromium.org>
| 2020-08-29 23:59:37 -0700 |
|---|---|---|
| committer |  Enric Balletbo i Serra <enric.balletbo@collabora.com>
| 2020-09-04 09:30:04 +0200 |
| commit | 1e7913ff5f9f1b73146ad8522958bd266f22a510 (patch) | |
| tree | d4304bcb86a5396b69f69b9f2407b2ab2b8b4f24 /drivers/platform/chrome | |
| parent | platform/chrome: cros_ec_trace: Add fields to command traces (diff) | |
| download | linux-1e7913ff5f9f1b73146ad8522958bd266f22a510.tar.gz linux-1e7913ff5f9f1b73146ad8522958bd266f22a510.tar.bz2 linux-1e7913ff5f9f1b73146ad8522958bd266f22a510.zip | |
platform/chrome: cros_ec_lightbar: Reduce ligthbar get version command
By default, the lightbar commands are set to the biggest lightbar command
and response. That length is greater than 128 bytes and may not work on
all machines. But all EC are probed for lightbar by sending a get version
request. Set that request size precisely.
Before the command would be:
cros_ec_cmd: version: 0, command: EC_CMD_LIGHTBAR_CMD, outsize: 194, insize: 128, result: 0
Afer:
cros_ec_cmd: version: 0, command: EC_CMD_LIGHTBAR_CMD, outsize: 1, insize: 8, result: 0
Fixes: a841178445bb7 ("mfd: cros_ec: Use a zero-length array for command data")
Signed-off-by: Gwendal Grignou <gwendal@chromium.org>
Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
Diffstat (limited to 'drivers/platform/chrome')
| -rw-r--r-- | drivers/platform/chrome/cros_ec_lightbar.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/platform/chrome/cros_ec_lightbar.c b/drivers/platform/chrome/cros_ec_lightbar.c index 8445cda57927..de8dfb12e486 100644 --- a/drivers/platform/chrome/cros_ec_lightbar.c +++ b/drivers/platform/chrome/cros_ec_lightbar.c @@ -116,6 +116,8 @@ static int get_lightbar_version(struct cros_ec_dev *ec, param = (struct ec_params_lightbar *)msg->data; param->cmd = LIGHTBAR_CMD_VERSION; + msg->outsize = sizeof(param->cmd); + msg->result = sizeof(resp->version); ret = cros_ec_cmd_xfer_status(ec->ec_dev, msg); if (ret < 0 && ret != -EINVAL) { ret = 0; |