Page MenuHomeFreeBSD

add manual pages for the superio driver and interface
ClosedPublic

Authored by avg on Oct 9 2019, 3:36 PM.
Tags
None
Referenced Files
F103483086: D21958.diff
Mon, Nov 25, 2:51 PM
F103471453: D21958.id63114.diff
Mon, Nov 25, 11:23 AM
F103424512: D21958.id63080.diff
Sun, Nov 24, 8:41 PM
Unknown Object (File)
Sat, Nov 23, 5:45 PM
Unknown Object (File)
Sat, Nov 23, 3:35 PM
Unknown Object (File)
Sat, Nov 23, 2:31 PM
Unknown Object (File)
Fri, Nov 22, 8:41 PM
Unknown Object (File)
Fri, Nov 22, 1:27 PM

Details

Summary

This adds basic documentation on what the superio driver is and how other
drivers can interact with it. I decided to also document superio's ivar
accessors.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 26973
Build 25273: arc lint + arc unit

Event Timeline

bcr added a subscriber: bcr.

The man pages look fine to me, thanks for writing them.

brueffer added a subscriber: brueffer.
brueffer added inline comments.
share/man/man4/superio.4
5

This can go.

108

This can be removed, it does nothing here.

mandoc -Tlint manpage.4 catches problems like this.

share/man/man9/superio.9
116

missing space, "dev ,"

123

Missing space, "SUPERIO_DEV_WDT ."

127

Missing space before "."

150

Missing space before "."

184

.Pp does nothing here and should be removed.

188

Trailing comma should be removed

This revision now requires changes to proceed.Oct 10 2019, 7:37 AM
avg marked 8 inline comments as done.

cleanups based on the review

Thank you very much for the review!
Problems should be fixed now.

This revision is now accepted and ready to land.Oct 10 2019, 8:09 AM

Hmm, I see that superio.9 could become a first conditionally installed manual page in man9.
Is that okay?
Or would it be better to merge superio.9 into superio.4?
I see a couple of examples in man4, e.g., cpufreq.4.

This revision now requires review to proceed.Oct 10 2019, 8:39 AM
yuripv added inline comments.
share/man/man9/superio.9
94

the vendor?

103

the device ID?

108

the revision ID?

160

the DMA channel number?

165

the base I/O port?

172

the interrupt number?

177

the Logical Device Number?

182

the type?

@yuripv , I noted that you ended your suggestions with question marks...
I am not a native speaker, so I am not sure if definite or indefinite article is more appropriate in those sentences.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 11 2019, 11:13 AM
This revision was automatically updated to reflect the committed changes.