From patchwork Fri Aug 27 19:42:17 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kyle Moffett X-Patchwork-Id: 62868 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@ozlabs.org Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 90EC5B710D for ; Sat, 28 Aug 2010 05:43:18 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753000Ab0H0Tm3 (ORCPT ); Fri, 27 Aug 2010 15:42:29 -0400 Received: from wsip-70-167-241-26.dc.dc.cox.net ([70.167.241.26]:59097 "EHLO firewall1" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752143Ab0H0Tm2 (ORCPT ); Fri, 27 Aug 2010 15:42:28 -0400 Received: from alexstrasza.exmeritus.com (firewall2.exmeritus.com [10.13.38.2]) by firewall1 (Postfix) with ESMTP id A42CFAC06E; Fri, 27 Aug 2010 15:42:27 -0400 (EDT) From: Kyle Moffett To: linux-kernel@vger.kernel.org Cc: Kyle Moffett , Stephen Hemminger , netdev@vger.kernel.org, Kyle Moffett Subject: [PATCH 1/2] ethtool.h: Add #defines for unidirectional ethernet duplex Date: Fri, 27 Aug 2010 15:42:17 -0400 Message-Id: <1282938138-17844-2-git-send-email-Kyle.D.Moffett@boeing.com> X-Mailer: git-send-email 1.7.1 In-Reply-To: <1282938138-17844-1-git-send-email-Kyle.D.Moffett@boeing.com> References: <1282938138-17844-1-git-send-email-Kyle.D.Moffett@boeing.com> Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org A large variety of fiber ethernet PHYs and some copper ethernet PHYs support forced "unidirectional link" modes. Some signalling modes are designed for last-mile ethernet plants while others are designed for strict security isolation (fiber with no return-path). In order to configure those kinds of forced modes from userspace, we need to add additional options to the "ethtool" interface. As such "unidirectional" ethernet modes most closely resemble ethernet "duplex", we add two additional DUPLEX_* defines to the ethtool interface: #define DUPLEX_TXONLY 0x02 #define DUPLEX_RXONLY 0x03 Most ethernet PHYs will still need to be configured internally in a mode with autoneg off and duplex full, except for a few model-specific PHY register adjustments. Most PHYs can simply use a regular forced-mode for rx-only configuration, but it may be useful in the ethernet driver to completely disable the TX queues or similar. Signed-off-by: Kyle Moffett --- drivers/net/phy/phy.c | 23 ++++++++++++++++------- include/linux/ethtool.h | 4 +++- 2 files changed, 19 insertions(+), 8 deletions(-) diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c index 64be466..1103a80 100644 --- a/drivers/net/phy/phy.c +++ b/drivers/net/phy/phy.c @@ -44,14 +44,23 @@ */ void phy_print_status(struct phy_device *phydev) { - pr_info("PHY: %s - Link is %s", dev_name(&phydev->dev), - phydev->link ? "Up" : "Down"); - if (phydev->link) - printk(" - %d/%s", phydev->speed, - DUPLEX_FULL == phydev->duplex ? - "Full" : "Half"); - - printk("\n"); + const char *strduplex = "Unknown"; + + /* Not much to show if the link is down */ + if (!phydev->link) { + pr_info("PHY: %s - Link is Down\n", dev_name(&phydev->dev)); + return; + } + + /* Otherwise we need to print out speed and duplex */ + switch (phydev->duplex) { + case DUPLEX_HALF: strduplex = "Half"; break; + case DUPLEX_FULL: strduplex = "Full"; break; + case DUPLEX_TXONLY: strduplex = "TX-Only"; break; + case DUPLEX_RXONLY: strduplex = "RX-Only"; break; + } + pr_info("PHY: %s - Link is Up - %d/%s\n", dev_name(&phydev->dev), + phydev->speed, strduplex); } EXPORT_SYMBOL(phy_print_status); diff --git a/include/linux/ethtool.h b/include/linux/ethtool.h index b4207ca..ccf2e32 100644 --- a/include/linux/ethtool.h +++ b/include/linux/ethtool.h @@ -703,9 +703,11 @@ struct ethtool_ops { #define SPEED_2500 2500 #define SPEED_10000 10000 -/* Duplex, half or full. */ +/* Duplex: half/full or unidirectional communication */ #define DUPLEX_HALF 0x00 #define DUPLEX_FULL 0x01 +#define DUPLEX_TXONLY 0x02 +#define DUPLEX_RXONLY 0x03 /* Which connector port. */ #define PORT_TP 0x00