Update iic(4) man page to reflect recent changes

[jah]

Summary

Update iic.4 to reflect new I2CSADDR ioctl and per-fd addressing state

Test Plan

Made sure things looked sane w/ 'man ./iic.4'

[wblock]

Run 'igor -R iic.4 | less -RS' (from textproc/igor) and 'mandoc -tlint iic.4' to check for various errors.

share/man/man4/iic.4

28

Remember to update this on commit.

56–57

Not your fault, but please fix this run-on, halting, comma-less sentence:

A read operation is initiated when the read/write bit is set, or a write operation when it is cleared.

60–62

The new sentence should start on a new line.

Use active voice when possible:

If successful, this command acquires

Can this sentence be simplified? There are a lot of trailing ofs and fors that make it kind of confusing. For example (maybe):

If successful, exclusive ownership of the specified file descriptor's underlying iicbus instance is acquired.

70–72

Again, start a new sentence on a new line.

72

"the I2CSTART command" might be redundant here. It could be just

...succeed,
.Dv I2CSTART
must have...

78

As above about "the I2CSTART command".

"this command will" is passive. Consider:

...file descriptor, the current transaction is terminated and exclusive ownership of the underlying iicbus instance is released.

80

New sentence on a new line. Also, passive->active:

Otherwise, no action is performed.

88

Good on not saying "the I2CSTART command"! (But new sentence on new line.)

89

"this command" and passive voice can be avoided:

issued, exclusive ownership of the underlying iicbus instance is released.

97

"command" is not needed. Agreed that "must" is better than "should" here.

117

"command" not needed.

137

"the ... command" not needed.

170

s/Associates/Associate/

176

"should" is a recommendation, not a requirement. But this is probably a requirement, and "is" or "must" are better for those:

The argument is an 8-bit address (that is, a 7-bit address << 1).

(i.e. and e.g. are academic things. Here, it probably should have been e.g., meaning "that is" rather than i.e.'s "for example". Best to avoid the Latin versions when possible.)

214

"the ... calls" can be removed:

It is also possible to use
.Xr read 2
or
.Xr write 2 ,
in which case the I2C start/stop handshake is
managed by
.Xr iicbus 4 .

225

Passive->active:
s/will clear/clears/

229

.Dv I2CSADDR ,

230

Passive->active:

stops any transaction established by a not-yet-terminated
.Dv I2CSTART
and releases iicbus ownership.

(.Pq can be used for sections in parentheses, but they are asides which are best avoided.)

236

Start new sentence on new line. Also, passive->active:

Concurrent transactions on those descriptors are

[jah]

Addressing wblock's feedback

[wblock]

Nice! Man page changes look good to me.