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: Fri, 13 May 2011 08:31:01 +0200 From: Giuseppe CAVALLARO <peppe.cavallaro@...com> To: David Decotigny <decot@...gle.com> Cc: Giuseppe CAVALLARO <peppe.cavallaro@...com>, "David S. Miller" <davem@...emloft.net>, Joe Perches <joe@...ches.com>, Stanislaw Gruszka <sgruszka@...hat.com>, netdev@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH 1/2] net/stmmac: don't go through ethtool to start autonegociation On 5/10/2011 3:47 PM, Giuseppe CAVALLARO wrote: > On 5/10/2011 2:19 AM, David Decotigny wrote: >> The driver used to call phy's ethtool configuration routine to start >> autonegociation. This change has it call directly phy's routine to >> start autonegociation. >> >> IMPORTANT: initial version was hiding phy_start_aneg() return value, >> this patch returns it (<0 upon error). >> >> Tested: module compiles, NOT tested on real hardware. >> Signed-off-by: David Decotigny <decot@...gle.com> > > Sorry for the delay, I'm doing some tests with the stmmac on a "real" > HW. I'll come back asap. Hello. I've tested the patch and, as discussed with David, I have sent it again plus a new fix in the stmmac_set_pauseparam I have done while testing ethtool -A|-a on my ST box. Best Regards, Peppe > > Regards > Peppe > >> --- >> drivers/net/stmmac/stmmac_ethtool.c | 13 ++----------- >> 1 files changed, 2 insertions(+), 11 deletions(-) >> >> diff --git a/drivers/net/stmmac/stmmac_ethtool.c b/drivers/net/stmmac/stmmac_ethtool.c >> index 6f5aaeb..9c05cf0 100644 >> --- a/drivers/net/stmmac/stmmac_ethtool.c >> +++ b/drivers/net/stmmac/stmmac_ethtool.c >> @@ -236,17 +236,8 @@ stmmac_set_pauseparam(struct net_device *netdev, >> priv->flow_ctrl = new_pause; >> >> if (phy->autoneg) { >> - if (netif_running(netdev)) { >> - struct ethtool_cmd cmd = { .cmd = ETHTOOL_SSET }; >> - /* auto-negotiation automatically restarted */ >> - cmd.supported = phy->supported; >> - cmd.advertising = phy->advertising; >> - cmd.autoneg = phy->autoneg; >> - ethtool_cmd_speed_set(&cmd, phy->speed); >> - cmd.duplex = phy->duplex; >> - cmd.phy_address = phy->addr; >> - ret = phy_ethtool_sset(phy, &cmd); >> - } >> + if (netif_running(netdev)) >> + ret = phy_start_aneg(phy); >> } else >> priv->hw->mac->flow_ctrl(priv->ioaddr, phy->duplex, >> priv->flow_ctrl, priv->pause); > > -- > To unsubscribe from this list: send the line "unsubscribe netdev" in > the body of a message to majordomo@...r.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists