Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966058AbWKTQWo (ORCPT ); Mon, 20 Nov 2006 11:22:44 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S966127AbWKTQWo (ORCPT ); Mon, 20 Nov 2006 11:22:44 -0500 Received: from mx1.redhat.com ([66.187.233.31]:27304 "EHLO mx1.redhat.com") by vger.kernel.org with ESMTP id S966058AbWKTQWm (ORCPT ); Mon, 20 Nov 2006 11:22:42 -0500 Message-ID: <4561D59F.5020206@redhat.com> Date: Mon, 20 Nov 2006 11:19:43 -0500 From: Chris Snook User-Agent: Thunderbird 1.5.0.7 (X11/20060911) MIME-Version: 1.0 To: Randy Dunlap CC: Jay Cliburn , jeff@garzik.org, shemminger@osdl.org, romieu@fr.zoreil.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/4] atl1: Build files for Attansic L1 driver References: <20061119202915.GB29736@osprey.hogchain.net> <20061119152432.a85d4166.randy.dunlap@oracle.com> <456145DA.804@redhat.com> <4561CCA6.8080209@oracle.com> In-Reply-To: <4561CCA6.8080209@oracle.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2674 Lines: 72 Randy Dunlap wrote: > Chris Snook wrote: >> Randy Dunlap wrote: >>> On Sun, 19 Nov 2006 14:29:15 -0600 Jay Cliburn wrote: >>> >>>> From: Jay Cliburn >>>> >>>> This patch contains the build files for the Attansic L1 gigabit >>>> ethernet >>>> adapter driver. >>>> >>>> Signed-off-by: Jay Cliburn >>>> --- >>>> >>>> Kconfig | 12 ++++++++++++ >>>> Makefile | 1 + >>>> atl1/Makefile | 30 ++++++++++++++++++++++++++++++ >>>> 3 files changed, 43 insertions(+) >>>> >>>> diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig >>>> index 6e863aa..f503d10 100644 >>>> --- a/drivers/net/Kconfig >>>> +++ b/drivers/net/Kconfig >>>> @@ -2329,6 +2329,18 @@ config QLA3XXX >>>> To compile this driver as a module, choose M here: the module >>>> will be called qla3xxx. >>>> >>>> +config ATL1 >>>> + tristate "Attansic(R) L1 Gigabit Ethernet support (EXPERIMENTAL)" >>>> + depends on NET_PCI && PCI && EXPERIMENTAL >>>> + select CRC32 >>>> + select MII >>>> + ---help--- >>>> + This driver supports Attansic L1 gigabit ethernet adapter. >>>> + >>>> + To compile this driver as a module, choose M here. The module >>>> + will be called atl1. >>>> + >>>> + >>>> endmenu >>> >>> One problem here is that MII depends on NET_ETHERNET, which is >>> 10/100 ethernet, which may not be enabled if someone has only >>> gigabit ethernet. :) >> >> I'm actually quite inclined to rip out all MII support entirely. >> There's a lot of code in this driver that needs cleaning up >> cosmetically, and removing deprecated features would certainly speed >> things up. What do you think? > > All of that sounds like a good idea to me. > I didn't realize that MII support is deprecated. Do you mean > that it's deprecated for gigabit? > To be precise, mii-tool is deprecated, in favor of ethtool. There are other things that use mii capabilities. Bonding uses miimon, but the arp monitor is an alternative, and I don't think there's much overlap between workloads where people care about bonding and workloads people run on an Asus M2V motherboard, so I'm not sure there's much justification to bother supporting the feature, except completeness. Thus my question to netdev: Would anyone care if we removed MII support from an experimental driver for a chip that is only known to ship on non-OEM desktop boards? -- Chris - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/