diff options
| author |   Dejin Zheng <zhengdejin5@gmail.com> | 2020-03-23 23:06:00 +0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2020-03-23 22:00:02 -0700 |
| commit | 704f691a5fb33c461ed9a7b6abe115ae3c323691 (patch) | |
| tree | 13d82e4cd420e6702ada735aeb165a2ff6694b91 /drivers/net/phy/nxp-tja11xx.c | |
| parent | net: phy: smsc: use phy_read_poll_timeout() to simplify the code (diff) | |
| download | wireguard-linux-704f691a5fb33c461ed9a7b6abe115ae3c323691.tar.xz wireguard-linux-704f691a5fb33c461ed9a7b6abe115ae3c323691.zip | |
net: phy: tja11xx: use phy_read_poll_timeout() to simplify the code
use phy_read_poll_timeout() to replace the poll codes for
simplify tja11xx_check() function.
Suggested-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: Dejin Zheng <zhengdejin5@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/phy/nxp-tja11xx.c')
| -rw-r--r-- | drivers/net/phy/nxp-tja11xx.c | 16 |
1 files changed, 3 insertions, 13 deletions
diff --git a/drivers/net/phy/nxp-tja11xx.c b/drivers/net/phy/nxp-tja11xx.c index b705d0bd798b..47caae770ffc 100644 --- a/drivers/net/phy/nxp-tja11xx.c +++ b/drivers/net/phy/nxp-tja11xx.c @@ -72,20 +72,10 @@ static struct tja11xx_phy_stats tja11xx_hw_stats[] = { static int tja11xx_check(struct phy_device *phydev, u8 reg, u16 mask, u16 set) { - int i, ret; - - for (i = 0; i < 200; i++) { - ret = phy_read(phydev, reg); - if (ret < 0) - return ret; - - if ((ret & mask) == set) - return 0; - - usleep_range(100, 150); - } + int val; - return -ETIMEDOUT; + return phy_read_poll_timeout(phydev, reg, val, (val & mask) == set, + 150, 30000, false); } static int phy_modify_check(struct phy_device *phydev, u8 reg, |