Hi David,

When I build latest git (pulled this morning) for my pxa255 platform I get the 
following error

  CC [M]  drivers/usb/gadget/ether.o
/home/icampbell/devel/kernel/2.6-git/drivers/usb/gadget/ether.c: In function 
`eth_bind':
/home/icampbell/devel/kernel/2.6-git/drivers/usb/gadget/ether.c:2510: error: 
`STATUS_BYTECOUNT' undeclared (first use in this function)
/home/icampbell/devel/kernel/2.6-git/drivers/usb/gadget/ether.c:2510: error: 
(Each undeclared identifier is reported only once
/home/icampbell/devel/kernel/2.6-git/drivers/usb/gadget/ether.c:2510: error: 
for each function it appears in.)

I have:
        CONFIG_USB_ETH=m
        # CONFIG_USB_ETH_RNDIS is not set

I'm afraid I have no idea whether this bit of code should be #ifdef'd or
if the definition of STATUS_BYTECOUNT should be pulled outside of its
#ifdef. Based on gut feeling and the other uses of STATUS_BYTECOUNT in
the file I'm going to take a punt on the former, if so here is a
patch ;-)

Signed-off-by: Ian Campbell <[EMAIL PROTECTED]>

Ian.

Index: 2.6-git/drivers/usb/gadget/ether.c
===================================================================
--- 2.6-git.orig/drivers/usb/gadget/ether.c     2005-04-28 11:02:16.000000000 
+0100
+++ 2.6-git/drivers/usb/gadget/ether.c  2005-06-13 13:50:42.000000000 +0100
@@ -2505,6 +2505,7 @@
        dev->req->complete = eth_setup_complete;
 
        /* ... and maybe likewise for status transfer */
+#ifdef DEV_CONFIG_CDC
        if (dev->status_ep) {
                dev->stat_req = eth_req_alloc (dev->status_ep,
                                        STATUS_BYTECOUNT);
@@ -2513,6 +2514,7 @@
                        goto fail;
                }
        }
+#endif
 
        /* finish hookup to lower layer ... */
        dev->gadget = gadget;


-- 
Ian Campbell, Senior Design Engineer
                                        Web: http://www.arcom.com
Arcom, Clifton Road,                    Direct: +44 (0)1223 403 465
Cambridge CB1 7EA, United Kingdom       Phone:  +44 (0)1223 411 200



-------------------------------------------------------
This SF.Net email is sponsored by: NEC IT Guy Games.  How far can you shotput
a projector? How fast can you ride your desk chair down the office luge track?
If you want to score the big prize, get to know the little guy.  
Play to win an NEC 61" plasma display: http://www.necitguy.com/?r=20
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to