RE: [PATCH] drivers/net: move the nvidia forcedeth driver from 100M group to 1000M group

2007-04-26 Thread Peer Chen
You are right,both ways would cause confusion,but sooner or later we need to 
move it because our NICs onward are all Gigabit and 100M NICs will disappear 
gradually in the future. Probably H.Peter's suggestion that have a single list 
for 100M and 1000M is a better choice.

-Original Message-
From: Lennart Sorensen [mailto:[EMAIL PROTECTED] 
Sent: Wednesday, April 25, 2007 10:06 PM
To: Peer Chen
Cc: linux-kernel@vger.kernel.org; [EMAIL PROTECTED]; [EMAIL PROTECTED]
Subject: Re: [PATCH] drivers/net: move the nvidia forcedeth driver from 100M 
group to 1000M group

On Wed, Apr 25, 2007 at 01:30:04PM +0800, Peer Chen wrote:
> nForce ehternet is a Gigabit NIC not 100M, move it to 1000M group to 
> avoid the confusion.

The forcedeth on my nforce2 board is 100Mbit.  I think the driver handles both 
100Mbit and GBit type devices.  Makes for an interesting categorization 
problem.  Moving it would cause confusion.  Leaving it where it is could cause 
confusion.  Moving it may confuse existing users more though so I would 
recommend leaving it alone unless you can somehow make it appear in both.

--
Len Sorensen
---
This email message is for the sole use of the intended recipient(s) and may 
contain
confidential information.  Any unauthorized review, use, disclosure or 
distribution
is prohibited.  If you are not the intended recipient, please contact the 
sender by
reply email and destroy all copies of the original message.
---
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: [PATCH] drivers/net: move the nvidia forcedeth driver from 100M group to 1000M group

2007-04-26 Thread Peer Chen
You are right,both ways would cause confusion,but sooner or later we need to 
move it because our NICs onward are all Gigabit and 100M NICs will disappear 
gradually in the future. Probably H.Peter's suggestion that have a single list 
for 100M and 1000M is a better choice.

-Original Message-
From: Lennart Sorensen [mailto:[EMAIL PROTECTED] 
Sent: Wednesday, April 25, 2007 10:06 PM
To: Peer Chen
Cc: linux-kernel@vger.kernel.org; [EMAIL PROTECTED]; [EMAIL PROTECTED]
Subject: Re: [PATCH] drivers/net: move the nvidia forcedeth driver from 100M 
group to 1000M group

On Wed, Apr 25, 2007 at 01:30:04PM +0800, Peer Chen wrote:
 nForce ehternet is a Gigabit NIC not 100M, move it to 1000M group to 
 avoid the confusion.

The forcedeth on my nforce2 board is 100Mbit.  I think the driver handles both 
100Mbit and GBit type devices.  Makes for an interesting categorization 
problem.  Moving it would cause confusion.  Leaving it where it is could cause 
confusion.  Moving it may confuse existing users more though so I would 
recommend leaving it alone unless you can somehow make it appear in both.

--
Len Sorensen
---
This email message is for the sole use of the intended recipient(s) and may 
contain
confidential information.  Any unauthorized review, use, disclosure or 
distribution
is prohibited.  If you are not the intended recipient, please contact the 
sender by
reply email and destroy all copies of the original message.
---
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] drivers/net: move the nvidia forcedeth driver from 100M group to 1000M group

2007-04-25 Thread Lennart Sorensen
On Wed, Apr 25, 2007 at 01:30:04PM +0800, Peer Chen wrote:
> nForce ehternet is a Gigabit NIC not 100M, move it to 1000M group to
> avoid the confusion.

The forcedeth on my nforce2 board is 100Mbit.  I think the driver
handles both 100Mbit and GBit type devices.  Makes for an interesting
categorization problem.  Moving it would cause confusion.  Leaving it
where it is could cause confusion.  Moving it may confuse existing users
more though so I would recommend leaving it alone unless you can somehow
make it appear in both.

--
Len Sorensen
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] drivers/net: move the nvidia forcedeth driver from 100M group to 1000M group

2007-04-25 Thread H. Peter Anvin
Peer Chen wrote:
> nForce ehternet is a Gigabit NIC not 100M, move it to 1000M group to
> avoid the confusion.

Overall, maybe the 100 Mbps vs 1000 Mbps distinction is getting less
useful and it would be better to have a single list.  There are a *lot*
of chipsets on the market now which have the same core and the only
difference is that one has a 100M PHY and the other has a 1000M PHY.
For example, there are 100M versions of sky2 and r8169.

-hpa
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] drivers/net: move the nvidia forcedeth driver from 100M group to 1000M group

2007-04-25 Thread H. Peter Anvin
Peer Chen wrote:
 nForce ehternet is a Gigabit NIC not 100M, move it to 1000M group to
 avoid the confusion.

Overall, maybe the 100 Mbps vs 1000 Mbps distinction is getting less
useful and it would be better to have a single list.  There are a *lot*
of chipsets on the market now which have the same core and the only
difference is that one has a 100M PHY and the other has a 1000M PHY.
For example, there are 100M versions of sky2 and r8169.

-hpa
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH] drivers/net: move the nvidia forcedeth driver from 100M group to 1000M group

2007-04-25 Thread Lennart Sorensen
On Wed, Apr 25, 2007 at 01:30:04PM +0800, Peer Chen wrote:
 nForce ehternet is a Gigabit NIC not 100M, move it to 1000M group to
 avoid the confusion.

The forcedeth on my nforce2 board is 100Mbit.  I think the driver
handles both 100Mbit and GBit type devices.  Makes for an interesting
categorization problem.  Moving it would cause confusion.  Leaving it
where it is could cause confusion.  Moving it may confuse existing users
more though so I would recommend leaving it alone unless you can somehow
make it appear in both.

--
Len Sorensen
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/