i3c: master: svc: remove redundant assignment to cmd->read_len
authorColin Ian King <colin.king@canonical.com>
Wed, 24 Feb 2021 15:13:49 +0000 (15:13 +0000)
committerAlexandre Belloni <alexandre.belloni@bootlin.com>
Tue, 9 Mar 2021 20:13:31 +0000 (21:13 +0100)
The assignment of xfer_len to cmd->read_len appears to be redundant
as the next statement re-assigns the value 0 to it.  Clean up the
code by removing the redundant first assignment.

Addresses-Coverity: ("Unused value")
Fixes: dd3c52846d59 ("i3c: master: svc: Add Silvaco I3C master driver")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
Link: https://lore.kernel.org/r/20210224151349.202332-1-colin.king@canonical.com
drivers/i3c/master/svc-i3c-master.c

index 8d990696676ef918e17804548e45fb2c0736595c..1f6ba4221817306e96cc2a3f6737f1c4dc2c7704 100644 (file)
@@ -1124,7 +1124,6 @@ static int svc_i3c_master_send_direct_ccc_cmd(struct svc_i3c_master *master,
        cmd->in = NULL;
        cmd->out = &ccc->id;
        cmd->len = 1;
-       cmd->read_len = xfer_len;
        cmd->read_len = 0;
        cmd->continued = true;