Page MenuHomeFreeBSD
Differential D15995

Add PNP info to PCI attachment of alc, amdsmb, bfe, bge, ce, dpt, dc, ciss, cp drivers
ClosedPublic
Actions

Authored by lakhanshiva_gmail.com on Jun 24 2018, 10:14 AM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 22 2024, 10:56 PM
Unknown Object (File)
Nov 11 2024, 2:46 AM
Unknown Object (File)
Nov 9 2024, 12:26 PM
Unknown Object (File)
Nov 5 2024, 8:03 PM
Unknown Object (File)
Oct 26 2024, 4:06 PM
Unknown Object (File)
Oct 23 2024, 7:08 PM
Unknown Object (File)
Oct 21 2024, 8:18 PM
Unknown Object (File)
Oct 17 2024, 6:11 PM
View All Files
Subscribers
lakhan

Details

Reviewers
imp
chuck
Commits
rS336106: Add PNP info to PCI attachments of ciss driver
rS336105: Add PNP info to PCI attachment of dc driver
rS336104: Add PNP info to PCI attachments of bge driver
rS336103: Add PNP info to PCI attachments of bfe driver
Summary

Update PNP info to PCI attachments of amdsmb driver

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

lakhanshiva_gmail.com created this revision.Jun 24 2018, 10:14 AM
Harbormaster completed remote builds in B17588: Diff 44383.Jun 24 2018, 10:14 AM
lakhanshiva_gmail.com retitled this revision from Summary: Add PNP info to PCI attachment of alc driver to Add PNP info to PCI attachment of alc, amdsmb, bfe, bge, ce, dpt, dc, ciss, cp drivers.Jun 24 2018, 10:16 AM
lakhanshiva_gmail.com edited the summary of this revision. (Show Details)
chuck requested changes to this revision.Jun 24 2018, 1:16 PM
chuck added inline comments.
sys/dev/amdsmb/amdsmb.c
151 ↗(On Diff #44383)

style nit, space between key word (if) and open parenthesis. Also needed on either side of comparator above (i.e. i < nitems(amdsmb_devs). Please fix here and check all other code modifications for this type of error.

sys/dev/bfe/if_bfe.c
160 ↗(On Diff #44383)

Should this have a description? If so, please fix here and check other code changes for this type of issue.

sys/dev/ciss/ciss.c
207 ↗(On Diff #44383)

Instead of moving this whole block below the ciss_vendor_data[] definitions, is it possible to only move the DRIVER_MODULE() and perhaps the MODULE_DEPEND() definitions to coincide with the new MODULE_PNP_INFO?

This revision now requires changes to proceed.Jun 24 2018, 1:16 PM
lakhanshiva_gmail.com added a comment.Jun 24 2018, 2:14 PM

Thanks Chuck, I will make the changes.

imp added a comment.Jun 24 2018, 7:06 PM

Please see https://reviews.freebsd.org/D15999 for a notion on how we might want to cut over the medium drivers to being table driven.

lakhanshiva_gmail.com added a comment.Jun 24 2018, 7:10 PM

If I m not wrong this approach is specifically for the atheros drivers.
Isn't it ?

imp added a comment.Jun 25 2018, 3:33 AM
In D15995#338704, @lakhanshiva_gmail.com wrote:

If I m not wrong this approach is specifically for the atheros drivers.
Isn't it ?

Why yes. Yes it is.

Or rather, it's inspired by it. And by the USB stuff, and by what Linux does. And it also takes a page from PC Card, where it's possible to have more complicated meta-data associated with the device than is otherwise allowed with the USB pattern.

lakhanshiva_gmail.com updated this revision to Diff 44615.Jun 29 2018, 2:32 AM
  • Update PNP info to PCI attachment of amdsmb driver
  • Update PNP info to PCI attachments of bfe driver
  • Update PNP info to PCI attachment of ce driver
  • Update PNP info to PCI attachment of dpt driver
  • Update PNP info for PCI attachment of cp driver
Harbormaster completed remote builds in B17713: Diff 44615.Jun 29 2018, 2:32 AM
lakhanshiva_gmail.com marked 2 inline comments as done.Jun 29 2018, 2:34 AM
lakhanshiva_gmail.com updated this revision to Diff 44616.Jun 29 2018, 2:41 AM
  • Update PNP info to PCI attachments of ciss driver
Harbormaster completed remote builds in B17714: Diff 44616.Jun 29 2018, 2:41 AM
lakhanshiva_gmail.com marked an inline comment as done.Jun 29 2018, 2:42 AM

Thanks for the comments chuck! Made the changes.

lakhanshiva_gmail.com updated this revision to Diff 44618.Jun 29 2018, 3:25 AM
lakhanshiva_gmail.com edited the summary of this revision. (Show Details)
lakhanshiva_gmail.com updated this revision to Diff 44715.Jul 1 2018, 11:01 AM
lakhanshiva_gmail.com edited the summary of this revision. (Show Details)
  • Update PNP info to PCI attachment of amdsmb driver
Harbormaster completed remote builds in B17761: Diff 44715.Jul 1 2018, 11:01 AM
lakhanshiva_gmail.com updated this revision to Diff 44811.EditedJul 3 2018, 4:35 PM
lakhanshiva_gmail.com edited the summary of this revision. (Show Details)

Converted spaces to tabs (in probe functions)

chuck added inline comments.Jul 4 2018, 8:31 PM
sys/dev/dpt/dpt_pci.c
84 ↗(On Diff #44811)

This line still appears to uses spaces

lakhanshiva_gmail.com marked an inline comment as done.Jul 5 2018, 1:05 AM
lakhanshiva_gmail.com added inline comments.
sys/dev/dpt/dpt_pci.c
84 ↗(On Diff #44811)

Converted them to tabs

lakhanshiva_gmail.com marked 2 inline comments as done.Jul 5 2018, 1:05 AM
This revision was not accepted when it landed; it landed in state Needs Review.Jul 8 2018, 8:40 PM
Closed by commit rS336103: Add PNP info to PCI attachments of bfe driver (authored by imp). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
bfe/
2 lines

Diff 45030

View Options

head/sys/dev/bfe/if_bfe.c

Loading...