This driver is for thermal sensor in Intel PCH chipset.
This exports temperature and some threshold temperature values via sysctl .
Details
- Reviewers
manu gonzo br - Group Reviewers
manpages - Commits
- rS359479: Add Platform Controller Hub built-in thermal management device driver.
Diff Detail
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 29995 Build 27808: arc lint + arc unit
Event Timeline
Nice! Looking forward to this :-).
share/man/man4/pchtherm.4 | ||
---|---|---|
51 | templature/s/temperature |
Nice. Tested on Kaby Lake-Y (Google Pixelbook, i7-7Y75).
Maybe only spam all the info into dmesg when bootverbose is on? I like cool new dmesg lines but 7 lines for a thermal sensor does seem like a bit too much.
share/man/man4/pchtherm.4 | ||
---|---|---|
75 | missing space "andunder" | |
sys/dev/intel/pchtherm.c | ||
89 | "Haswel" → "Haswell" | |
172 | alart? Alert? (this same mistake/typo is in all the places) | |
194 | "Throtting" → "Throttling" | |
234 | typo "Cucrrent" → "Current" | |
262 | typo "Catastorophic" → "Catastrophic" |
Fix typo.
Hide normal informational message under bootverbose.
Add comment on alert register handling.
share/man/man4/pchtherm.4 | ||
---|---|---|
70 | When reached ... will shutdown — sequence of tenses seems wrong here, suggest getting advice from a native speaker. Also, each sentence should start on a new line (after period). | |
74 | Should it be spelled as .Va t0temp? | |
76 | Should it be spelled as .Va t1temp? | |
79 | Missing newline after dot (end of sentence) and space before opening parenthesis. | |
81 | Missing newline after dot (end of sentence) and space before opening parenthesis. | |
83 | Sentence break => new line; should it be spelled as .Va pmtime (as this is a reference to adjacent variable)? | |
93 | Perhaps should be: read-only, do not support... | |
106 | I think you've also spelled your name wrong. :-) | |
sys/dev/intel/pchtherm.c | ||
75 | Should #include block be visually separated with an extra blank line? | |
102 | Local variables are typically separated from the function body with a blank line. | |
126 | Inconsistent vertical spacing in local variables block. | |
152 | Missing space after if and before { (ditto for the line below). | |
156 | Missing space after else. | |
161 | Missing spaces. | |
169 | Superfluous spaces (ditto below). | |
243 | Missing space after | (also below). | |
268 | Missing spaces again here and below. | |
290 | Functions should be visually separated from each other for better readability. | |
297 | Ditto (and below). |
Thanks. Couple of style bugs are still there (search for if( without the space), but I don't want to cherry pick each particular case and just leave it at your discretion.
It seems that you never actually added the files.x86 change.
At least, I do not see it.