Subject: Re: [PATCH 17/21] e1000: add ich8lan core functions

From: Auke Kok <[email-masked]>
Date: Tue, 27 Jun 2006 09:12:09 -0700

Jeff Garzik wrote:
> Kok, Auke wrote:
>> This implements the core new functions needed for ich8's internal
>> NIC. This includes:
>>
>> * ich8 specific read/write code
>> * flash/nvm access code
>> * software semaphore flag functions
>> * 10/100 PHY (fe - no gigabit speed) support for low-end versions
>> * A workaround for a powerdown sequence problem discovered that
>> affects a small number of motherboard.
>>
>> Signed-off-by: Jesse Brandeburg <[email-masked]>
>> Signed-off-by: Auke Kok <[email-masked]>
>> ---
>>
>>  drivers/net/e1000/e1000_hw.c    | 1000 
>> +++++++++++++++++++++++++++++++++++++++
>>  drivers/net/e1000/e1000_hw.h    |  386 +++++++++++++++
>>  drivers/net/e1000/e1000_osdep.h |   13 +
>>  3 files changed, 1392 insertions(+), 7 deletions(-)
> 
> If it takes this much code to support ICH8, it seems like a e1000-ich8.c 
> would be warranted...

that's work in progress - Jeb Cramer has been working on this for a while now 
but unfortunately it's not ready, and getting ich8 supported in a way that we 
know that doesn't introduce new bugs is more important. This patch adds tested 
and validated support for these chipsets that has been hammered by our test team.

We are planning (working) on cleaning it all up (including whitespace!) - but 
getting the ich8 support out is more important - people can buy the hardware 
today.

Cheers,

Auke