Re: [PATCH] USB: usbfs: deprecate and hide option for !embedded

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

 



On Sat, Apr 25, 2009 at 12:35:12AM +0200, Kay Sievers wrote:
> On Sat, Apr 25, 2009 at 00:12, Greg KH <gregkh@xxxxxxx> wrote:
> > On Fri, Apr 24, 2009 at 12:11:33PM -0400, Alan Stern wrote:
> >> On Thu, 23 Apr 2009, Greg KH wrote:
> >>
> >> > > Greg, any objections to rip out the usb_host class?
> >> >
> >> > I see the "companion" attribute still present, but if Alan says it's not
> >> > needed, hey, no objection from me :)
> >>
> >> No, that's not what I said.  It is still needed, but it doesn't have to
> >> go in the class directory.  It can go in the device directory for the
> >> host controller instead; then there won't be anything remaining in the
> >> usb_host class.
> >
> > Hm, I just noticed that the wireless usb controllers put a number of
> > files in the usb_host directory that they reside in.  Kay, care to poke
> > into this to see if it's really needed or not?
> 
> It looks to me they should also move where they belong, to the real
> usb_host device on the usb bus, and not in some disconnected class
> device. If it really causes issues for anybody, we can always create a
> symlink from the usb_host directory to the real device. But I'd rather
> remove that whole thing if possible.

I agree, I'd like to remove it as well.  But we need to check with the
wireless usb people first.

> >>       4. Move it to debugfs.
> >
> > I like this option.  So much so I just went and implemented it, I'll
> > post the patch series to do this to linux-usb in a bit :)
> 
> Sounds great. Then people can do this, if they are crazy enough. :)
>   touch /dev/bus/usb/devices
>   mount --bind /sys/kernel/debug/<some name> /dev/bus/usb/devices

Heh, that would be crazy :)

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