platform/chrome: cros_ec_lpc: remove redundant pointer request
authorColin Ian King <colin.king@canonical.com>
Tue, 31 Oct 2017 10:27:47 +0000 (10:27 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 5 Jun 2018 09:41:57 +0000 (11:41 +0200)
commit d3b56c566d4ba8cae688baf3cca94425d57ea783 upstream.

Pointer request is being assigned but never used, so remove it. Cleans
up the clang warning:

drivers/platform/chrome/cros_ec_lpc.c:68:2: warning: Value stored to
'request' is never read

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Benson Leung <bleung@chromium.org>
Cc: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/platform/chrome/cros_ec_lpc.c

index 1baf720faf690abda2b769a338b6bd73782738c8..87e9747d229afe330a8e1b0b671b86c413fd0bd1 100644 (file)
@@ -54,7 +54,6 @@ static int ec_response_timed_out(void)
 static int cros_ec_pkt_xfer_lpc(struct cros_ec_device *ec,
                                struct cros_ec_command *msg)
 {
-       struct ec_host_request *request;
        struct ec_host_response response;
        u8 sum;
        int ret = 0;
@@ -65,8 +64,6 @@ static int cros_ec_pkt_xfer_lpc(struct cros_ec_device *ec,
        /* Write buffer */
        cros_ec_lpc_write_bytes(EC_LPC_ADDR_HOST_PACKET, ret, ec->dout);
 
-       request = (struct ec_host_request *)ec->dout;
-
        /* Here we go */
        sum = EC_COMMAND_PROTOCOL_3;
        cros_ec_lpc_write_bytes(EC_LPC_ADDR_HOST_CMD, 1, &sum);