power: supply: isp1704: remove redundant assignment to variable ret
authorColin Ian King <colin.king@canonical.com>
Fri, 5 Jul 2019 09:36:12 +0000 (10:36 +0100)
committerSebastian Reichel <sebastian.reichel@collabora.com>
Mon, 2 Sep 2019 19:12:45 +0000 (21:12 +0200)
The variable ret is being assigned with a value that is never
read and it is being updated later with a new value. The
assignment is redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
drivers/power/supply/isp1704_charger.c

index b48cb7a..4812ac1 100644 (file)
@@ -342,7 +342,7 @@ static inline int isp1704_test_ulpi(struct isp1704_charger *isp)
        int vendor;
        int product;
        int i;
-       int ret = -ENODEV;
+       int ret;
 
        /* Test ULPI interface */
        ret = isp1704_write(isp, ULPI_SCRATCH, 0xaa);