Page MenuHomeFreeBSD

hwt(4) man page
ClosedPublic

Authored by br on Mon, Jul 7, 2:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jul 20, 4:50 AM
Unknown Object (File)
Thu, Jul 17, 7:33 PM
Unknown Object (File)
Wed, Jul 9, 4:02 PM
Subscribers

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

br requested review of this revision.Mon, Jul 7, 2:50 PM
br created this revision.

Fix issues found by mandoc lint

I've been so excited for this! Awesome! Here's what I could find so far:

share/man/man4/hwt.4
2

You can use the SPDX short form as shown in style.9 or https://docs.freebsd.org/en/articles/license-guide/

30

This shouldn't be quoted

32–59

AIUC?

63

So, hwt is a framework, and coresight or pt are the drivers?

64

Roff wants each sentence on it's own line

77
87
88

Macros in argument width is an undefined behavior that misbehaves

90
92–93
99
101
109
127

For maintainability

128–144

Ioctls are usually Dv's afaict. I should probably put that in style.mdoc(5).

130
145–146
br marked 16 inline comments as done.

Address @ziaee comments

share/man/man4/hwt.4
32–59

You don't like this?

share/man/man4/hwt.4
32–59

yeah, it does not outline that HWT_HOOKS is mandatory option. Also not sure what is AUIC

63

Yes

share/man/man4/hwt.4
32–59

Okay, thanks. "AIUC" is "am I understanding correctly".

Can we describe it in DESCRIPTION?

I'm trying to increase the consistency and predictability of the manual. I gave a 5 min lightning talk about it at BSDCan.

https://people.freebsd.org/~ziaee/tmp/bsdcan25lightning-ziaee.webm

share/man/man4/hwt.4
32–59

Can you point to a similar man page as an example ?

https://man-dev.freebsd.org/vt
And in current (not synced to web interface yet), check out uftdi, uchcom, or pf

https://man-dev.freebsd.org/vt
And in current (not synced to web interface yet), check out uftdi, uchcom, or pf

Thanks. Could you take a look once again and accept once happy ?

Just this one stupid nit, thank you.
I wonder if we should MLINKS this page to the drivers, spe/coresight/pt?

share/man/man4/hwt.4
2
This revision is now accepted and ready to land.Wed, Jul 9, 12:44 PM

Add hwt.4 to the build and create MLINKS

This revision now requires review to proceed.Wed, Jul 9, 1:11 PM
This revision was not accepted when it landed; it landed in state Needs Review.Wed, Jul 9, 4:00 PM
This revision was automatically updated to reflect the committed changes.