Page MenuHomeFreeBSD

Add PNP info to PCI attachment of arcmsr, bt, cbb, de, ebus, emujoy, fixup_pci, hifn, hptiop, imcsmb, snd_emu10k1 (emu_pci) drivers
ClosedPublic

Authored by lakhanshiva_gmail.com on Jun 24 2018, 12:07 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 26, 3:17 AM
Unknown Object (File)
Thu, Mar 21, 8:22 AM
Unknown Object (File)
Mon, Mar 11, 5:50 PM
Unknown Object (File)
Sat, Mar 9, 9:14 AM
Unknown Object (File)
Feb 22 2024, 10:20 PM
Unknown Object (File)
Feb 22 2024, 1:58 PM
Unknown Object (File)
Feb 11 2024, 1:00 PM
Unknown Object (File)
Feb 1 2024, 1:22 PM
Subscribers

Details

Reviewers
imp
chuck
Summary

Add PNP info to PCI attachment of hptiop driver

Add PNP info to PCI attachment of arcmsr driver

Add PNP info to PCI attachment of hifn driver

Add PNP INFO to PCI attachment of bt driver

Add PNP info to PCI attachment of emujoy, snd_emu10k1 (emu_pci) drivers

Add PNP info to PCI attachments of ebus driver

Add PNP info to PCI attachments of cbb driver

Add PNP info to PCI attachments of de driver

Add PNP info to PCI attachments of fixup_pci driver

Add PNP info to PCI attachment of imcsmb driver

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 17685
Build 17490: arc lint + arc unit

Event Timeline

  • Update PNP info to PCI attachment of arcmsr driver
  • Update PNP info to PCI attachment of hptiop driver
  • Update PNP INFO to PCI attachment of bt driver
  • Update PNP info to PCI attachment of emujoy, snd_emu10k1 (emu_pci) drivers
  • Update PNP info to PCI attachments of cbb driver
  • Update PNP info to PCI attachments of fixup_pci driver
  • Update PNP info to PCI attachment of imcsmb driver
  • Update PNP info to PCI attachments of ebus driver
  • Update PNP INFO to PCI attachment of bt driver
  • Update PNP info to PCI attachment of arcmsr driver
  • Update PNP INFO to PCI attachment of bt driver
  • Update PNP info to PCI attachment of emujoy, snd_emu10k1 (emu_pci) drivers
sys/dev/buslogic/bt_pci.c
121–135

This doesn't look right at all...

  • Update PNP info to PCI attachment of arcmsr driver
  • Update PNP info to PCI attachment of hifn driver
  • Update PNP INFO to PCI attachment of bt driver
  • Update PNP info to PCI attachments of ebus driver
sys/dev/buslogic/bt_pci.c
126

Since the bt_devs[] is null terminated, shouldn't the bounds check be nitems() instead of nitems() - 1?

sys/dev/hptiop/hptiop.c
1857

Should this get converted to using the new table of devices?

sys/sparc64/ebus/ebus.c
285

Should the check be for nitems()?

sys/dev/buslogic/bt_pci.c
126

So, in this case nitems(bt_devs) comes out to be 3, so the for loop is for(i=0; i<2; i++). This iterates through the two valid devices present in bt_devs. It doesn't check the sentinel. This is what i wanted to iterate on. Does it sound right ?

lakhanshiva_gmail.com updated this revision to Diff 44892.
lakhanshiva_gmail.com edited the summary of this revision. (Show Details)
lakhanshiva_gmail.com edited the summary of this revision. (Show Details)
lakhanshiva_gmail.com added inline comments.
sys/dev/hptiop/hptiop.c
1857

Made the probe routine table driven. Same in other cases where it is necessary.

This revision is now accepted and ready to land.Dec 31 2018, 7:13 PM