diff mbox series

phy: broadcom: don't duplicate genphy_startup

Message ID 20230328211247.1987820-1-rasmus.villemoes@prevas.dk
State Accepted
Delegated to: Ramon Fried
Headers show
Series phy: broadcom: don't duplicate genphy_startup | expand

Commit Message

Rasmus Villemoes March 28, 2023, 9:12 p.m. UTC 
Functionally, bcm_cygnus_startup() is an exact copy of
genphy_startup().

Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>
---
 drivers/net/phy/broadcom.c | 14 +-------------
 1 file changed, 1 insertion(+), 13 deletions(-)

Comments

Tom Rini May 6, 2023, 2:53 p.m. UTC | #1 
On Tue, Mar 28, 2023 at 11:12:47PM +0200, Rasmus Villemoes wrote:

> Functionally, bcm_cygnus_startup() is an exact copy of
> genphy_startup().
> 
> Signed-off-by: Rasmus Villemoes <rasmus.villemoes@prevas.dk>

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/drivers/net/phy/broadcom.c b/drivers/net/phy/broadcom.c
index 566fcb8de7..5499ff12df 100644
--- a/drivers/net/phy/broadcom.c
+++ b/drivers/net/phy/broadcom.c
@@ -162,18 +162,6 @@  static int bcm5482_config(struct phy_device *phydev)
 	return 0;
 }
 
-static int bcm_cygnus_startup(struct phy_device *phydev)
-{
-	int ret;
-
-	/* Read the Status (2x to make sure link is right) */
-	ret = genphy_update_link(phydev);
-	if (ret)
-		return ret;
-
-	return genphy_parse_link(phydev);
-}
-
 static void bcm_cygnus_afe(struct phy_device *phydev)
 {
 	/* ensures smdspclk is enabled */
@@ -359,7 +347,7 @@  static struct phy_driver BCM_CYGNUS_driver = {
 	.mask = 0xfffff0,
 	.features = PHY_GBIT_FEATURES,
 	.config = &bcm_cygnus_config,
-	.startup = &bcm_cygnus_startup,
+	.startup = &genphy_startup,
 	.shutdown = &genphy_shutdown,
 };