Re: [PATCH] net: phy: mdio-bcm-unimac: remove redundant !timeout check

From: Florian Fainelli
Date: Wed Mar 27 2019 - 12:21:37 EST


On 3/27/19 9:15 AM, Colin King wrote:
> From: Colin Ian King <colin.king@xxxxxxxxxxxxx>
>
> 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@xxxxxxxxxxxxx>

Reviewed-by: Florian Fainelli <f.fainelli@xxxxxxxxx>
--
Florian