ChangeSet 1.1561, 2003/12/29 12:30:55-08:00, [EMAIL PROTECTED]
[PATCH] USB: let USB_{PEGASUS,USBNET} depend on NET_ETHERNET
Adrian Bunk wrote:
> I observed the following small problem in 2.6:
>
> - MII depends on NET_ETHERNET
> - USB_PEGASUS and USB_USBNET select MII, but they depend only on NET
>
> The patch below lets USB_PEGASUS and USB_USBNET depend on NET_ETHERNET
> instead of NET to fix this issue.
Actually how about this one instead? The PEGASUS bit is the same.
The difference is that MII (and CRC32) are only attributed to the
driver code that needs those ... AX8817X needs both, ZAURUS just
needs CRC32. The core (which should eventually become a separate
module) shouldn't depend on those modules at all.
Also both CDCETHER and AX8817X are marked as non-experimental;
I recall Dave Hollis submitted a patch to do that for AX8817X,
and CDCETHER now seems to have gotten enough success reports too.
drivers/usb/net/Kconfig | 13 ++++++-------
1 files changed, 6 insertions(+), 7 deletions(-)
diff -Nru a/drivers/usb/net/Kconfig b/drivers/usb/net/Kconfig
--- a/drivers/usb/net/Kconfig Mon Dec 29 14:20:22 2003
+++ b/drivers/usb/net/Kconfig Mon Dec 29 14:20:22 2003
@@ -69,7 +69,7 @@
config USB_PEGASUS
tristate "USB Pegasus/Pegasus-II based ethernet device support"
- depends on USB && NET
+ depends on USB && NET_ETHERNET
select MII
---help---
Say Y here if you know you have Pegasus or Pegasus-II based adapter.
@@ -96,8 +96,6 @@
config USB_USBNET
tristate "Multi-purpose USB Networking Framework"
depends on USB && NET
- select CRC32
- select MII
---help---
This driver supports several kinds of network links over USB,
with "minidrivers" built around a common network driver core
@@ -206,6 +204,7 @@
config USB_ZAURUS
boolean "Sharp Zaurus (stock ROMs)"
depends on USB_USBNET
+ select CRC32
default y
help
Choose this option to support the usb networking links used by
@@ -217,9 +216,7 @@
config USB_CDCETHER
boolean "CDC Ethernet support (smart devices such as cable modems)"
- # experimental primarily because cdc-ether was.
- # make it non-experimental after more interop testing
- depends on USB_USBNET && EXPERIMENTAL
+ depends on USB_USBNET
default y
help
This option supports devices conforming to the Communication Device
@@ -247,7 +244,9 @@
config USB_AX8817X
boolean "ASIX AX88172 Based USB 2.0 Ethernet Devices"
- depends on USB_USBNET
+ depends on USB_USBNET && NET_ETHERNET
+ select CRC32
+ select MII
default y
help
-------------------------------------------------------
This SF.net email is sponsored by: IBM Linux Tutorials.
Become an expert in LINUX or just sharpen your skills. Sign up for IBM's
Free Linux Tutorials. Learn everything from the bash shell to sys admin.
Click now! http://ads.osdn.com/?ad_id78&alloc_id371&op=click
_______________________________________________
[EMAIL PROTECTED]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
