]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
net: phy: mdio-bcm-unimac: remove redundant !timeout check
authorColin Ian King <colin.king@canonical.com>
Wed, 27 Mar 2019 16:15:20 +0000 (16:15 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 27 Mar 2019 21:27:30 +0000 (14:27 -0700)
The check for zero timeout is always true at the end of the proceeding
while loop; the only other exit path in the loop is if the unimac MDIO
is not busy.  Remove the redundant zero timeout check and always
return -ETIMEDOUT on this timeout return path.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/phy/mdio-bcm-unimac.c

index 3a592629dc7ee6403f4ccfe6b7da30322851df82..4a28fb29adaa8d5df841e61d3e86ab372fb7b267 100644 (file)
@@ -92,10 +92,7 @@ static int unimac_mdio_poll(void *wait_func_data)
                usleep_range(1000, 2000);
        } while (--timeout);
 
-       if (!timeout)
-               return -ETIMEDOUT;
-
-       return 0;
+       return -ETIMEDOUT;
 }
 
 static int unimac_mdio_read(struct mii_bus *bus, int phy_id, int reg)