ata: ahci-imx: remove redundant assignment to ret
authorColin Ian King <colin.king@canonical.com>
Sun, 5 Apr 2020 11:51:20 +0000 (12:51 +0100)
committerJens Axboe <axboe@kernel.dk>
Tue, 7 Apr 2020 20:49:45 +0000 (14:49 -0600)
The variable ret is being initialized with a value that is never read
and it is being updated later with a new value.  The initialization is
redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/ata/ahci_imx.c

index 948d2c6..388baf5 100644 (file)
@@ -782,7 +782,7 @@ static int ahci_imx_softreset(struct ata_link *link, unsigned int *class,
        struct ata_host *host = dev_get_drvdata(ap->dev);
        struct ahci_host_priv *hpriv = host->private_data;
        struct imx_ahci_priv *imxpriv = hpriv->plat_data;
-       int ret = -EIO;
+       int ret;
 
        if (imxpriv->type == AHCI_IMX53)
                ret = ahci_pmp_retry_srst_ops.softreset(link, class, deadline);