From: Stephen Hemminger <shemminger@osdl.org>
To: Auke Kok <auke-jan.h.kok@intel.com>
Cc: Andrew Morton <akpm@osdl.org>, Jeff Garzik <jgarzik@pobox.com>,
NetDev <netdev@vger.kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Arjan van de Ven <arjan@linux.intel.com>
Subject: Re: [PATCH -MM] e1000: rewrite hardware initialization code
Date: Tue, 9 Jan 2007 10:33:09 -0800 [thread overview]
Message-ID: <20070109103309.0b872a53@freekitty> (raw)
In-Reply-To: <45A3D29D.1000202@intel.com>
On Tue, 09 Jan 2007 09:36:29 -0800
Auke Kok <auke-jan.h.kok@intel.com> wrote:
>
> Andrew, All,
>
> This patch contains a major rewrite to the e1000 driver that groups and separates e1000
> hardware by chipset family. It abstracts the hardware specific code into an API that
> will allow us to continue to maintain the complex e1000 driver and add new hardware
> support to it without touching code that affects older chipsets.
Thats good. but:
> drivers/net/e1000/Makefile | 19
> drivers/net/e1000/e1000.h | 95
> drivers/net/e1000/e1000_80003es2lan.c | 1330 +++++
> drivers/net/e1000/e1000_80003es2lan.h | 89
> drivers/net/e1000/e1000_82540.c | 586 ++
> drivers/net/e1000/e1000_82541.c | 1164 ++++
> drivers/net/e1000/e1000_82541.h | 86
> drivers/net/e1000/e1000_82542.c | 466 ++
> drivers/net/e1000/e1000_82543.c | 1397 +++++
> drivers/net/e1000/e1000_82543.h | 45
> drivers/net/e1000/e1000_82571.c | 1132 ++++
> drivers/net/e1000/e1000_82571.h | 42
> drivers/net/e1000/e1000_api.c | 1077 ++++
> drivers/net/e1000/e1000_api.h | 159 +
> drivers/net/e1000/e1000_defines.h | 1289 +++++
> drivers/net/e1000/e1000_ethtool.c | 470 +-
> drivers/net/e1000/e1000_hw.c | 9038 ---------------------------------
> drivers/net/e1000/e1000_hw.h | 3859 ++------------
> drivers/net/e1000/e1000_ich8lan.c | 2353 +++++++++
> drivers/net/e1000/e1000_ich8lan.h | 108
> drivers/net/e1000/e1000_mac.c | 1921 +++++++
> drivers/net/e1000/e1000_mac.h | 84
> drivers/net/e1000/e1000_main.c | 1002 ++--
> drivers/net/e1000/e1000_manage.c | 387 +
> drivers/net/e1000/e1000_manage.h | 83
> drivers/net/e1000/e1000_nvm.c | 860 +++
> drivers/net/e1000/e1000_nvm.h | 61
> drivers/net/e1000/e1000_osdep.h | 56
> drivers/net/e1000/e1000_param.c | 115
> drivers/net/e1000/e1000_phy.c | 1932 +++++++
> drivers/net/e1000/e1000_phy.h | 157 +
> drivers/net/e1000/e1000_regs.h | 236 +
> 32 files changed, 18538 insertions(+), 13160 deletions(-)
Is lots of little files really progress?
--
Stephen Hemminger <shemminger@osdl.org>
next prev parent reply other threads:[~2007-01-09 18:37 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-01-09 17:36 [PATCH -MM] e1000: rewrite hardware initialization code Auke Kok
2007-01-09 18:33 ` Stephen Hemminger [this message]
2007-01-09 18:48 ` Auke Kok
2007-01-09 18:45 ` Andrew Morton
2007-01-09 19:10 ` Auke Kok
2007-01-09 22:24 ` Auke Kok
2007-01-09 19:16 ` Krzysztof Halasa
2007-01-09 19:24 ` Randy Dunlap
2007-01-09 19:45 ` Auke Kok
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20070109103309.0b872a53@freekitty \
--to=shemminger@osdl.org \
--cc=akpm@osdl.org \
--cc=arjan@linux.intel.com \
--cc=auke-jan.h.kok@intel.com \
--cc=jgarzik@pobox.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).