Re: [PATCH] driverfs support for USB - take 2

From: Greg KH (greg@kroah.com)
Date: Tue Jan 29 2002 - 23:09:08 EST


On Tue, Jan 29, 2002 at 06:15:13PM -0800, David Brownell wrote:
> > > > > Yes, I need to have better names for the devices than just "usb_bus",
> > > > > any suggestions? These devices nodes are really the USB root hubs in
> > > > > the USB controller, so they could just have the USB number as the name
> > > > > like the other USB devices (001), but that's pretty boring :)
>
> Actually one of my criticisms of Greg's patch is that
> it hides the actual device tree. The root hub is easily
> distinguishable, it's the topmost one in the tree! There
> should be no need to name it specially.
>
> I'd really rather move away from the model which
> exposes a USB bus as a flat non-hierarchical
> setup, and move instead to a model reflects the
> actual topology of the USB devices and hubs.

<good description of what the topology tree should show snipped>

I completly agree. My main concern with these first patches, was
getting the driverfs - usb subsystem linkage working properly. I wasn't
trying to work on the names yet (you should have seen some of my testing
patches, the names there were pretty horrible :)

As we wait for the rest of Pat's patches to be merged into the kernel,
we can work on nailing down the proper naming scheme for the USB trees.

However, the root hub name _does_ propose a problem. I feel we have two
solutions:
        - use the bus number (usb_bus_00x)
          Pros:
                  matches the usbfs naming and directory structure
          Cons:
                  depends on the initialization order of the busses.

        - use a generic name like I did (usb_bus)
          Pros:
                  does not depend on the init order, and relies on the
                location in the entire pci topology tree to show its
                uniqueness.
          Cons:
                  boring :)

And also remember, the status file in a device's directory also provides
a _lot_ of information. We haven't even started to fill up the fields
there...

thanks,

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Thu Jan 31 2002 - 21:01:13 EST