RE: PATCH net-drivers-2.6 1/8] e1000: Support for 82571 and 82572 controllers

2005-08-26 Thread Venkatesan, Ganesh
: Jeff Garzik [mailto:[EMAIL PROTECTED] >Sent: Thursday, August 25, 2005 9:52 PM >To: Venkatesan, Ganesh >Cc: Chilakala, Mallikarjuna; Ronciak, John; Brandeburg, Jesse; netdev >Subject: Re: PATCH net-drivers-2.6 1/8] e1000: Support for 82571 and 82572 >controllers > >Venkat

Re: PATCH net-drivers-2.6 1/8] e1000: Support for 82571 and 82572 controllers

2005-08-25 Thread Jeff Garzik
Venkatesan, Ganesh wrote: Jeff: E1000_osdep.h has a macro msec_delay that does exactly what you've mentioned below (check for the correct context and call msleep). Did you mean that you did not want drivers to have their own msec_delay() macros? I meant: have a human check the context. In thi

RE: PATCH net-drivers-2.6 1/8] e1000: Support for 82571 and 82572 controllers

2005-08-25 Thread Venkatesan, Ganesh
rjuna >Sent: Thursday, August 25, 2005 1:33 PM >To: Venkatesan, Ganesh; Ronciak, John; Brandeburg, Jesse >Subject: FW: PATCH net-drivers-2.6 1/8] e1000: Support for 82571 and 82572 >controllers > > > >-Original Message- >From: Jeff Garzik [mailto:[EMAIL PROTECTED]

Re: PATCH net-drivers-2.6 1/8] e1000: Support for 82571 and 82572 controllers

2005-08-25 Thread Jeff Garzik
Malli Chilakala wrote: -/* Simply wait for 10ms */ -msec_delay(10); +switch (hw->mac_type) { +default: +msec_delay(10); +break; +case e1000_82571: +case e1000_82572: +while (timeout) { +if (E1000_READ_REG(hw, EEMNGCTL) & cfg_mask) +

PATCH net-drivers-2.6 1/8] e1000: Support for 82571 and 82572 controllers

2005-08-25 Thread Malli Chilakala
Support for 82571 and 82572 controllers Signed-off-by: Mallikarjuna R Chilakala <[EMAIL PROTECTED]> Signed-off-by: Ganesh Venkatesan <[EMAIL PROTECTED]> Signed-off-by: John Ronciak <[EMAIL PROTECTED]> diff -up netdev-2.6/drivers/net/e1000/e1000_ethtool.c netdev-2.6.new/drivers/net/e1000/e1000_et