ip17xx: correct aneg_done return value
authorSergey Ryazanov <ryazanov.s.a@gmail.com>
Tue, 6 Jun 2017 22:25:32 +0000 (01:25 +0300)
committerJohn Crispin <john@phrozen.org>
Wed, 7 Jun 2017 09:06:29 +0000 (11:06 +0200)
PHY core treats any positive return value as the auto-negotiation done
indication. Since we do not actually check any device register in this
callback then update it to return positive value with a neutral meaning
instead of the register flag to avoid  confusing for future readers.

Signed-off-by: Sergey Ryazanov <ryazanov.s.a@gmail.com>
target/linux/generic/files/drivers/net/phy/ip17xx.c

index 9eedfd4d28c3906e1ccec86a0513c8c075648809..85a9617a5dad5922a59ebe16ace22b579028d7e6 100644 (file)
@@ -1334,7 +1334,7 @@ static int ip17xx_config_aneg(struct phy_device *pdev)
 
 static int ip17xx_aneg_done(struct phy_device *pdev)
 {
-       return BMSR_ANEGCOMPLETE;
+       return 1;       /* Return any positive value */
 }
 
 static int ip17xx_update_link(struct phy_device *pdev)