Re: [patch] Re: Gigabit Kconfig problems with yesterday's update

[Linus Torvalds]

On Sun, 30 May 2004, Adrian Bunk wrote:
> 
> 
> This is not sufficient since it's still under the
> "Ethernet (10 or 100Mbit)" menu.
> 
> What about the patch below?
>
>  config NET_ETHERNET
> -	bool "Ethernet (10 or 100Mbit)"
> +	bool "Ethernet (10/100/1000/10000 Mbit)"

Ok, this part is obviously sane.

However:

> +menu "Ethernet (10/100 Mbit)"
> +	depends on NET_ETHERNET
> +
>  
>  menu "Gigabit Ethernet (1000/10000 Mbit)"
> -	depends on NETDEVICES
> +	depends on NET_ETHERNET

is it really sane these days to split out gigabit from the "regular" 
ethernets? gigabit ethernet is getting quite a bit more common than it 
used to be, and a lot of the gigabit devices are just "standard ethernet" 
as far as the user is concerned, and in fact they are often _used_ in just 
regular 10/100Mbps setups.

In other words: it's quite understandable if somebody doesn't even
_realize_ that his chip supports gigabit speeds these days.

So wouldn't the right fix be to just remove the distinction between 
"regular" and "gigabit" ethernet, and put them all in the same menu?

If the menu then grows really big, maybe we can split it according to some 
_saner_ split. Not that I can see off-hand what that would be, but..

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