Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1740286

Re: [PATCH net-next 0/5] net: dsa: use generic slave phydev

Show all headers | View raw

On 09/26/2017 02:15 PM, Vivien Didelot wrote:
> DSA currently stores a phy_device pointer in each slave private
> structure. This requires to implement our own ethtool ksettings
> accessors and such.
> 
> This patchset removes the private phy_device in favor of the one
> provided in the net_device structure, and thus allows us to use the
> generic phy_ethtool_* functions.

For this series:

Tested-by: Florian Fainelli <f.fainelli@gmail.com>

On bcm_sf2 (7445 and 7278) along with the externally attached BCM53125
switch that needs the special MDIO read/write divert. We properly attach
to the right PHY devices in all cases.

Also tested unbind/bind, working correctly.

Thanks!

> 
> Vivien Didelot (5):
>   net: dsa: return -ENODEV is there is no slave PHY
>   net: dsa: use slave device phydev
>   net: dsa: use phy_ethtool_get_link_ksettings
>   net: dsa: use phy_ethtool_set_link_ksettings
>   net: dsa: use phy_ethtool_nway_reset
> 
>  net/dsa/dsa_priv.h |   1 -
>  net/dsa/slave.c    | 143 +++++++++++++++++++----------------------------------
>  2 files changed, 52 insertions(+), 92 deletions(-)
> 


-- 
Florian

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH net-next 0/5] net: dsa: use generic slave phydev Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-09-26 23:20 +0200
  [PATCH net-next 1/5] net: dsa: return -ENODEV is there is no slave PHY Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-09-26 23:30 +0200
    Re: [PATCH net-next 1/5] net: dsa: return -ENODEV is there is no  slave PHY Andrew Lunn <andrew@lunn.ch> - 2017-09-26 23:50 +0200
    Re: [PATCH net-next 1/5] net: dsa: return -ENODEV is there is no  slave PHY Florian Fainelli <f.fainelli@gmail.com> - 2017-09-27 00:10 +0200
  [PATCH net-next 5/5] net: dsa: use phy_ethtool_nway_reset Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-09-26 23:30 +0200
    Re: [PATCH net-next 5/5] net: dsa: use phy_ethtool_nway_reset Andrew Lunn <andrew@lunn.ch> - 2017-09-27 00:00 +0200
    Re: [PATCH net-next 5/5] net: dsa: use phy_ethtool_nway_reset Florian Fainelli <f.fainelli@gmail.com> - 2017-09-27 00:10 +0200
  [PATCH net-next 3/5] net: dsa: use phy_ethtool_get_link_ksettings Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-09-26 23:30 +0200
    Re: [PATCH net-next 3/5] net: dsa: use phy_ethtool_get_link_ksettings Andrew Lunn <andrew@lunn.ch> - 2017-09-27 00:00 +0200
    Re: [PATCH net-next 3/5] net: dsa: use phy_ethtool_get_link_ksettings Florian Fainelli <f.fainelli@gmail.com> - 2017-09-27 00:10 +0200
  [PATCH net-next 4/5] net: dsa: use phy_ethtool_set_link_ksettings Vivien Didelot <vivien.didelot@savoirfairelinux.com> - 2017-09-26 23:30 +0200
    Re: [PATCH net-next 4/5] net: dsa: use phy_ethtool_set_link_ksettings Andrew Lunn <andrew@lunn.ch> - 2017-09-27 00:00 +0200
    Re: [PATCH net-next 4/5] net: dsa: use phy_ethtool_set_link_ksettings Florian Fainelli <f.fainelli@gmail.com> - 2017-09-27 00:10 +0200
  Re: [PATCH net-next 0/5] net: dsa: use generic slave phydev Florian Fainelli <f.fainelli@gmail.com> - 2017-09-27 02:00 +0200
  Re: [PATCH net-next 0/5] net: dsa: use generic slave phydev David Miller <davem@davemloft.net> - 2017-09-27 05:10 +0200

csiph-web