Page MenuHomeFreeBSD

Improve formatting of efidev(4)
ClosedPublic

Authored by 0mp on Aug 17 2018, 11:41 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 30, 3:39 AM
Unknown Object (File)
Tue, Apr 30, 3:37 AM
Unknown Object (File)
Tue, Apr 30, 3:37 AM
Unknown Object (File)
Tue, Apr 30, 3:37 AM
Unknown Object (File)
Mon, Apr 29, 7:58 PM
Unknown Object (File)
Sun, Apr 28, 9:07 AM
Unknown Object (File)
Feb 27 2024, 5:29 AM
Unknown Object (File)
Jan 7 2024, 10:56 AM
Subscribers

Details

Summary

Changes:

  • Move some information out of the SYNOPSIS section because it is formated in a broken way by mandoc(1) otherwise.
  • Improve the formatting of the list of provided ioctls.
  • Use "Fa" for struct fields.
  • Pet mandoc and igor.

Also:

  • It looks like efirt(9) is missing.
  • /dev/efi does not have a description in the FILES section altough a -tags list is used.
Test Plan

igor & mandoc: OK

Diff Detail

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

Event Timeline

Can you define "missing"? =9 I had to double check, but I did seem to commit both the Makefile bits and the page itself- https://svnweb.freebsd.org/base/head/share/man/man9/efirt.9

share/man/man4/efidev.4
67 ↗(On Diff #46830)

I think this looks kinda goofy if we maintain how I had it in SYNOPSIS, perhaps better if the two paragraphs were joined in a way that feels less redundant.

111 ↗(On Diff #46830)

Typo in both your version and mine, should read data

  • Fix a typo (s/aata/data)
  • Reword the part about disabling the driver via a tunable and put it into the SYNOPSIS section. Avoid the Nm macro, which causes funny formatting issues.
  • Move the note about the CPU architecture limitations to the BUGS section, which is defined as a section for limitations (according to mdoc(7)).
0mp marked an inline comment as done.Aug 17 2018, 12:53 PM

Oh I see now! Sorry, I wasn't looking in the right place for the efirt manpage.

This revision is now accepted and ready to land.Aug 17 2018, 2:23 PM
0mp requested review of this revision.Aug 20 2018, 8:26 PM
0mp added reviewers: mat, krion.
0mp added subscribers: krion, mat.

Thanks for reviewing those changes, @kevans!

@mat, @krion, it looks like everything is fine. May I ask you for an approval to commit?

This revision is now accepted and ready to land.Aug 20 2018, 8:56 PM
This revision was automatically updated to reflect the committed changes.