lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 1 Oct 2019 20:40:52 +0200 From: Heiner Kallweit <hkallweit1@...il.com> To: Oleksij Rempel <o.rempel@...gutronix.de>, Andrew Lunn <andrew@...n.ch>, Florian Fainelli <f.fainelli@...il.com> Cc: Pengutronix Kernel Team <kernel@...gutronix.de>, "David S. Miller" <davem@...emloft.net>, netdev@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH v2 3/3] net: phy: at803x: remove probe and struct at803x_priv 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 <hkallweit1@...il.com> > Signed-off-by: Oleksij Rempel <o.rempel@...gutronix.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 <hkallweit1@...il.com>
Powered by blists - more mailing lists