On Wednesday 14 June 2006 11:33 pm, Greg KH wrote: > + snprintf(ep_dev->dev.bus_id, BUS_ID_SIZE, "usbdev%d.%d_ep%02x", > + udev->bus->busnum, udev->devnum, > + endpoint->desc.bEndpointAddress); > + > + retval = device_register(&ep_dev->dev); > + if (retval) > + goto error; > + sysfs_create_group(&ep_dev->dev.kobj, &ep_dev_attr_grp); > + > + endpoint->ep_dev = ep_dev; > + > + /* create the symlink to the old-style "ep_XX" directory */ > + sprintf(name, "ep_%02x", endpoint->desc.bEndpointAddress); > + sysfs_create_link(&parent->kobj, &endpoint->ep_dev->dev.kobj, name);
Hmm, would anyone object if this changed to use the epNin/epNout convention instead of ep8N/ep0N? (Better yet, using decimal not hex ...) That's the convention used everywhere else that we print an endpoint descriptions, since it less obfuscatory. :) If no objections, and nobody beats me to it, I'll submit a patch to do this after these other updates get into kernel.org GIT; right now the pending USB patches are too different from what I'm running to let me generate such a patch! - Dave _______________________________________________ linux-usb-devel@lists.sourceforge.net To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
