Page MenuHomeFreeBSD
Differential D15999

Create PCI_MATCH and pci_match_device
ClosedPublic
Actions

Authored by imp on Jun 24 2018, 6:59 PM.
Tags
None
Referenced Files
F142352650: D15999.diff
Sun, Jan 18, 10:50 PM
Unknown Object (File)
Dec 20 2025, 7:32 AM
Unknown Object (File)
Dec 11 2025, 4:15 PM
Unknown Object (File)
Dec 7 2025, 11:22 PM
Unknown Object (File)
Nov 25 2025, 9:09 PM
Unknown Object (File)
Nov 21 2025, 12:36 AM
Unknown Object (File)
Nov 21 2025, 12:32 AM
Unknown Object (File)
Nov 21 2025, 12:31 AM
View All Files
Subscribers
adrian
chuck
lakhanshiva_gmail.com
rpokala

Details

Reviewers
None
Commits
rS336066: Create PCI_MATCH and pci_match_device
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 Passed
Unit
No Test Coverage
Build Status
Buildable 17595
Build 17408: arc lint + arc unit

Event Timeline

imp created this revision.Jun 24 2018, 6:59 PM
Harbormaster completed remote builds in B17595: Diff 44394.Jun 24 2018, 6:59 PM
imp updated this revision to Diff 44395.Jun 24 2018, 7:05 PM
  • Create PCI_MATCH and pci_match_device
  • Switch to using new PCI_MATCH stuff.
Harbormaster completed remote builds in B17597: Diff 44395.Jun 24 2018, 7:05 PM
adrian added a subscriber: adrian.Jun 24 2018, 7:38 PM

yaaaaayayaya do it~!

imp added inline comments.Jun 25 2018, 7:32 PM
sys/dev/ath/if_ath_pci_devlist.h
31

Need to mark the module still for PNP info

sys/dev/pci/pci.c
6248

This needs comments and a man page.

rpokala added a subscriber: rpokala.Jun 25 2018, 7:55 PM
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
6249

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

imp updated this revision to Diff 44573.Jun 28 2018, 5:52 AM

update, per jhb feedback

  • Create PCI_MATCH and pci_match_device
  • Switch to using new PCI_MATCH stuff.
Harbormaster completed remote builds in B17689: Diff 44573.Jun 28 2018, 5:52 AM
lakhanshiva_gmail.com added a subscriber: lakhanshiva_gmail.com.Jun 28 2018, 8:00 AM
chuck added a subscriber: chuck.Jun 28 2018, 4:48 PM
chuck added inline comments.Jun 28 2018, 4:52 PM
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
Closed by commit rS336066: Create PCI_MATCH and pci_match_device (authored by imp). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
sys/
dev/
ath/
86 lines
2 lines
bktr/
4 lines
14 lines
18 lines
pci/
41 lines
60 lines

Diff 44394

View Options

sys/dev/ath/if_ath_pci.c

Loading...
View Options

sys/dev/ath/if_ath_pci_devlist.h

Loading...
View Options

sys/dev/bktr/bktr_core.c

Loading...
View Options

sys/dev/bktr/bktr_os.c

Loading...
View Options

sys/dev/bktr/bktr_reg.h

Loading...
View Options

sys/dev/pci/pci.c

Loading...
View Options

sys/dev/pci/pcivar.h

Loading...