* [PATCH net-next] mdio: unused ethtool functions
@ 2013-12-26 18:12 Stephen Hemminger
2013-12-26 18:30 ` David Miller
2013-12-27 9:06 ` Ben Hutchings
0 siblings, 2 replies; 3+ messages in thread
From: Stephen Hemminger @ 2013-12-26 18:12 UTC (permalink / raw)
To: Ben Hutchings, David Miller; +Cc: netdev
Use it or lose it.
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
drivers/net/mdio.c | 28 ----------------------------
include/linux/mdio.h | 3 ---
2 files changed, 31 deletions(-)
--- a/drivers/net/mdio.c 2013-12-24 15:37:35.779661759 -0800
+++ b/drivers/net/mdio.c 2013-12-24 15:38:26.054970175 -0800
@@ -342,34 +342,6 @@ void mdio45_ethtool_gset_npage(const str
EXPORT_SYMBOL(mdio45_ethtool_gset_npage);
/**
- * mdio45_ethtool_spauseparam_an - set auto-negotiated pause parameters
- * @mdio: MDIO interface
- * @ecmd: Ethtool request structure
- *
- * This function assumes that the PHY has an auto-negotiation MMD. It
- * will enable and disable advertising of flow control as appropriate.
- */
-void mdio45_ethtool_spauseparam_an(const struct mdio_if_info *mdio,
- const struct ethtool_pauseparam *ecmd)
-{
- int adv, old_adv;
-
- WARN_ON(!(mdio->mmds & MDIO_DEVS_AN));
-
- old_adv = mdio->mdio_read(mdio->dev, mdio->prtad, MDIO_MMD_AN,
- MDIO_AN_ADVERTISE);
- adv = ((old_adv & ~(ADVERTISE_PAUSE_CAP | ADVERTISE_PAUSE_ASYM)) |
- mii_advertise_flowctrl((ecmd->rx_pause ? FLOW_CTRL_RX : 0) |
- (ecmd->tx_pause ? FLOW_CTRL_TX : 0)));
- if (adv != old_adv) {
- mdio->mdio_write(mdio->dev, mdio->prtad, MDIO_MMD_AN,
- MDIO_AN_ADVERTISE, adv);
- mdio45_nway_restart(mdio);
- }
-}
-EXPORT_SYMBOL(mdio45_ethtool_spauseparam_an);
-
-/**
* mdio_mii_ioctl - MII ioctl interface for MDIO (clause 22 or 45) PHYs
* @mdio: MDIO interface
* @mii_data: MII ioctl data structure
--- a/include/linux/mdio.h 2013-12-24 15:37:35.779661759 -0800
+++ b/include/linux/mdio.h 2013-12-24 15:38:26.054970175 -0800
@@ -70,9 +70,6 @@ extern int mdio45_nway_restart(const str
extern void mdio45_ethtool_gset_npage(const struct mdio_if_info *mdio,
struct ethtool_cmd *ecmd,
u32 npage_adv, u32 npage_lpa);
-extern void
-mdio45_ethtool_spauseparam_an(const struct mdio_if_info *mdio,
- const struct ethtool_pauseparam *ecmd);
/**
* mdio45_ethtool_gset - get settings for ETHTOOL_GSET
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH net-next] mdio: unused ethtool functions
2013-12-26 18:12 [PATCH net-next] mdio: unused ethtool functions Stephen Hemminger
@ 2013-12-26 18:30 ` David Miller
2013-12-27 9:06 ` Ben Hutchings
1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2013-12-26 18:30 UTC (permalink / raw)
To: stephen; +Cc: bhutchings, netdev
From: Stephen Hemminger <stephen@networkplumber.org>
Date: Thu, 26 Dec 2013 10:12:41 -0800
> Use it or lose it.
>
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Applied.
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] mdio: unused ethtool functions
2013-12-26 18:12 [PATCH net-next] mdio: unused ethtool functions Stephen Hemminger
2013-12-26 18:30 ` David Miller
@ 2013-12-27 9:06 ` Ben Hutchings
1 sibling, 0 replies; 3+ messages in thread
From: Ben Hutchings @ 2013-12-27 9:06 UTC (permalink / raw)
To: Stephen Hemminger; +Cc: David Miller, netdev
On Thu, 2013-12-26 at 10:12 -0800, Stephen Hemminger wrote:
> Use it or lose it.
>
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Right, this was called by sfc for a while but not since 2.6.33 as we set
all the autoneg bits at once.
Ben.
> ---
> drivers/net/mdio.c | 28 ----------------------------
> include/linux/mdio.h | 3 ---
> 2 files changed, 31 deletions(-)
>
> --- a/drivers/net/mdio.c 2013-12-24 15:37:35.779661759 -0800
> +++ b/drivers/net/mdio.c 2013-12-24 15:38:26.054970175 -0800
> @@ -342,34 +342,6 @@ void mdio45_ethtool_gset_npage(const str
> EXPORT_SYMBOL(mdio45_ethtool_gset_npage);
>
> /**
> - * mdio45_ethtool_spauseparam_an - set auto-negotiated pause parameters
> - * @mdio: MDIO interface
> - * @ecmd: Ethtool request structure
> - *
> - * This function assumes that the PHY has an auto-negotiation MMD. It
> - * will enable and disable advertising of flow control as appropriate.
> - */
> -void mdio45_ethtool_spauseparam_an(const struct mdio_if_info *mdio,
> - const struct ethtool_pauseparam *ecmd)
> -{
> - int adv, old_adv;
> -
> - WARN_ON(!(mdio->mmds & MDIO_DEVS_AN));
> -
> - old_adv = mdio->mdio_read(mdio->dev, mdio->prtad, MDIO_MMD_AN,
> - MDIO_AN_ADVERTISE);
> - adv = ((old_adv & ~(ADVERTISE_PAUSE_CAP | ADVERTISE_PAUSE_ASYM)) |
> - mii_advertise_flowctrl((ecmd->rx_pause ? FLOW_CTRL_RX : 0) |
> - (ecmd->tx_pause ? FLOW_CTRL_TX : 0)));
> - if (adv != old_adv) {
> - mdio->mdio_write(mdio->dev, mdio->prtad, MDIO_MMD_AN,
> - MDIO_AN_ADVERTISE, adv);
> - mdio45_nway_restart(mdio);
> - }
> -}
> -EXPORT_SYMBOL(mdio45_ethtool_spauseparam_an);
> -
> -/**
> * mdio_mii_ioctl - MII ioctl interface for MDIO (clause 22 or 45) PHYs
> * @mdio: MDIO interface
> * @mii_data: MII ioctl data structure
> --- a/include/linux/mdio.h 2013-12-24 15:37:35.779661759 -0800
> +++ b/include/linux/mdio.h 2013-12-24 15:38:26.054970175 -0800
> @@ -70,9 +70,6 @@ extern int mdio45_nway_restart(const str
> extern void mdio45_ethtool_gset_npage(const struct mdio_if_info *mdio,
> struct ethtool_cmd *ecmd,
> u32 npage_adv, u32 npage_lpa);
> -extern void
> -mdio45_ethtool_spauseparam_an(const struct mdio_if_info *mdio,
> - const struct ethtool_pauseparam *ecmd);
>
> /**
> * mdio45_ethtool_gset - get settings for ETHTOOL_GSET
--
Ben Hutchings, Staff Engineer, Solarflare
Not speaking for my employer; that's the marketing department's job.
They asked us to note that Solarflare product names are trademarked.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-12-27 9:06 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-26 18:12 [PATCH net-next] mdio: unused ethtool functions Stephen Hemminger
2013-12-26 18:30 ` David Miller
2013-12-27 9:06 ` Ben Hutchings
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox