From: Chuanhong Guo Date: Thu, 10 Jan 2019 06:54:56 +0000 (+0800) Subject: generic: ar8216: move ar8xxx_id_chip into ar8xxx_phy_probe X-Git-Tag: v19.07.0-rc1~959 X-Git-Url: http://git.ozo.com/?p=openwrt%2F.git;a=commitdiff_plain;h=da64a8c65630388369fc3ea600f1b09cc5364f13 generic: ar8216: move ar8xxx_id_chip into ar8xxx_phy_probe ar8xxx_id_chip is used to determine current ar8xxx_chip using switch id and this isn't needed during mdiodev probing. Move it out of ar8xxx_probe_switch so that we can skip it. Signed-off-by: Chuanhong Guo --- diff --git a/target/linux/generic/files/drivers/net/phy/ar8216.c b/target/linux/generic/files/drivers/net/phy/ar8216.c index 10bdf52353..b8cc2c8f4c 100644 --- a/target/linux/generic/files/drivers/net/phy/ar8216.c +++ b/target/linux/generic/files/drivers/net/phy/ar8216.c @@ -1975,10 +1975,6 @@ ar8xxx_probe_switch(struct ar8xxx_priv *priv) struct switch_dev *swdev; int ret; - ret = ar8xxx_id_chip(priv); - if (ret) - return ret; - chip = priv->chip; swdev = &priv->dev; @@ -2208,6 +2204,10 @@ ar8xxx_phy_probe(struct phy_device *phydev) priv->mii_bus = phydev->mdio.bus; priv->pdev = &phydev->mdio.dev; + ret = ar8xxx_id_chip(priv); + if (ret) + goto free_priv; + ret = ar8xxx_probe_switch(priv); if (ret) goto free_priv;