Re: [PATCH 0/3] Remove usb_host class

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 29, 2009 at 12:16:53PM -0400, Alan Stern wrote:
> On Wed, 29 Apr 2009, Greg KH wrote:
> 
> > On Wed, Apr 29, 2009 at 12:07:37PM +0200, Kay Sievers wrote:
> > > On Wed, Apr 29, 2009 at 05:23, Greg KH <greg@xxxxxxxxx> wrote:
> > > > On Tue, Apr 28, 2009 at 08:11:32AM -0700, Greg KH wrote:
> > > >> On Tue, Apr 28, 2009 at 02:50:07PM +0200, Kay Sievers wrote:
> > > >> > What about moving the endpoints to the "usb" bus instead of its own
> > > >> > weird class? I don't really know why there is an enpoint class at all,
> > > >> > and what makes them different from usb-devices or usb-interfaces? The
> > > >> > endpoints are part of the core just like the interfaces are, right?
> > > >> >
> > > >> > Can't we just have them as:
> > > >> >   /devices/pci.../usb4/4-0:1.0/4-0:1.0_ep81
> > > >> > instead of:
> > > >> >   /devices/pci.../usb4/4-0:1.0/usb_endpoint/usbdev4.1_ep81
> > > >> >
> > > >> > The usbdevX.Y* name is not used anywhere else.
> > > >>
> > > >> Yeah, I'll go change that as well.  I have a 4 hour phone meeting today,
> > > >> so I should have plenty of time for things like this :)
> > > >
> > > > Hm, in looking at these, we want that information in sysfs (endpoint
> > > > descriptor info), and want the device nodes someday.
> > > >
> > > > So I'll just leave them for now.
> > > 
> > > Sure we want, but we want them as:
> > >   /sys/bus/usb/devices/4-0:1.0_ep81/
> > > I think. And not as a separate class with these weird usbdev* device names?
> > 
> > As a "real" usb device?  Hm, ok, I can do that, seems a bit odd, but if
> > you are happy with it... :)
> 
> Is that really what Kay wanted?  It's not quite the same as what he
> wrote earlier.
> 
> I don't see any point in putting endpoints on the usb bus.  They might 
> as well remain "busless", if at all possible.  So we would have
> 
> 	/sys/bus/usb/devices/4-0:1.0/4-0:1.0_ep81/
> 
> or even just
> 
> 	/sys/bus/usb/devices/4-0:1.0/ep81/

Heh, that's what they used to be :)

But then we wanted a /dev node for them, so they had to belong to some
bus/class in order for them to be able to notify udev that a device node
needs to be generated.

Kay, any thoughts?

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

[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux