Page MenuHomeFreeBSD

Create PCI_MATCH and pci_match_device
ClosedPublic

Authored by imp on Jun 24 2018, 6:59 PM.

Details

Summary

Don't use generic PCI_VENDOR and PCI_PRODUCT macros. Prefix them with BKTR_ to avoid possible conflicts.

Create a covenience function to match PCI device IDs. It's about 15
years overdue.

Switch to using new PCI_MATCH stuff.

Diff Detail

Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 17689
Build 17494: arc lint + arc unit

Event Timeline

  • Create PCI_MATCH and pci_match_device
  • Switch to using new PCI_MATCH stuff.
sys/dev/ath/if_ath_pci_devlist.h
31

Need to mark the module still for PNP info

sys/dev/pci/pci.c
6247

This needs comments and a man page.

rpokala added inline comments.
sys/dev/ath/if_ath_pci.c
85–86

If this is ath-specific, please prefix with ATH_ ; if not, shouldn't it be in one of the PCI files? (Same comments for PCI_DEVICE_SUB()

sys/dev/pci/pci.c
6248

"elt", "nelt" -- ltrs rn't xpensv nymr.

update, per jhb feedback

  • Create PCI_MATCH and pci_match_device
  • Switch to using new PCI_MATCH stuff.
sys/dev/pci/pcivar.h
310

Does this universally ditch the device description?

This revision was not accepted when it landed; it landed in state Needs Review.Jul 7 2018, 3:25 PM
This revision was automatically updated to reflect the committed changes.