diff options
| author | Colin Ian King <colin.king@canonical.com> | 2020-09-10 16:04:10 +0100 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2020-09-14 15:50:04 +0100 |
| commit | 4a6c7d6f940107d6383291e2cb450039790b752d (patch) | |
| tree | 406f425c6ef86a94291c5d0181b7ec2f63189826 /drivers/spi/spi-qup.c | |
| parent | 6c557d24fa2622bca5b950dae791d3f0d18ff2d3 (diff) | |
| download | lwn-4a6c7d6f940107d6383291e2cb450039790b752d.tar.gz lwn-4a6c7d6f940107d6383291e2cb450039790b752d.zip | |
spi: qup: remove redundant assignment to variable ret
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>
Link: https://lore.kernel.org/r/20200910150410.750959-1-colin.king@canonical.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/spi/spi-qup.c')
| -rw-r--r-- | drivers/spi/spi-qup.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/spi/spi-qup.c b/drivers/spi/spi-qup.c index a364b99497e2..8dcb2e70735c 100644 --- a/drivers/spi/spi-qup.c +++ b/drivers/spi/spi-qup.c @@ -848,7 +848,7 @@ static int spi_qup_transfer_one(struct spi_master *master, { struct spi_qup *controller = spi_master_get_devdata(master); unsigned long timeout, flags; - int ret = -EIO; + int ret; ret = spi_qup_io_prep(spi, xfer); if (ret) |