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