From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff Kirsher Subject: [PATCH ] ethtool: Remove duplex info from CTRL register dump Date: Wed, 11 Jan 2006 23:16:42 -0800 Message-ID: <20060112071642.29428.23880.stgit@Gecko.tarbal.com> Cc: John Ronciak , Jesse Brandeburg Return-path: To: Jeff Garzik , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org The duplex control register is used for setting the driver and is not necessary for debug purposes. The value of the duplex control register is what the register's current value is and may not reflect the correct status of te current connection. That is what the duplex status register is used for. To keep from confusing the user, we are removing the duplex register from the ethtool dump of the registers. Signed-off-by: Jeff Kirsher Signed-off-by: Jesse Brandeburg Signed-off-by: John Ronciak --- e1000.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/e1000.c b/e1000.c index 77d799f..6de27ca 100644 --- a/e1000.c +++ b/e1000.c @@ -282,7 +282,6 @@ e1000_dump_regs(struct ethtool_drvinfo * reg = regs_buff[0]; fprintf(stdout, "0x00000: CTRL (Device control register) 0x%08X\n" - " Duplex: %s\n" " Endian mode (buffers): %s\n" " Link reset: %s\n" " Set link up: %s\n" @@ -291,7 +290,6 @@ e1000_dump_regs(struct ethtool_drvinfo * " Transmit flow control: %s\n" " VLAN mode: %s\n", reg, - reg & E1000_CTRL_FD ? "full" : "half", reg & E1000_CTRL_BEM ? "big" : "little", reg & E1000_CTRL_LRST ? "reset" : "normal", reg & E1000_CTRL_SLU ? "1" : "0",