Messages in this thread | | | From | Guenter Roeck <> | Date | Mon, 6 Jun 2022 08:46:04 -0700 | Subject | Re: [PATCH 05/13] platform/chrome: cros_ec_proto: remove redundant NULL check |
| |
On Mon, Jun 6, 2022 at 7:12 AM Tzung-Bi Shih <tzungbi@kernel.org> wrote: > > send_command() already checks if `ec_dev->pkt_xfer` is NULL. Remove the > redundant check. > > Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Guenter Roeck <groeck@chromium.org>
> --- > drivers/platform/chrome/cros_ec_proto.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/platform/chrome/cros_ec_proto.c b/drivers/platform/chrome/cros_ec_proto.c > index 629dce3e6ab3..1b851dcd20c9 100644 > --- a/drivers/platform/chrome/cros_ec_proto.c > +++ b/drivers/platform/chrome/cros_ec_proto.c > @@ -281,9 +281,6 @@ static int cros_ec_host_command_proto_query(struct cros_ec_device *ec_dev, > */ > int ret; > > - if (!ec_dev->pkt_xfer) > - return -EPROTONOSUPPORT; > - > memset(msg, 0, sizeof(*msg)); > msg->command = EC_CMD_PASSTHRU_OFFSET(devidx) | EC_CMD_GET_PROTOCOL_INFO; > msg->insize = sizeof(struct ec_response_get_protocol_info); > -- > 2.36.1.255.ge46751e96f-goog >
| |