Re: [PATCH][usbutils] lsusb: port to hwdb

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

 



On Sun, Jul 21, 2013 at 05:49:05PM +0200, Kay Sievers wrote:
> On Sun, Jul 21, 2013 at 5:36 PM, Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> > On Sun, Jul 21, 2013 at 01:05:31PM +0200, Kay Sievers wrote:
> >> > However, hwdb only contains vendor, product, class, subclass and
> >> > protocol. So if you drop usb.ids the rest of the information will be
> >> > lost.
> >> >
> >> > Maybe split the rest out into a separate file and ship only that?
> >> >
> >> > Or is there a way to get this info into hwdb? Kay?
> >>
> >> It should work to add some of that data to the existing modalias,
> >> right? For some things we probably need to "invent" new synthetic
> >> modaliases to query these strings. We should give it a try, I think.
> >> Having lsusb shipping a private file only for that seems ugly.
> >
> > These are things that are just described in the USB-IF specifications,
> > (HID table commands, class codes, etc.) and really should not be part of
> > any "hardware database" as they don't change, and come directly from the
> > spec.
> >
> > I can just move those into the usbutils package, and get them removed
> > from the usb.ids file, sound good?
> 
> Sounds great to me, yeah.
> 
> You are right, I think, the reason we *can* stuff it into the hwdb, is
> probably not a good enough reason to do it. lsusb would very likely be
> the only user ever.

Ok, I'll do that this week, and work with the owner of usb.ids to see if
we can just remove them entirely from the file.

Thanks again Tom for this patch, much appreciated.

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