On Thu, Apr 01, 2010 at 12:18:58AM +0100, Ben Hutchings wrote: > On Wed, 2010-03-31 at 21:42 +0200, L. Alberto Gim??nez wrote: > [...] > > --- /dev/null > > +++ b/drivers/net/usb/ipheth.c > [...] > > +#include <linux/kernel.h> > > +#include <linux/errno.h> > > +#include <linux/init.h> > > +#include <linux/slab.h> > > +#include <linux/module.h> > > +#include <linux/netdevice.h> > > +#include <linux/etherdevice.h> > > +#include <linux/ethtool.h> > > +#include <linux/uaccess.h> > > I don't think you need this header. > > > +#include <linux/usb.h> > > +#include <linux/workqueue.h> > > + > > +#define USB_VENDOR_APPLE 0x05ac > > +#define USB_PRODUCT_IPHETH 0x1290 > > +#define USB_PRODUCT_IPHETH_3G 0x1292 > > +#define USB_PRODUCT_IPHETH_3GS 0x1294 > > Apple doesn't assign device ids to ipheth so either the names are > incorrect or you should get proper device ids. I believe the Linux > Foundation has a USB vendor id and could assign device ids under that. You mean there is an application on the iphone that sets the device id here? I thought this was the iphone device ids already assigned to apple. If we need a linux-specific device id, I can assign them from the Linux foundations pool of device ids, but so far I've only done that for devices that either run Linux (usb gadgets), or for the usb root hubs. But hey, if it's controlable from software, let me know and I'll dole one out. In that case, why would you need 3 different device ids? thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
