On 01.10.2019 08:08, Oleksij Rempel wrote: > struct at803x_priv is never used in this driver. So remove it > and the probe function allocating it. > > Suggested-by: Heiner Kallweit <hkallwe...@gmail.com> > Signed-off-by: Oleksij Rempel <o.rem...@pengutronix.de> > --- > drivers/net/phy/at803x.c | 21 --------------------- > 1 file changed, 21 deletions(-) > > diff --git a/drivers/net/phy/at803x.c b/drivers/net/phy/at803x.c > index 42492f83c8d7..e64f77e152f4 100644 > --- a/drivers/net/phy/at803x.c > +++ b/drivers/net/phy/at803x.c > @@ -63,10 +63,6 @@ MODULE_DESCRIPTION("Atheros 803x PHY driver"); > MODULE_AUTHOR("Matus Ujhelyi"); > MODULE_LICENSE("GPL"); > > -struct at803x_priv { > - bool phy_reset:1; > -}; > - > struct at803x_context { > u16 bmcr; > u16 advertise; > @@ -232,20 +228,6 @@ static int at803x_resume(struct phy_device *phydev) > return phy_modify(phydev, MII_BMCR, BMCR_PDOWN | BMCR_ISOLATE, 0); > } > > -static int at803x_probe(struct phy_device *phydev) > -{ > - struct device *dev = &phydev->mdio.dev; > - struct at803x_priv *priv; > - > - priv = devm_kzalloc(dev, sizeof(*priv), GFP_KERNEL); > - if (!priv) > - return -ENOMEM; > - > - phydev->priv = priv; > - > - return 0; > -} > - > static int at803x_config_init(struct phy_device *phydev) > { > int ret; > @@ -367,7 +349,6 @@ static struct phy_driver at803x_driver[] = { > /* ATHEROS 8035 */ > PHY_ID_MATCH_EXACT(ATH8035_PHY_ID), > .name = "Atheros 8035 ethernet", > - .probe = at803x_probe, > .config_init = at803x_config_init, > .set_wol = at803x_set_wol, > .get_wol = at803x_get_wol, > @@ -380,7 +361,6 @@ static struct phy_driver at803x_driver[] = { > /* ATHEROS 8030 */ > PHY_ID_MATCH_EXACT(ATH8030_PHY_ID), > .name = "Atheros 8030 ethernet", > - .probe = at803x_probe, > .config_init = at803x_config_init, > .link_change_notify = at803x_link_change_notify, > .set_wol = at803x_set_wol, > @@ -394,7 +374,6 @@ static struct phy_driver at803x_driver[] = { > /* ATHEROS 8031 */ > PHY_ID_MATCH_EXACT(ATH8031_PHY_ID), > .name = "Atheros 8031 ethernet", > - .probe = at803x_probe, > .config_init = at803x_config_init, > .set_wol = at803x_set_wol, > .get_wol = at803x_get_wol, >
Reviewed-by: Heiner Kallweit <hkallwe...@gmail.com>