Page MenuHomeFreeBSD
Differential D20652

Document the PCIOCATTACHED ioctl(2) in pci(4)
ClosedPublic
Actions

Authored by zeising on Jun 15 2019, 1:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Feb 5, 10:32 PM
Unknown Object (File)
Jan 14 2025, 10:20 PM
Unknown Object (File)
Jan 7 2025, 10:28 PM
Unknown Object (File)
Jan 6 2025, 1:29 PM
Unknown Object (File)
Dec 24 2024, 5:37 AM
Unknown Object (File)
Oct 7 2024, 4:54 PM
Unknown Object (File)
Oct 7 2024, 12:50 PM
Unknown Object (File)
Sep 29 2024, 9:49 PM
View All 76 Files
Subscribers
None

Details

Reviewers
imp
jhb
bcr
Group Reviewers
manpages
Commits
rS349606: MFC 349133 349146 349150: document PCIOCATTACHED
rS349605: MFC 349133 349146 349150: document PCIOCATTACHED
rS349133: pci(4): Document PCIOCATTACHED
Summary

Document the PCIOCATTACHED ioctl(2) in the pci(4) manual page, so that it is easier to find.

Diff Detail

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

Event Timeline

zeising created this revision.Jun 15 2019, 1:08 PM
Herald added a reviewer: manpages. · View Herald TranscriptJun 15 2019, 1:08 PM
Harbormaster completed remote builds in B24866: Diff 58664.Jun 15 2019, 1:08 PM
bcr added a comment.Jun 15 2019, 2:00 PM

Can you bump the .Dd when you commit it for this content change? Do an extra "mandoc -Tlint" and textproc/igor run over the man page just to be sure there is nothing that I missed in my pass. Thank you!

share/man/man4/pci.4
346 ↗(On Diff #58664)

s/aboove/above/

zeising updated this revision to Diff 58670.Jun 15 2019, 2:21 PM

Fixes based on comments from @bcr .
mandoc -Tlint complains a lot about useless macro: Tn for all the .Tn PCI. This style is all over the file, and should probably be changed in a separate pass.

Harbormaster completed remote builds in B24867: Diff 58670.Jun 15 2019, 2:21 PM
zeising marked an inline comment as done.Jun 15 2019, 2:22 PM
bcr accepted this revision.Jun 15 2019, 2:36 PM

I agree about the second pass for the macros. Good to go then!

This revision is now accepted and ready to land.Jun 15 2019, 2:36 PM
imp added inline comments.Jun 16 2019, 5:45 PM
share/man/man4/pci.4
350 ↗(On Diff #58670)

I think you need to add the word 'fields' before 'are' here.

351 ↗(On Diff #58670)

I'd suggest "The status of the device is stored in" instead of this and the next two lines. The return value of the ioctl(2) call is different and that phrase is likely to cause confusion.

zeising updated this revision to Diff 58712.Jun 16 2019, 6:21 PM

Updates based on comments from @imp and some wordsmithing.

This revision now requires review to proceed.Jun 16 2019, 6:21 PM
Harbormaster completed remote builds in B24885: Diff 58712.Jun 16 2019, 6:21 PM
zeising marked 2 inline comments as done.Jun 16 2019, 6:21 PM
imp accepted this revision.Jun 17 2019, 3:05 AM
This revision is now accepted and ready to land.Jun 17 2019, 3:05 AM
Closed by commit rS349133: pci(4): Document PCIOCATTACHED (authored by zeising). · Explain WhyJun 17 2019, 5:41 AM
This revision was automatically updated to reflect the committed changes.
zeising added a commit: rS349133: pci(4): Document PCIOCATTACHED.
zeising added a commit: rS349605: MFC 349133 349146 349150: document PCIOCATTACHED.Jul 2 2019, 5:23 PM
zeising added a commit: rS349606: MFC 349133 349146 349150: document PCIOCATTACHED.

Revision Contents
Changeset List

PathSize
head/
share/
man/
man4/
22 lines

Diff 58724

View Options

head/share/man/man4/pci.4

Loading...