Page MenuHomeFreeBSD
Differential D14055

jedec_ts(4) is broken on stable/10
ClosedPublic
Actions

Authored by rpokala on Jan 26 2018, 12:39 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 26, 12:57 PM
Unknown Object (File)
Tue, Nov 26, 9:56 AM
Unknown Object (File)
Tue, Nov 26, 9:41 AM
Unknown Object (File)
Sun, Nov 24, 4:56 PM
Unknown Object (File)
Tue, Nov 19, 6:44 PM
Unknown Object (File)
Tue, Nov 19, 5:11 PM
Unknown Object (File)
Tue, Nov 19, 10:04 AM
Unknown Object (File)
Tue, Nov 19, 7:28 AM
View All 57 Files
Subscribers
imp

Details

Reviewers
avg
Commits
rS329015: jedec_ts(4) uses a sysctl format specifier of "IK4", to indicate that it
Summary

jedec_ts(4) uses a sysctl format specifier of "IK4", to indicate that it
reports milliKelvin. However, sysctl(8) on stable/10 only knows about "IK",
without a numeric suffix, which represents deciKelvin.

Adjust the format specifier, and round the temperature value reported.

Test Plan

Load jedec_ts(4) on stable/10; note that the temperature reported is reasonable.

Diff Detail

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

Event Timeline

rpokala created this revision.Jan 26 2018, 12:39 AM
Herald added a subscriber: imp. · View Herald TranscriptJan 26 2018, 12:39 AM
Harbormaster completed remote builds in B14584: Diff 38466.Jan 26 2018, 12:39 AM
avg added inline comments.Jan 31 2018, 11:54 AM
sys/dev/jedec_ts/jedec_ts.c
197 ↗(On Diff #38466)

Would (temp + 500) / 1000 work just well here?

rpokala added inline comments.Jan 31 2018, 3:13 PM
sys/dev/jedec_ts/jedec_ts.c
197 ↗(On Diff #38466)

Yes, of course. That was the exact one-liner idiom I was thinking of, but couldn't remember at the time.

rpokala updated this revision to Diff 38720.Jan 31 2018, 3:16 PM

Use the obvious one-liner, which somehow escaped me at the time of the
original diff. Thanks, avg@!

Harbormaster completed remote builds in B14700: Diff 38720.Jan 31 2018, 3:16 PM
avg accepted this revision.Jan 31 2018, 3:31 PM

LGTM.
Please fix the whitespace issue.

sys/dev/jedec_ts/jedec_ts.c
197 ↗(On Diff #38466)

Thanks! Seems like indentation of this line is incorrect now (spaces vs tabs).

This revision is now accepted and ready to land.Jan 31 2018, 3:31 PM
rpokala updated this revision to Diff 38723.Jan 31 2018, 4:00 PM

Leading 8-space -> hard-tab. Whoops!

This revision now requires review to proceed.Jan 31 2018, 4:00 PM
Harbormaster completed remote builds in B14702: Diff 38723.Jan 31 2018, 4:00 PM
rpokala marked 3 inline comments as done.Jan 31 2018, 4:01 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 8 2018, 9:24 AM
Closed by commit rS329015: jedec_ts(4) uses a sysctl format specifier of "IK4", to indicate that it (authored by rpokala). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
stable/
10/
sys/
dev/
jedec_ts/
6 lines

Diff 39038

View Options

stable/10/sys/dev/jedec_ts/jedec_ts.c

Loading...