Re: [ECOS] USB slave ethernet device driver

[Bart Veer <bartv at ecoscentric dot com>]

>>>>> "wpd" == Doyle, Patrick <WPD at dtccom dot com> writes:

    wpd> Before I go digging through this too much, I thought I would toss out the
    wpd> question...

    wpd> It appears that the USB slave ethernet driver has not been
    wpd> updated to reflect the new network stack. That is, when I
    wpd> attempt to build an ecos (I picked assabet as a platform,
    wpd> just because it sounded good at the time) with the "net"
    wpd> template, I get compile errors. But when I use the "old_net"
    wpd> template, I don't.

    wpd> Before I go charging off fixing the compile errors, I just
    wpd> wanted to check...

    wpd> Is this to be expected?
    wpd> Has anybody else fixed this already?

Try the patch below, which seems to fix the build problem. I have not
actually tried the resulting build. Also note that there are still
some pending patches for the host-side Linux driver.

    wpd> Is it worth fixing?
    wpd> Does anybody have a list of things that change when switching from "old_net"
    wpd> to "net"?  (i.e. "don't include "netstuff.h" anymore -- just include
    wpd> "net.h", or whatever).

The big issue is multicast support. IPv6 requires that, but not all
ethernet drivers implement it.

Bart

Index: io/usb/eth/slave/current/cdl/usbs_eth.cdl
===================================================================
RCS file: /cvs/ecos/ecos/packages/io/usb/eth/slave/current/cdl/usbs_eth.cdl,v
retrieving revision 1.5
diff -u -r1.5 usbs_eth.cdl
--- io/usb/eth/slave/current/cdl/usbs_eth.cdl	24 Feb 2003 14:25:32 -0000	1.5
+++ io/usb/eth/slave/current/cdl/usbs_eth.cdl	17 Apr 2003 17:21:35 -0000
@@ -118,5 +119,35 @@
 	    implements    CYGHWR_NET_DRIVER_ETH1
 	    requires      !CYGHWR_NET_DRIVER_ETH1_BOOTP
 	}
+    }
+    
+    cdl_component CYGPKG_IO_USB_SLAVE_ETH_OPTIONS {
+	display     "Build options"
+	flavor      none
+
+	description "
+	    Package-specific build options including control over compiler
+	    flags used only in building this package."
+
+	cdl_option CYGPKG_IO_USB_SLAVE_ETH_CFLAGS_ADD {
+            display "Additional compiler flags"
+            flavor  data
+            no_define
+            default_value { "-D_KERNEL -D__ECOS" }
+            description   "
+                This option modifies the set of compiler flags for
+                building this package. These flags are used in addition
+                to the set of global flags."
+	}
+        cdl_option CYGPKG_IO_USB_SLAVE_ETH_CFLAGS_REMOVE {
+            display "Suppressed compiler flags"
+            flavor  data
+            no_define
+            default_value { "" }
+            description   "
+                This option modifies the set of compiler flags for
+                building this package. These flags are removed from
+                the set of global flags if present."
+        }
     }
 }

-- 
Bart Veer                       eCos Configuration Architect
http://www.ecoscentric.com/     The eCos and RedBoot experts

-- 
Before posting, please read the FAQ: http://sources.redhat.com/fom/ecos
and search the list archive: http://sources.redhat.com/ml/ecos-discuss