[PATCH 2/2] net: phy: air_en8811h: simplify en8811h_probe() trailing return
From: Vitaliy Sochnev
Date: Sun Aug 23 2026 - 07:08:00 EST
en8811h_clk_provider_setup() is now the last statement in the
function, so the intermediate "if (ret) return ret; return 0;" is
redundant.
Signed-off-by: Vitaliy Sochnev <sochnev.v.74@xxxxxxxxx>
---
drivers/net/phy/air_en8811h.c | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/net/phy/air_en8811h.c b/drivers/net/phy/air_en8811h.c
index 0eeb7b9a4e26..8f4498cf0a7c 100644
--- a/drivers/net/phy/air_en8811h.c
+++ b/drivers/net/phy/air_en8811h.c
@@ -1169,11 +1169,7 @@ static int en8811h_probe(struct phy_device *phydev)
priv->phydev = phydev;
/* Co-Clock Output */
- ret = en8811h_clk_provider_setup(&phydev->mdio.dev, &priv->hw);
- if (ret)
- return ret;
-
- return 0;
+ return en8811h_clk_provider_setup(&phydev->mdio.dev, &priv->hw);
}
static int an8811hb_config_serdes_polarity(struct phy_device *phydev)
--
2.55.0