rtc: meson: remove redundant assignment to variable retries
authorColin Ian King <colin.king@canonical.com>
Fri, 22 Nov 2019 22:52:10 +0000 (22:52 +0000)
committerAlexandre Belloni <alexandre.belloni@bootlin.com>
Wed, 27 Nov 2019 08:31:13 +0000 (09:31 +0100)
The variable retries is being initialized with a value that is never
read and it is being updated later with a new value in a for-loop.
The initialization is redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Link: https://lore.kernel.org/r/20191122225210.109172-1-colin.king@canonical.com
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
drivers/rtc/rtc-meson.c

index 9bd8478..47ebcf8 100644 (file)
@@ -131,7 +131,7 @@ static u32 meson_rtc_get_data(struct meson_rtc *rtc)
 
 static int meson_rtc_get_bus(struct meson_rtc *rtc)
 {
-       int ret, retries = 3;
+       int ret, retries;
        u32 val;
 
        /* prepare bus for transfers, set all lines low */