On Mon, Jan 31, 2005 at 03:46:07PM -0800, David S. Miller wrote: > On Tue, 1 Feb 2005 00:41:58 +0100 > Adrian Bunk <bunk@xxxxxxxxx> wrote: > > > dp83840.h is included once but none of the definitions it contains is > > actually used. > > > > Ralf Baechle wants that it stays as documentation, so this patch moves > > it under Documentation/ . > > No, let's kill this thing altogether. The only driver in the world > using the CSCONFIG_* defines in there is the sunhme driver and it > defines it's own macros in drivers/net/sunhme.h This header is more > than useless these days. > > The header still exists in older trees and the revision history. > So people can still get to it there. OK, patch below cu Adrian <-- snip --> dp83840.h is included once but none of the definitions it contains is actually used. Signed-off-by: Adrian Bunk <bunk@xxxxxxxxx> --- drivers/net/ioc3-eth.c | 1 include/linux/dp83840.h | 41 --------------------------- 2 files changed, 42 deletions(-) --- linux-2.6.10-rc2-mm3-full/drivers/net/ioc3-eth.c.old 2004-11-29 12:14:25.000000000 +0100 +++ linux-2.6.10-rc2-mm3-full/drivers/net/ioc3-eth.c 2004-11-29 12:14:34.000000000 +0100 @@ -56,7 +56,6 @@ #include <linux/etherdevice.h> #include <linux/ethtool.h> #include <linux/skbuff.h> -#include <linux/dp83840.h> #include <net/ip.h> #include <asm/byteorder.h> --- linux-2.6.10-rc2-mm3-full/include/linux/dp83840.h 2004-10-18 23:54:32.000000000 +0200 +++ /dev/null 2004-11-25 03:16:25.000000000 +0100 @@ -1,41 +0,0 @@ -/* - * linux/dp83840.h: definitions for DP83840 MII-compatible transceivers - * - * Copyright (C) 1996, 1999 David S. Miller (davem@xxxxxxxxxx) - */ -#ifndef __LINUX_DP83840_H -#define __LINUX_DP83840_H - -#include <linux/mii.h> - -/* - * Data sheets and programming docs for the DP83840 are available at - * from http://www.national.com/ - * - * The DP83840 is capable of both 10 and 100Mbps ethernet, in both - * half and full duplex mode. It also supports auto negotiation. - * - * But.... THIS THING IS A PAIN IN THE ASS TO PROGRAM! - * Debugging eeprom burnt code is more fun than programming this chip! - */ - -/* First, the MII register numbers (actually DP83840 register numbers). */ -#define MII_CSCONFIG 0x17 /* CS configuration */ - -/* The Carrier Sense config register. */ -#define CSCONFIG_RESV1 0x0001 /* Unused... */ -#define CSCONFIG_LED4 0x0002 /* Pin for full-dplx LED4 */ -#define CSCONFIG_LED1 0x0004 /* Pin for conn-status LED1 */ -#define CSCONFIG_RESV2 0x0008 /* Unused... */ -#define CSCONFIG_TCVDISAB 0x0010 /* Turns off the transceiver */ -#define CSCONFIG_DFBYPASS 0x0020 /* Bypass disconnect function */ -#define CSCONFIG_GLFORCE 0x0040 /* Good link force for 100mbps */ -#define CSCONFIG_CLKTRISTATE 0x0080 /* Tristate 25m clock */ -#define CSCONFIG_RESV3 0x0700 /* Unused... */ -#define CSCONFIG_ENCODE 0x0800 /* 1=MLT-3, 0=binary */ -#define CSCONFIG_RENABLE 0x1000 /* Repeater mode enable */ -#define CSCONFIG_TCDISABLE 0x2000 /* Disable timeout counter */ -#define CSCONFIG_RESV4 0x4000 /* Unused... */ -#define CSCONFIG_NDISABLE 0x8000 /* Disable NRZI */ - -#endif /* __LINUX_DP83840_H */ - : send the line "unsubscribe linux-net" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html