Re: [PATCH] platform/chrome: cros_ec_debugfs: Use cros_ec_cmd_xfer_status helper

[Guenter Roeck]

On Mon, Apr 15, 2019 at 2:35 PM Enric Balletbo i Serra
<enric.balletbo@xxxxxxxxxxxxx> 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@xxxxxxxxxxxxx>


Reviewed-by: Guenter Roeck <linux@xxxxxxxxxxxx>

> ---
>
>  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
>