Page MenuHomeFreeBSD

add manual pages for the superio driver and interface

Authored by avg on Oct 9 2019, 3:36 PM.



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

Diff Detail

rS FreeBSD src repository
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

avg created this revision.Oct 9 2019, 3:36 PM
bcr accepted this revision as: manpages.Oct 10 2019, 7:29 AM
bcr added a subscriber: bcr.

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

brueffer requested changes to this revision.Oct 10 2019, 7:37 AM
brueffer added a subscriber: brueffer.
brueffer added inline comments.
4 ↗(On Diff #63080)

This can go.

107 ↗(On Diff #63080)

This can be removed, it does nothing here.

mandoc -Tlint manpage.4 catches problems like this.

115 ↗(On Diff #63080)

missing space, "dev ,"

122 ↗(On Diff #63080)

Missing space, "SUPERIO_DEV_WDT ."

126 ↗(On Diff #63080)

Missing space before "."

149 ↗(On Diff #63080)

Missing space before "."

183 ↗(On Diff #63080)

.Pp does nothing here and should be removed.

187 ↗(On Diff #63080)

Trailing comma should be removed

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

cleanups based on the review

avg added a comment.Oct 10 2019, 8:07 AM

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

brueffer accepted this revision.Oct 10 2019, 8:09 AM

Looks good, thanks!

This revision is now accepted and ready to land.Oct 10 2019, 8:09 AM
avg added a comment.Oct 10 2019, 8:21 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.

avg updated this revision to Diff 63113.Oct 10 2019, 8:39 AM

add makefile changes

This revision now requires review to proceed.Oct 10 2019, 8:39 AM
avg updated this revision to Diff 63114.Oct 10 2019, 8:40 AM

man4 makefile too

yuripv added a subscriber: yuripv.Oct 10 2019, 11:19 PM
yuripv added inline comments.
93 ↗(On Diff #63114)

the vendor?

102 ↗(On Diff #63114)

the device ID?

107 ↗(On Diff #63114)

the revision ID?

159 ↗(On Diff #63114)

the DMA channel number?

164 ↗(On Diff #63114)

the base I/O port?

171 ↗(On Diff #63114)

the interrupt number?

176 ↗(On Diff #63114)

the Logical Device Number?

181 ↗(On Diff #63114)

the type?

avg added a comment.Oct 11 2019, 5:50 AM

@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.