(Please keep me CC'd as I am not subscribed to freebsd-usb) It seems recent commits/patches that people have commit have appended things to the bottom of the quirks list, but I don't see any comments indicating that their location is justified (e.g. probe order, etc.).
Is there a particular reason for the lack of alphabetic ordering in usb_quirk.c? Can they be re-alphabetised safely? The reason I ask: I was in the process of writing a USB quirk when I found that PR 153599 already addressed my needs. However, the vendor ID string ("Feiya") is no longer valid; Feiya was bought by Silicon Motion, Inc. in 2002: http://www.siliconmotion.com/A2.D_About_us.php?s=3 So the usbdevs entry for FEIYA should be changed to SILICONMOTION and refer to Silicon Motion, Inc. (which is both what USB-IF has as well as the Manufacturer string in a device itself (see below)). I found that the quirks were in alphabetical order based on vendor name (not vendor ID number), yet some weren't[2]. Reviewing CVS annotations and associated PRs with patches didn't provide any answers. Let me know so I can submit appropriate patches and update appropriate PRs. Thanks! [1]: ugen7.2: <Generic USB2.0 card Silicon Motion, Inc.> at usbus7, cfg=0 md=HOST spd=HIGH (480Mbps) pwr=ON bLength = 0x0012 bDescriptorType = 0x0001 bcdUSB = 0x0200 bDeviceClass = 0x0000 bDeviceSubClass = 0x0000 bDeviceProtocol = 0x0000 bMaxPacketSize0 = 0x0040 idVendor = 0x090c idProduct = 0x6200 bcdDevice = 0x0100 iManufacturer = 0x0001 <Silicon Motion, Inc.> iProduct = 0x0002 <Generic USB2.0 card > iSerialNumber = 0x0003 <12345678901234567890> bNumConfigurations = 0x0001 [2]: 448 USB_QUIRK(YEDATA, FLASHBUSTERU, 0x0081, 0xFFFF, UQ_MSC_FORCE_WIRE_CBI_I, 449 UQ_MSC_FORCE_PROTO_UFI, UQ_MSC_NO_RS_CLEAR_UA, UQ_MSC_FLOPPY_SPEED, 450 UQ_MSC_NO_GETMAXLUN), 451 USB_QUIRK(ZORAN, EX20DSC, 0x0000, 0xffff, UQ_MSC_FORCE_WIRE_CBI, 452 UQ_MSC_FORCE_PROTO_ATAPI), 453 USB_QUIRK(MEIZU, M6_SL, 0x0000, 0xffff, UQ_MSC_FORCE_WIRE_BBB, 454 UQ_MSC_FORCE_PROTO_SCSI, UQ_MSC_NO_INQUIRY, UQ_MSC_NO_SYNC_CACHE), 455 USB_QUIRK(ACTIONS, MP4, 0x0000, 0xffff, UQ_MSC_FORCE_WIRE_BBB, 456 UQ_MSC_FORCE_PROTO_SCSI, UQ_MSC_NO_SYNC_CACHE), 457 USB_QUIRK(ASUS, GMSC, 0x0000, 0xffff, UQ_MSC_NO_SYNC_CACHE), 458 USB_QUIRK(CHIPSBANK, USBMEMSTICK, 0x0000, 0xffff, UQ_MSC_NO_SYNC_CACHE), 459 USB_QUIRK(CHIPSBANK, USBMEMSTICK1, 0x0000, 0xffff, UQ_MSC_NO_SYNC_CACHE), 460 USB_QUIRK(NEWLINK, USB2IDEBRIDGE, 0x0000, 0xffff, UQ_MSC_NO_SYNC_CACHE), -- | Jeremy Chadwick j...@parodius.com | | Parodius Networking http://www.parodius.com/ | | UNIX Systems Administrator Mountain View, CA, USA | | Making life hard for others since 1977. PGP 4BD6C0CB | _______________________________________________ freebsd-usb@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/freebsd-usb To unsubscribe, send any mail to "freebsd-usb-unsubscr...@freebsd.org"