Page MenuHomeFreeBSD
Differential D17084

doc update for bcm238x_pwm(4) to reflect changes made to driver
ClosedPublic
Actions

Authored by bobf_mrp3.com on Sep 9 2018, 1:45 AM.
Tags
Referenced Files
F81660242: D17084.diff
Fri, Apr 19, 2:43 PM
Unknown Object (File)
Dec 20 2023, 2:37 AM
Unknown Object (File)
Nov 14 2023, 3:20 AM
Unknown Object (File)
Oct 13 2023, 2:21 AM
Unknown Object (File)
Sep 14 2023, 6:14 AM
Unknown Object (File)
Jul 15 2023, 8:50 PM
Unknown Object (File)
Jul 15 2023, 8:27 AM
Unknown Object (File)
Apr 21 2023, 1:10 AM
View All 9 Files
Subscribers
0mp
andrew
imp

Details

Reviewers
gonzo
0mp
Group Reviewers
manpages
Commits
rS338574: [bcm283x_pwm] Add documentation for the second PWM channel
Summary

a month or so ago changes were committed to the bcm283x_pwm driver but these changes weren't reflected in the documentation. This modifies the man page to reflect the changes.

Diff Detail

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

Event Timeline

bobf_mrp3.com created this revision.Sep 9 2018, 1:45 AM
Herald added a reviewer: manpages. · View Herald TranscriptSep 9 2018, 1:45 AM
Herald added subscribers: andrew, imp. · View Herald Transcript
gonzo accepted this revision.Sep 10 2018, 7:45 PM
This revision is now accepted and ready to land.Sep 10 2018, 7:45 PM
0mp accepted this revision as: 0mp.Sep 10 2018, 9:45 PM
0mp added a subscriber: 0mp.

Looks fine when it comes to mdoc syntax.

Do you think you could address the issues I pointed out in this revision as well?

Thanks!

share/man/man4/man4.arm/bcm283x_pwm.4
41 ↗(On Diff #47824)

Shouldn't we write Raspberry Pi instead of Rpi?

80 ↗(On Diff #47824)

Missing period.

88 ↗(On Diff #47824)

Not part of your changes but there's a period missing at the end of the sentence.

0mp added inline comments.Sep 10 2018, 9:55 PM
share/man/man4/man4.arm/bcm283x_pwm.4
29 ↗(On Diff #47824)

Also, remember to bump Dd.

Closed by commit rS338574: [bcm283x_pwm] Add documentation for the second PWM channel (authored by gonzo). · Explain WhySep 10 2018, 10:48 PM
This revision was automatically updated to reflect the committed changes.
gonzo added a comment.Sep 11 2018, 6:48 PM
In D17084#364832, @0mp wrote:

Looks fine when it comes to mdoc syntax.

Do you think you could address the issues I pointed out in this revision as well?

Thanks!

Thanks for review Mateusz,

I added your revisions to the commit

0mp added a comment.Sep 11 2018, 7:58 PM

Happy to help :)

Revision Contents
Changeset List

PathSize
head/
share/
man/
man4/
man4.arm/
45 lines

Diff 47900

View Options

head/share/man/man4/man4.arm/bcm283x_pwm.4

Loading...