diff mbox

spi: orion: remove redundant assignment of status to zero

Message ID 20171102121625.20387-1-colin.king@canonical.com (mailing list archive)
State Accepted
Commit f747c3104efd1e21528d368910afd978fbcd6a78
Headers show

Commit Message

Colin King Nov. 2, 2017, 12:16 p.m. UTC 
From: Colin Ian King <colin.king@canonical.com>

The assignment of status to zero is never read, status is either
updated in the next iteration of the of the loop or several
lines after the end of the loop.  Remove it, cleans up clang warning:

drivers/spi/spi-orion.c:674:4: warning: Value stored to 'status'
is never read

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 drivers/spi/spi-orion.c | 1 -
 1 file changed, 1 deletion(-)
diff mbox

Patch

diff --git a/drivers/spi/spi-orion.c b/drivers/spi/spi-orion.c
index 4b6dd73b80da..8974bb340b3a 100644
--- a/drivers/spi/spi-orion.c
+++ b/drivers/spi/spi-orion.c
@@ -671,7 +671,6 @@  static int orion_spi_probe(struct platform_device *pdev)
 			dev_err(&pdev->dev,
 				"%pOF has no valid 'reg' property (%d)\n",
 				np, status);
-			status = 0;
 			continue;
 		}