[PATCH] Enhanced ethtool API of CPSW driver in order to be able to address even more than one PHY.

Baruch Siach baruch at tkos.co.il
Thu Apr 21 13:11:56 EDT 2011


Hi Sven,

On Thu, Apr 21, 2011 at 06:17:18PM +0200, Sven Fischer wrote:
> Linked ethtool ioctls ETHTOOL_GSET and ETHTOOL_SSET to the CPSW switch driver and added a new ioctl ETHTOOL_GPHYLINK.
> 
> The new ETHTOOL_GPHYLINK, which gets the PHY link status of a specific PHY, makes ETHTOOL_GLINK obsolete, because it was not working PHY specific.
> 
> ETHTOOL_GSET and ETHTOOL_SSET ioctls are now able to access a specific PHY as well.

A few comments:

1. Please wrap the patch description lines at 80 characters

2. You should Cc the networking devices mailing list at netdev at vger.kernel.org 
   on this patch

3. Please separate changes to generic code (like ethtool, phy) from changes to 
   a specific driver (like cpsw).

By the way, this cpsw driver does not appear to exist in the current mainline 
kernel. When sending a patch against a development tree, please specify which 
one.

baruch

> ---
>  drivers/net/cpsw.c      |   53 +++++++++++++++++++++++++++++++++++++++++++++++
>  drivers/net/phy/phy.c   |    9 ++++++++
>  include/linux/ethtool.h |   12 +++++++++-
>  include/linux/phy.h     |    1 +
>  net/core/ethtool.c      |   28 ++++++++++++++++++++++++-
>  5 files changed, 101 insertions(+), 2 deletions(-)

-- 
                                                     ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -



More information about the linux-arm-kernel mailing list