Messages in this thread | | | From | Guenter Roeck <> | Date | Mon, 15 Apr 2019 14:37:30 -0700 | Subject | Re: [PATCH] platform/chrome: cros_ec_debugfs: Use cros_ec_cmd_xfer_status helper |
| |
On Mon, Apr 15, 2019 at 2:35 PM Enric Balletbo i Serra <enric.balletbo@collabora.com> wrote: > > This patch makes use of cros_ec_cmd_xfer_status() instead of > cros_ec_cmd_xfer() so we can remove some redundant code. > > Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
> --- > > drivers/platform/chrome/cros_ec_debugfs.c | 30 ++++++----------------- > 1 file changed, 8 insertions(+), 22 deletions(-) > > diff --git a/drivers/platform/chrome/cros_ec_debugfs.c b/drivers/platform/chrome/cros_ec_debugfs.c > index a712508e4646..0bd163723548 100644 > --- a/drivers/platform/chrome/cros_ec_debugfs.c > +++ b/drivers/platform/chrome/cros_ec_debugfs.c > @@ -72,15 +72,9 @@ static void cros_ec_console_log_work(struct work_struct *__work) > int buf_space; > int ret; > > - ret = cros_ec_cmd_xfer(ec->ec_dev, &snapshot_msg); > - if (ret < 0) { > - dev_err(ec->dev, "EC communication failed\n"); > - goto resched; > - } > - if (snapshot_msg.result != EC_RES_SUCCESS) { > - dev_err(ec->dev, "EC failed to snapshot the console log\n"); > + ret = cros_ec_cmd_xfer_status(ec->ec_dev, &snapshot_msg); > + if (ret < 0) > goto resched; > - } > > /* Loop until we have read everything, or there's an error. */ > mutex_lock(&debug_info->log_mutex); > @@ -95,16 +89,10 @@ static void cros_ec_console_log_work(struct work_struct *__work) > > memset(read_params, '\0', sizeof(*read_params)); > read_params->subcmd = CONSOLE_READ_RECENT; > - ret = cros_ec_cmd_xfer(ec->ec_dev, debug_info->read_msg); > - if (ret < 0) { > - dev_err(ec->dev, "EC communication failed\n"); > - break; > - } > - if (debug_info->read_msg->result != EC_RES_SUCCESS) { > - dev_err(ec->dev, > - "EC failed to read the console log\n"); > + ret = cros_ec_cmd_xfer_status(ec->ec_dev, > + debug_info->read_msg); > + if (ret < 0) > break; > - } > > /* If the buffer is empty, we're done here. */ > if (ret == 0 || ec_buffer[0] == '\0') > @@ -290,9 +278,8 @@ static int ec_read_version_supported(struct cros_ec_dev *ec) > params->cmd = EC_CMD_CONSOLE_READ; > response = (struct ec_response_get_cmd_versions *)msg->data; > > - ret = cros_ec_cmd_xfer(ec->ec_dev, msg) >= 0 && > - msg->result == EC_RES_SUCCESS && > - (response->version_mask & EC_VER_MASK(1)); > + ret = cros_ec_cmd_xfer_status(ec->ec_dev, msg) >= 0 && > + response->version_mask & EC_VER_MASK(1); > > kfree(msg); > > @@ -372,9 +359,8 @@ static int cros_ec_create_panicinfo(struct cros_ec_debugfs *debug_info) > msg->command = EC_CMD_GET_PANIC_INFO; > msg->insize = insize; > > - ret = cros_ec_cmd_xfer(ec_dev, msg); > + ret = cros_ec_cmd_xfer_status(ec_dev, msg); > if (ret < 0) { > - dev_warn(debug_info->ec->dev, "Cannot read panicinfo.\n"); > ret = 0; > goto free; > } > -- > 2.20.1 >
| |