Add Cavium ThunderX xHCI controller PCI ID

[emaste]

Summary

See P74 for the full pciconf -lv from the system

none1@pci0:0:16:0:      class=0x0c0330 card=0xa11b177d chip=0xa01b177d rev=0x01 hdr=0x00
    vendor     = 'Cavium, Inc.'
    device     = 'THUNDERX xHCI USB Controller'
    class      = serial bus
    subclass   = USB
none2@pci0:0:17:0:      class=0x0c0330 card=0xa11b177d chip=0xa01b177d rev=0x01 hdr=0x00
    vendor     = 'Cavium, Inc.'
    device     = 'THUNDERX xHCI USB Controller'
    class      = serial bus
 subclass = USB

[• hselasky]

You are sure this device does not need any port routing quirks?

[emaste]

sys/dev/usb/controller/xhci_pci.c

116–118 ↗

(On Diff #8755)

One question - the sorting here is going to get awkward as we add more xHCI controllers. Should we sort alphabetically by mfgr name and then by device_id? We do want all of the mfgr entries to remain together I'd say, so sorting just by device_id is insufficient. Sorting by "lower 16 bytes of device_id then by upper 16 bits" would work but is awkward.

[• hselasky]

sys/dev/usb/controller/xhci_pci.c

116–118 ↗

(On Diff #8755)

The compiler usually sorts the entries, though sorting first by the lower 16-bits then the higher 16-bits is fine by me. This should be done as a separate commit. It would be really cool if these PCI strings were provided the PCI space, instead of by hand :-)

--HPS

[emaste]

The compiler usually sorts the entries, though sorting first by the lower 16-bits then the higher 16-bits is fine by me.

Right, I'm just talking about sorting for code maintainability. It will get to be a real mess if we have entries ending up in arbitrary order.